commit 2a3eb9f367a70e2f5060d9dea6edd5ccd326de61
author erikchen <erikchen@chromium.org> Tue Nov 15 18:24:43 2016
committer Commit bot <commit-bot@chromium.org> Tue Nov 15 18:24:53 2016
tree 8fff77c6e29118e037982335ee8012af98a72275
parent 5d54e185d50a3a177676168486af402f287feca7

mac: Fix screen capture on secondary displays.

The old API CGScreenRegisterMoveCallback returned update rects in desktop
coordinates [secondary display has an origin != 0,0]. The new CGDisplayStream
API returns update rects in display coordinates [origin == 0,0]. Translating the
update rect based on the display's position on the desktop is now incorrect.

BUG=webrtc:6702

Review-Url: https://codereview.webrtc.org/2496413002
Cr-Commit-Position: refs/heads/master@{#15092}

webrtc/modules/desktop_capture/screen_capturer_mac.mm

diff --git a/webrtc/modules/desktop_capture/screen_capturer_mac.mm b/webrtc/modules/desktop_capture/screen_capturer_mac.mm
index 34f5087..f11b36d 100644
--- a/webrtc/modules/desktop_capture/screen_capturer_mac.mm
+++ b/webrtc/modules/desktop_capture/screen_capturer_mac.mm
@@ -990,13 +990,9 @@
     ScreenConfigurationChanged();
 
   DesktopRegion region;
-  DesktopVector translate_vector =
-      DesktopVector().subtract(screen_pixel_bounds_.top_left());
   for (CGRectCount i = 0; i < count; ++i) {
     // Convert from Density-Independent Pixel to physical pixel coordinates.
     DesktopRect rect = ScaleAndRoundCGRect(rect_array[i], dip_to_pixel_scale_);
-    // Translate from local desktop to capturer framebuffer coordinates.
-    rect.Translate(translate_vector);
     region.AddRect(rect);
   }