ozone: dri: Cache display bounds in DriCursor

ui/ozone/platform/dri/dri_cursor.cc

 // Copyright 2014 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 "ui/ozone/platform/dri/dri_cursor.h"
 
 #include "ui/base/cursor/ozone/bitmap_cursor_factory_ozone.h"
 #include "ui/gfx/geometry/point.h"
 #include "ui/gfx/geometry/point_conversions.h"
 #include "ui/gfx/geometry/point_f.h"
@@ skipping 43 matching lines @@
   DCHECK_NE(cursor_window_, gfx::kNullAcceleratedWidget);
   sender_->SetHardwareCursor(cursor_window_, std::vector<SkBitmap>(),
                              gfx::Point(), 0);
 }
 
 void DriCursor::MoveCursorTo(gfx::AcceleratedWidget widget,
                              const gfx::PointF& location) {
   if (widget != cursor_window_ && cursor_window_ != gfx::kNullAcceleratedWidget)
     HideCursor();
 
+  DriWindow* window = window_manager_->GetWindow(widget);
+
   cursor_window_ = widget;
   cursor_location_ = location;
+  cursor_display_bounds_ = window->GetBounds();
 
-  if (cursor_window_ == gfx::kNullAcceleratedWidget)
-    return;
-
-  DriWindow* window = window_manager_->GetWindow(cursor_window_);
-  const gfx::Size& size = window->GetBounds().size();
+  const gfx::Size& size = cursor_display_bounds_.size();
   cursor_location_.SetToMax(gfx::PointF(0, 0));
   // Right and bottom edges are exclusive.
   cursor_location_.SetToMin(gfx::PointF(size.width() - 1, size.height() - 1));
 
   if (cursor_.get())
     sender_->MoveHardwareCursor(cursor_window_, bitmap_location());
 }
 
 void DriCursor::MoveCursorTo(const gfx::PointF& location) {
   DriWindow* window =
       window_manager_->GetWindowAt(gfx::ToFlooredPoint(location));
   if (!window)
     return;
 
   MoveCursorTo(window->GetAcceleratedWidget(),
                location - window->GetBounds().OffsetFromOrigin());
 }
 
 void DriCursor::MoveCursor(const gfx::Vector2dF& delta) {
+  if (cursor_window_ == gfx::kNullAcceleratedWidget)
+    return;
+
 #if defined(OS_CHROMEOS)
   gfx::Vector2dF transformed_delta = delta;
   ui::CursorController::GetInstance()->ApplyCursorConfigForWindow(
       cursor_window_, &transformed_delta);
   MoveCursorTo(cursor_window_, cursor_location_ + transformed_delta);
 #else
   MoveCursorTo(cursor_window_, cursor_location_ + delta);
 #endif
 }
 
 gfx::Rect DriCursor::GetCursorDisplayBounds() {
-  if (cursor_window_ == gfx::kNullAcceleratedWidget)
-    return gfx::Rect();
-
-  DriWindow* window = window_manager_->GetWindow(cursor_window_);
-  return window->GetBounds();
+  return cursor_display_bounds_;
 }
 
 gfx::AcceleratedWidget DriCursor::GetCursorWindow() {
   return cursor_window_;
 }
 
 bool DriCursor::IsCursorVisible() {
   return cursor_.get();
 }
 
 gfx::PointF DriCursor::location() {
   return cursor_location_;
 }
 
 gfx::Point DriCursor::bitmap_location() {
   return gfx::ToFlooredPoint(cursor_location_) -
          cursor_->hotspot().OffsetFromOrigin();
 }
 
 }  // namespace ui