Fix memory leak when using screensharing on macOS (#133)

Author: alexlapa

CHANGELOG.md

@@ -25,13 +25,15 @@ All user visible changes to this project will be documented in this file. This p
 ### Fixed
 
 - Video renderer stretching a picture after rotation. ([#124])
+- Screen sharing leaking memory on [macOS]. ([#133])
 
 [#117]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/117
 [#119]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/119
 [#120]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/120
 [#123]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/123
 [#124]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/124
 [#125]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/125
+[#133]: https://github.com/instrumentisto/medea-flutter-webrtc/pull/133
 [116.0.5845.110]: https://github.com/instrumentisto/libwebrtc-bin/releases/tag/116.0.5845.110
 
 

Cargo.lock

@@ -0,0 +1,33 @@
+#pragma once
+
+#if __APPLE__
+
+#include "modules/desktop_capture/mouse_cursor_monitor.h"
+
+namespace bridge {
+
+// Captures mouse shape and position.
+// Wraps `webrtc::MouseCursorMonitor` wrapping `Capture` calls with @autorelease
+// block to prevent memory leaking.
+class MouseCursorMonitorMac : public webrtc::MouseCursorMonitor {
+ public:
+  // Creates a new `MouseCursorMonitorMac`.
+  MouseCursorMonitorMac(
+      std::unique_ptr<webrtc::MouseCursorMonitor> mouse_monitor);
+
+  // Initializes the monitor with the `callback`, which must remain valid until
+  // the capturer is destroyed.
+  virtual void Init(Callback* callback, Mode mode) override;
+
+  // Captures the current cursor shape and position
+  void Capture() override;
+
+ private:
+  // Inner `MouseCursorMonitor` that this `MouseCursorMonitorMac` delegates
+  // calls to.
+  std::unique_ptr<webrtc::MouseCursorMonitor> mouse_monitor_;
+};
+
+}  // namespace bridge
+
+#endif