Use std::unique_ptr<> to pass frame ownership in DesktopCapturer impls.

webrtc/modules/desktop_capture/win/screen_capturer_win_magnifier.h

Index: webrtc/modules/desktop_capture/win/screen_capturer_win_magnifier.h
diff --git a/webrtc/modules/desktop_capture/win/screen_capturer_win_magnifier.h b/webrtc/modules/desktop_capture/win/screen_capturer_win_magnifier.h
index 623c8a3003432602c080a9c196718e1464852701..a5d67aee0d5fa5271111984f96d38aed23d11f55 100644
--- a/webrtc/modules/desktop_capture/win/screen_capturer_win_magnifier.h
+++ b/webrtc/modules/desktop_capture/win/screen_capturer_win_magnifier.h
@@ -106,12 +106,12 @@ class ScreenCapturerWinMagnifier : public ScreenCapturer {
   static Atomic32 tls_index_;
 
   std::unique_ptr<ScreenCapturer> fallback_capturer_;
-  bool fallback_capturer_started_;
-  Callback* callback_;
+  bool fallback_capturer_started_ = false;
+  Callback* callback_ = nullptr;
   std::unique_ptr<SharedMemoryFactory> shared_memory_factory_;
-  ScreenId current_screen_id_;
+  ScreenId current_screen_id_ = kFullDesktopScreenId;
   std::wstring current_device_key_;
-  HWND excluded_window_;
+  HWND excluded_window_ = nullptr;

[Wez, 2016/05/18 01:29:50]

nit: See comments elsewhere re leaving Windows handles set to NULL.

[Sergey Ulanov, 2016/05/31 12:02:49]

Done.

 
   // A thread-safe list of invalid rectangles, and the size of the most
   // recently captured screen.
@@ -124,31 +124,31 @@ class ScreenCapturerWinMagnifier : public ScreenCapturer {
   std::unique_ptr<Differ> differ_;
 
   // Used to suppress duplicate logging of SetThreadExecutionState errors.
-  bool set_thread_execution_state_failed_;
+  bool set_thread_execution_state_failed_ = false;
 
   ScopedThreadDesktop desktop_;
 
   // Used for getting the screen dpi.
-  HDC desktop_dc_;
+  HDC desktop_dc_ = nullptr;
 
-  HMODULE mag_lib_handle_;
-  MagInitializeFunc mag_initialize_func_;
-  MagUninitializeFunc mag_uninitialize_func_;
-  MagSetWindowSourceFunc set_window_source_func_;
-  MagSetWindowFilterListFunc set_window_filter_list_func_;
-  MagSetImageScalingCallbackFunc set_image_scaling_callback_func_;
+  HMODULE mag_lib_handle_ = nullptr;
+  MagInitializeFunc mag_initialize_func_ = nullptr;
+  MagUninitializeFunc mag_uninitialize_func_ = nullptr;
+  MagSetWindowSourceFunc set_window_source_func_ = nullptr;
+  MagSetWindowFilterListFunc set_window_filter_list_func_ = nullptr;
+  MagSetImageScalingCallbackFunc set_image_scaling_callback_func_ = nullptr;
 
   // The hidden window hosting the magnifier control.
-  HWND host_window_;
+  HWND host_window_ = nullptr;
   // The magnifier control that captures the screen.
-  HWND magnifier_window_;
+  HWND magnifier_window_ = nullptr;
 
   // True if the magnifier control has been successfully initialized.
-  bool magnifier_initialized_;
+  bool magnifier_initialized_ = false;
 
   // True if the last OnMagImageScalingCallback was called and handled
   // successfully. Reset at the beginning of each CaptureImage call.
-  bool magnifier_capture_succeeded_;
+  bool magnifier_capture_succeeded_ = true;
 
   RTC_DISALLOW_COPY_AND_ASSIGN(ScreenCapturerWinMagnifier);
 };