exo: Fix multi-display hardware cursor

components/exo/pointer.cc

 // Copyright 2015 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 "components/exo/pointer.h"
 
+#include <algorithm>
+
 #include "ash/public/cpp/shell_window_ids.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/copy_output_result.h"
 #include "components/exo/pointer_delegate.h"
 #include "components/exo/pointer_stylus_delegate.h"
 #include "components/exo/surface.h"
 #include "components/exo/wm_helper.h"
 #include "ui/aura/client/cursor_client.h"
 #include "ui/aura/env.h"
 #include "ui/aura/window.h"
+#include "ui/base/cursor/cursor_util.h"
 #include "ui/display/manager/display_manager.h"
 #include "ui/display/manager/managed_display_info.h"
 #include "ui/display/screen.h"
 #include "ui/events/event.h"
 #include "ui/gfx/geometry/vector2d_conversions.h"
 #include "ui/gfx/transform_util.h"
 
 #if defined(USE_OZONE)
 #include "ui/ozone/public/cursor_factory_ozone.h"
 #endif
 
 #if defined(USE_X11)
 #include "ui/base/cursor/cursor_loader_x11.h"
 #endif
 
 namespace exo {
 namespace {
 
-const float kLargeCursorScale = 2.8f;
+// TODO(oshima): Some accessibility features, including large cursors, disable

[reveman, 2017/05/23 17:06:57]

please minimize the patch and avoid this change that doesn't seem necessary for
it's functionality

[Dominik Laskowski, 2017/05/24 00:43:04]

It's needed due to the new logic for updating scale.

[reveman, 2017/05/25 10:52:55]

I'm failing to see why. Please explain.

[Dominik Laskowski, 2017/05/31 02:06:26]

Folded into UpdateCursor.

+// hardware cursors. Ash does not support compositing for custom cursors, so it
+// replaces them with the default cursor. As a result, this scale has no effect
+// for now. See crbug.com/708378.
+float GetCursorScale(ui::CursorSetType cursor_set) {
+  return cursor_set == ui::CURSOR_SET_LARGE ? 2.8f : 1.0f;
+}
 
 // Synthesized events typically lack floating point precision so to avoid
 // generating mouse event jitter we consider the location of these events
 // to be the same as |location| if floored values match.
 bool SameLocation(const ui::LocatedEvent* event, const gfx::PointF& location) {
   if (event->flags() & ui::EF_IS_SYNTHESIZED)
     return event->location() == gfx::ToFlooredPoint(location);
 
   return event->location_f() == location;
 }
 
+inline const display::ManagedDisplayInfo& GetDisplayInfo(
+    const display::Display& display) {
+  return WMHelper::GetInstance()->GetDisplayInfo(display.id());
+}
+
+display::Display GetCaptureDisplay() {

[reveman, 2017/05/23 17:06:57]

What if no DSF 2.0 display exists but is later added? Can we instead always use
the primary display and set the layer scale to always capture at 2x?

[Dominik Laskowski, 2017/05/24 00:43:04]

Good point. Capturing at constant pixel size is simpler and more predictable.
Done.

+  const auto& displays = display::Screen::GetScreen()->GetAllDisplays();
+  DCHECK(!displays.empty());
+  return *std::max_element(
+      displays.begin(), displays.end(),
+      [](const display::Display& lhs, const display::Display& rhs) -> bool {
+        return GetDisplayInfo(lhs).device_scale_factor() <
+               GetDisplayInfo(rhs).device_scale_factor();
+      });
+}
+
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // Pointer, public:
 
 Pointer::Pointer(PointerDelegate* delegate)
     : delegate_(delegate),
-      cursor_(ui::CursorType::kNull),
       cursor_capture_source_id_(base::UnguessableToken::Create()),
       cursor_capture_weak_ptr_factory_(this) {
   auto* helper = WMHelper::GetInstance();
   helper->AddPreTargetHandler(this);
   helper->AddCursorObserver(this);
+

[reveman, 2017/05/23 17:06:57]

why are the following 4 lines needed?

[Dominik Laskowski, 2017/05/24 00:43:04]

To initialize the state updated by OnCursorSetChanged and
OnCursorDisplayChanging. I had the impression that Ash moved the cursor to the
primary display on startup, but that's not the case so I've updated the code
here.

+  cursor_scale_ = GetCursorScale(helper->GetCursorSet());
+
+  OnCursorDisplayChanging(display::Screen::GetScreen()->GetPrimaryDisplay());
 }
 
 Pointer::~Pointer() {
   delegate_->OnPointerDestroying(this);
   if (surface_)
     surface_->RemoveSurfaceObserver(this);
   if (focus_) {
     focus_->RemoveSurfaceObserver(this);
     focus_->UnregisterCursorProvider(this);
   }
@@ skipping 25 matching lines @@
       surface_->RemoveSurfaceObserver(this);
     }
     surface_ = surface;
     if (surface_) {
       surface_->SetSurfaceDelegate(this);
       surface_->AddSurfaceObserver(this);
       // Note: Surface window needs to be added to the tree so we can take a
       // snapshot. Where in the tree is not important but we might as well use
       // the cursor container.
       WMHelper::GetInstance()
-          ->GetContainer(ash::kShellWindowId_MouseCursorContainer)
+          ->GetContainer(GetCaptureDisplay().id(),
+                         ash::kShellWindowId_MouseCursorContainer)
           ->AddChild(surface_->window());
     }
     cursor_changed = true;
   }
 
   // Update hotspot.
   if (hotspot != hotspot_) {
     hotspot_ = hotspot;
     cursor_changed = true;
   }
 
   // Early out if cursor did not change.
   if (!cursor_changed)
     return;
 
   // If |surface_| is set then asynchronously capture a snapshot of cursor,
   // otherwise cancel pending capture and immediately set the cursor to "none".
   if (surface_) {
     CaptureCursor();
   } else {
+    cursor_.reset();
     cursor_capture_weak_ptr_factory_.InvalidateWeakPtrs();
-    cursor_ = ui::CursorType::kNone;
-    UpdateCursor();
+    SetCursor(ui::CursorType::kNone);
   }
 }
 
 gfx::NativeCursor Pointer::GetCursor() {
-  return cursor_;
+  if (focus_)
+    if (auto* root_window = focus_->window()->GetRootWindow())
+      if (auto* cursor_client = aura::client::GetCursorClient(root_window))
+        return cursor_client->GetCursor();
+
+  return ui::CursorType::kNull;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ui::EventHandler overrides:
 
 void Pointer::OnMouseEvent(ui::MouseEvent* event) {
   Surface* target = GetEffectiveTargetForEvent(event);
 
   // If target is different than the current pointer focus then we need to
   // generate enter and leave events.
   if (target != focus_) {
     // First generate a leave event if we currently have a target in focus.
     if (focus_) {
       delegate_->OnPointerLeave(focus_);
       focus_->RemoveSurfaceObserver(this);
       // Require SetCursor() to be called and cursor to be re-defined in
       // response to each OnPointerEnter() call.
       focus_->UnregisterCursorProvider(this);
       focus_ = nullptr;
-      cursor_ = ui::CursorType::kNull;
       cursor_capture_weak_ptr_factory_.InvalidateWeakPtrs();
     }
     // Second generate an enter event if focus moved to a new target.
     if (target) {
       delegate_->OnPointerEnter(target, event->location_f(),
                                 event->button_flags());
       location_ = event->location_f();
       focus_ = target;
       focus_->AddSurfaceObserver(this);
       focus_->RegisterCursorProvider(this);
@@ skipping 69 matching lines @@
     case ui::ET_MOUSE_ENTERED:
     case ui::ET_MOUSE_EXITED:
     case ui::ET_MOUSE_CAPTURE_CHANGED:
       break;
     default:
       NOTREACHED();
       break;
   }
 
   last_event_type_ = event->type();
-  UpdateCursorScale();
 }
 
 void Pointer::OnScrollEvent(ui::ScrollEvent* event) {
   OnMouseEvent(event);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // WMHelper::CursorObserver overrides:
 
 void Pointer::OnCursorSetChanged(ui::CursorSetType cursor_set) {
+  cursor_scale_ = GetCursorScale(cursor_set);
   if (focus_)
-    UpdateCursorScale();
+    UpdateCursor();
+}
+
+void Pointer::OnCursorDisplayChanging(const display::Display& display) {
+  device_scale_factor_ = GetDisplayInfo(display).device_scale_factor();
+  rotation_ = display.rotation();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // SurfaceDelegate overrides:
 
 void Pointer::OnSurfaceCommit() {
   surface_->CheckIfSurfaceHierarchyNeedsCommitToNewSurfaces();
   surface_->CommitSurfaceHierarchy();
 
   // Capture new cursor to reflect result of commit.
@@ skipping 23 matching lines @@
 
 Surface* Pointer::GetEffectiveTargetForEvent(ui::Event* event) const {
   Surface* target =
       Surface::AsSurface(static_cast<aura::Window*>(event->target()));
   if (!target)
     return nullptr;
 
   return delegate_->CanAcceptPointerEventsForSurface(target) ? target : nullptr;
 }
 
-void Pointer::UpdateCursorScale() {
-  DCHECK(focus_);
-
-  display::Screen* screen = display::Screen::GetScreen();
-  WMHelper* helper = WMHelper::GetInstance();
-
-  // Update cursor scale if the effective UI scale has changed.
-  display::Display display = screen->GetDisplayNearestWindow(focus_->window());
-  float scale = helper->GetDisplayInfo(display.id()).GetEffectiveUIScale();
-
-  if (display::Display::HasInternalDisplay()) {
-    float primary_device_scale_factor =
-        screen->GetPrimaryDisplay().device_scale_factor();
-    // The size of the cursor surface is the quotient of its physical size and
-    // the DSF of the primary display. The physical size is proportional to the
-    // DSF of the internal display. For external displays (and the internal
-    // display when secondary to a display with a different DSF), scale the
-    // cursor so its physical size matches with the single display case.
-    if (!display.IsInternal() ||
-        display.device_scale_factor() != primary_device_scale_factor) {
-      scale *= primary_device_scale_factor /
-               helper->GetDisplayInfo(display::Display::InternalDisplayId())
-                   .device_scale_factor();
-    }
-  }
-
-  if (helper->GetCursorSet() == ui::CURSOR_SET_LARGE)
-    scale *= kLargeCursorScale;
-
-  if (scale != cursor_scale_) {
-    cursor_scale_ = scale;
-    if (surface_)
-      CaptureCursor();
-  }
-}
-
 void Pointer::CaptureCursor() {
   DCHECK(surface_);
   DCHECK(focus_);
 
-  // Set UI scale before submitting capture request.
-  surface_->window()->layer()->SetTransform(
-      gfx::GetScaleTransform(gfx::Point(), cursor_scale_));
-
-  float primary_device_scale_factor =
-      display::Screen::GetScreen()->GetPrimaryDisplay().device_scale_factor();
+  // Surface size is in DIPs, while layer size is in pseudo-DIP units that
+  // depend on the DSF of the display mode. Scale the layer to capture the
+  // surface at a constant pixel size converted from DIPs using the DSF of
+  // the capture display, regardless of its UI scale and display mode DSF.
+  display::Display display = GetCaptureDisplay();
+  const auto& info = GetDisplayInfo(display);
+  float display_scale = info.GetEffectiveUIScale() * info.device_scale_factor();
+  surface_->window()->SetTransform(gfx::GetScaleTransform(
+      gfx::Point(), display_scale / display.device_scale_factor()));
 
   std::unique_ptr<cc::CopyOutputRequest> request =
       cc::CopyOutputRequest::CreateBitmapRequest(
           base::Bind(&Pointer::OnCursorCaptured,
-                     cursor_capture_weak_ptr_factory_.GetWeakPtr(),
-                     gfx::ScaleToFlooredPoint(
-                         hotspot_,
-                         // |hotspot_| is in surface coordinate space so apply
-                         // both device scale and UI scale.
-                         cursor_scale_ * primary_device_scale_factor)));
+                     cursor_capture_weak_ptr_factory_.GetWeakPtr()));
+
   request->set_source(cursor_capture_source_id_);
   surface_->window()->layer()->RequestCopyOfOutput(std::move(request));
 }
 
-void Pointer::OnCursorCaptured(const gfx::Point& hotspot,
-                               std::unique_ptr<cc::CopyOutputResult> result) {
+void Pointer::OnCursorCaptured(std::unique_ptr<cc::CopyOutputResult> result) {
   if (!focus_)
     return;
 
-  cursor_ = ui::CursorType::kNone;
-  if (!result->IsEmpty()) {
-    DCHECK(result->HasBitmap());
-    std::unique_ptr<SkBitmap> bitmap = result->TakeBitmap();
-
-    ui::PlatformCursor platform_cursor;
-#if defined(USE_OZONE)
-    // TODO(reveman): Add interface for creating cursors from GpuMemoryBuffers
-    // and use that here instead of the current bitmap API. crbug.com/686600
-    platform_cursor = ui::CursorFactoryOzone::GetInstance()->CreateImageCursor(
-        *bitmap.get(), hotspot, cursor_scale_);
-#elif defined(USE_X11)
-    XcursorImage* image = ui::SkBitmapToXcursorImage(bitmap.get(), hotspot);
-    platform_cursor = ui::CreateReffedCustomXCursor(image);
-#endif
-    cursor_ = ui::CursorType::kCustom;
-    cursor_.SetPlatformCursor(platform_cursor);
-#if defined(USE_OZONE)
-    ui::CursorFactoryOzone::GetInstance()->UnrefImageCursor(platform_cursor);
-#elif defined(USE_X11)
-    ui::UnrefCustomXCursor(platform_cursor);
-#endif
+  if (result->IsEmpty()) {
+    cursor_.reset();
+    SetCursor(ui::CursorType::kNone);
+    return;
   }
 
+  DCHECK(result->HasBitmap());
+  cursor_ = *result->TakeBitmap();
   UpdateCursor();
 }
 
 void Pointer::UpdateCursor() {
   DCHECK(focus_);
 
+  if (cursor_.drawsNothing())
+    return;
+
+  const auto& info = GetDisplayInfo(GetCaptureDisplay());
+
+  SkBitmap bitmap = cursor_;
+  gfx::Point hotspot =
+      gfx::ScaleToFlooredPoint(hotspot_, info.device_scale_factor());
+
+  ui::ScaleAndRotateCursorBitmapAndHotpoint(
+      cursor_scale_ * device_scale_factor_ / info.device_scale_factor(),
+      rotation_, &bitmap, &hotspot);
+
+  ui::PlatformCursor platform_cursor;
+#if defined(USE_OZONE)
+  // TODO(reveman): Add interface for creating cursors from GpuMemoryBuffers
+  // and use that here instead of the current bitmap API. crbug.com/686600
+  platform_cursor = ui::CursorFactoryOzone::GetInstance()->CreateImageCursor(
+      bitmap, hotspot, 0);
+#elif defined(USE_X11)
+  XcursorImage* image = ui::SkBitmapToXcursorImage(&bitmap, hotspot);
+  platform_cursor = ui::CreateReffedCustomXCursor(image);
+#endif
+  gfx::NativeCursor cursor = ui::CursorType::kCustom;
+  cursor.SetPlatformCursor(platform_cursor);
+#if defined(USE_OZONE)
+  ui::CursorFactoryOzone::GetInstance()->UnrefImageCursor(platform_cursor);
+#elif defined(USE_X11)
+  ui::UnrefCustomXCursor(platform_cursor);
+#endif
+
+  SetCursor(cursor);
+}
+
+void Pointer::SetCursor(gfx::NativeCursor cursor) {
+  DCHECK(focus_);
+
   aura::Window* root_window = focus_->window()->GetRootWindow();
   if (!root_window)
     return;
 
   aura::client::CursorClient* cursor_client =
       aura::client::GetCursorClient(root_window);
   if (cursor_client)
-    cursor_client->SetCursor(cursor_);
+    cursor_client->SetCursor(cursor);
 }
 
 }  // namespace exo