exo: Fix dragging edge cases

components/exo/display.h

 // 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.
 
 #ifndef COMPONENTS_EXO_DISPLAY_H_
 #define COMPONENTS_EXO_DISPLAY_H_
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
 
 #include "base/macros.h"
 #include "base/memory/shared_memory_handle.h"
+#include "ui/gfx/geometry/point.h"
 
 #if defined(USE_OZONE)
 #include "base/files/scoped_file.h"
 #include "ui/gfx/buffer_types.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/native_pixmap_handle.h"
 #endif
 
-namespace gfx {
-class Point;
-}
-
 namespace exo {
 class NotificationSurface;
 class NotificationSurfaceManager;
 class SharedMemory;
 class ShellSurface;
 class SubSurface;
 class Surface;
 
 #if defined(USE_OZONE)
 class Buffer;
 #endif
 
 // The core display class. This class provides functions for creating surfaces
 // and is in charge of combining the contents of multiple surfaces into one
 // displayable output.

[oshima, 2016/10/12 01:56:54]

can you add documentation how coordinates work between ARC++ and Chrome?

[Dominik Laskowski, 2016/10/13 03:21:16]

Done.

 class Display {
  public:
   Display();
   explicit Display(NotificationSurfaceManager* notification_surface_manager);
   ~Display();
 
+  void set_virtual_origin(const gfx::Point& virtual_origin) {
+    virtual_origin_ = virtual_origin;
+  }
+
+  template <typename Point>
+  void ConvertFromVirtualToScreen(Point* point) const {
+    *point += virtual_origin_.OffsetFromOrigin();
+  }
+
+  template <typename Point>
+  void ConvertFromScreenToVirtual(Point* point) const {
+    *point -= virtual_origin_.OffsetFromOrigin();
+  }
+
   // Creates a new surface.
   std::unique_ptr<Surface> CreateSurface();
 
   // Creates a shared memory segment from |handle| of |size| with the
   // given |id|. This function takes ownership of |handle|.
   std::unique_ptr<SharedMemory> CreateSharedMemory(
       const base::SharedMemoryHandle& handle,
       size_t size);
 
 #if defined(USE_OZONE)
@@ skipping 23 matching lines @@
   // a child of |parent|.
   std::unique_ptr<SubSurface> CreateSubSurface(Surface* surface,
                                                Surface* parent);
 
   // Creates a notification surface for a surface and notification id.
   std::unique_ptr<NotificationSurface> CreateNotificationSurface(
       Surface* surface,
       const std::string& notification_id);
 
  private:
+  // Origin of the virtual screen relative to the primary display.
+  gfx::Point virtual_origin_;
+
   NotificationSurfaceManager* const notification_surface_manager_;
 
   DISALLOW_COPY_AND_ASSIGN(Display);
 };
 
 }  // namespace exo
 
 #endif  // COMPONENTS_EXO_DISPLAY_H_