Add support for high-DPI hosts under Mac OS X.

remoting/host/event_executor_mac.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/host/event_executor.h"
 
 #include <ApplicationServices/ApplicationServices.h>
 #include <Carbon/Carbon.h>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/memory/ref_counted.h"
 #include "base/single_thread_task_runner.h"
+#include "remoting/capturer/mac/desktop_configuration.h"
 #include "remoting/host/clipboard.h"
 #include "remoting/proto/internal.pb.h"
 #include "remoting/protocol/message_decoder.h"
+#include "skia/ext/skia_utils_mac.h"
 #include "third_party/skia/include/core/SkPoint.h"
 #include "third_party/skia/include/core/SkRect.h"
 
 namespace remoting {
 
 namespace {
 
 using protocol::ClipboardEvent;
 using protocol::KeyEvent;
 using protocol::MouseEvent;
 
 // USB to Mac keycode mapping table.
 #define USB_KEYMAP(usb, xkb, win, mac) {usb, mac}
 #include "ui/base/keycodes/usb_keycode_map.h"
 #undef USB_KEYMAP
 
 // skia/ext/skia_utils_mac.h only defines CGRectToSkRect().
 SkIRect CGRectToSkIRect(const CGRect& rect) {
-  SkIRect sk_rect = {
-    SkScalarRound(rect.origin.x),
-    SkScalarRound(rect.origin.y),
-    SkScalarRound(rect.origin.x + rect.size.width),
-    SkScalarRound(rect.origin.y + rect.size.height)
-  };
-  return sk_rect;
+  SkIRect result;
+  gfx::CGRectToSkRect(rect).round(&result);
+  return result;
 }
 
 // A class to generate events on Mac.
 class EventExecutorMac : public EventExecutor {
  public:
   explicit EventExecutorMac(
       scoped_refptr<base::SingleThreadTaskRunner> task_runner);
   virtual ~EventExecutorMac();
 
   // ClipboardStub interface.
@@ skipping 128 matching lines @@
     // On multi-monitor systems (0,0) refers to the top-left of the "main"
     // display, whereas our coordinate scheme places (0,0) at the top-left of
     // the bounding rectangle around all the displays, so we need to translate
     // accordingly.
     // TODO(wez): Move display config tracking into a separate class used both
     // here and in the Capturer.
 
     // Set the mouse position assuming single-monitor.
     mouse_pos_ = SkIPoint::Make(event.x(), event.y());
 
-    // Determine how many active displays there are.
-    CGDisplayCount display_count;
-    CGError error = CGGetActiveDisplayList(0, NULL, &display_count);
-    CHECK_EQ(error, CGDisplayNoErr);
+    // Fetch the desktop configuration.
+    // TODO(wez): Optimize this out, or at least only enumerate displays in
+    // response to display-changed events. VideoFrameCapturer's VideoFrames
+    // could be augmented to include native cursor coordinates for use by
+    // MouseClampingFilter, removing the need for translation here.
+    MacDesktopConfiguration desktop_config
+        = MacDesktopConfiguration::GetCurrent();
 
-    if (display_count > 1) {
-      // Determine the bounding box of the displays, to get the top-left origin.
-      std::vector<CGDirectDisplayID> display_ids(display_count);
-      error = CGGetActiveDisplayList(display_count, &display_ids[0],
-                                     &display_count);
-      CHECK_EQ(error, CGDisplayNoErr);
-      CHECK_EQ(display_count, display_ids.size());
+    // Translate the mouse position into desktop coordinates.
+    mouse_pos_ += SkIPoint::Make(desktop_config.pixel_bounds.left(),
+                                 desktop_config.pixel_bounds.top());
 
-      SkIRect desktop_bounds = SkIRect::MakeEmpty();
-      for (unsigned int d = 0; d < display_count; ++d) {
-        CGRect display_bounds = CGDisplayBounds(display_ids[d]);
-        desktop_bounds.join(CGRectToSkIRect(display_bounds));
-      }
+    // Convert from pixel to logical coordinates.
+    mouse_pos_ = SkIPoint::Make(
+        SkScalarRound(mouse_pos_.x() / desktop_config.logical_to_pixel_scale),
+        SkScalarRound(mouse_pos_.y() / desktop_config.logical_to_pixel_scale));
 
-      // Adjust the injected mouse event position.
-      mouse_pos_ += SkIPoint::Make(desktop_bounds.left(), desktop_bounds.top());
-    }
-
-    VLOG(3) << "Moving mouse to " << event.x() << "," << event.y();
+    VLOG(3) << "Moving mouse to " << mouse_pos_.x() << "," << mouse_pos_.y();
   }
   if (event.has_button() && event.has_button_down()) {
     if (event.button() >= 1 && event.button() <= 3) {
       VLOG(2) << "Button " << event.button()
               << (event.button_down() ? " down" : " up");
       int button_change = 1 << (event.button() - 1);
       if (event.button_down())
         mouse_button_state_ |= button_change;
       else
         mouse_button_state_ &= ~button_change;
@@ skipping 61 matching lines @@
 
 }  // namespace
 
 scoped_ptr<EventExecutor> EventExecutor::Create(
     scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner) {
   return scoped_ptr<EventExecutor>(new EventExecutorMac(main_task_runner));
 }
 
 }  // namespace remoting