Move two utility functions from ash/screen_util.h to ui/wm/core/coordinate_conversion.h

ui/wm/core/coordinate_conversion.cc

Index: ui/wm/core/coordinate_conversion.cc
diff --git a/ui/wm/core/coordinate_conversion.cc b/ui/wm/core/coordinate_conversion.cc
index 1d97f04f028b77142df28ad6abda174162b042f9..e5cfe07ece8d512be0ccc6e7997a82c856d08e0b 100644
--- a/ui/wm/core/coordinate_conversion.cc
+++ b/ui/wm/core/coordinate_conversion.cc
@@ -6,6 +6,7 @@
 
 #include "ui/aura/client/screen_position_client.h"
 #include "ui/gfx/geometry/point.h"
+#include "ui/gfx/geometry/rect.h"
 
 namespace wm {
 
@@ -26,4 +27,17 @@ void ConvertPointFromScreen(const aura::Window* window,
       ConvertPointFromScreen(window, point_in_screen);
 }
 
+void ConvertRectToScreen(const aura::Window* window, gfx::Rect* rect) {
+  gfx::Point point = rect->origin();
+  ConvertPointToScreen(window, &point);
+  rect->set_origin(point);

[sadrul, 2017/01/31 16:58:12]

It is possible for the window to be rotated/scaled etc., right? In that case,
you can't just change the origin, you also need to update the size.

[yhanada, 2017/02/01 06:39:27]

Done.

[oshima, 2017/02/01 18:50:56]

It is not possible to convert rect to rect over arbitrary transform, because tbe
result may not longer be rectangular after transformed.

This is at least consistent with what aura window does:

https://cs.chromium.org/chromium/src/ui/aura/window.cc?rcl=5e61cddf30c3bd3490...

I believe this assumes that it's not used for such case, and probably
won't cause any issue though, although you may want to keep consistent.
(or change aura window).

[sadrul, 2017/02/02 17:33:17]

We should fix aura window. Even the new code isn't entirely correct (but is
better than this). We should get the transform and transform the entire rect
(using gfx::Transform::TransformRect, like we do in views:
https://cs.chromium.org/chromium/src/ui/views/view.cc?type=cs&sq=package:chro...
)

+}
+
+void ConvertRectFromScreen(const aura::Window* window,
+                           gfx::Rect* rect_in_screen) {
+  gfx::Point point = rect_in_screen->origin();
+  ConvertPointFromScreen(window, &point);
+  rect_in_screen->set_origin(point);
+}
+
 }  // namespace wm