Remove DesktopRegion parameter from DesktopCapturer::Capture

webrtc/modules/desktop_capture/desktop_capturer.h

Index: webrtc/modules/desktop_capture/desktop_capturer.h
diff --git a/webrtc/modules/desktop_capture/desktop_capturer.h b/webrtc/modules/desktop_capture/desktop_capturer.h
index 0c7b68e85d8ec9fd0502c9bb469c9efaf6095575..812e0928bfc929c83c90eec03ae6be61632c0297 100644
--- a/webrtc/modules/desktop_capture/desktop_capturer.h
+++ b/webrtc/modules/desktop_capture/desktop_capturer.h
@@ -66,12 +66,12 @@ class DesktopCapturer {
   virtual void SetSharedMemoryFactory(
       std::unique_ptr<SharedMemoryFactory> shared_memory_factory) {}
 
-  // This is a legacy interface, consumers should call CaptureFrame() function.
-  virtual void Capture(const DesktopRegion& region) { CaptureFrame(); }
-
   // Captures next frame, and involve callback provided by Start() function.
   // Pending capture requests are canceled when DesktopCapturer is deleted.
-  virtual void CaptureFrame() { Capture(DesktopRegion()); }
+  virtual void Capture() { CaptureFrame(); }

[Sergey Ulanov, 2016/10/18 23:37:10]

I think CaptureFrame() is a better name than Capture(), so maybe just remove the
old Capture()?

[Hzj_jie, 2016/10/19 00:49:17]

No opinion, yes, I can remove old Capture() function.

+
+  // This is a legacy interface, consumers should call Capture() function.
+  virtual void CaptureFrame() { Capture(); }
 
   // Sets the window to be excluded from the captured image in the future
   // Capture calls. Used to exclude the screenshare notification window for