MacViews: Implement Set/Get*Bounds for NativeWidgetMac

ui/gfx/mac/point_utils.h

+// 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.
+
+#ifndef UI_GFX_MAC_POINT_UTILS_H_

[Robert Sesek, 2014/06/25 17:06:04]

Tests for these functions would be good.

[tapted, 2014/06/26 08:44:31]

Done in https://codereview.chromium.org/345243007/ - needs to touch a few files
to move ScopedClassSwizzler to base, and it will conflict with the
EventGenerator CL that's scheduled for an OWNERS review next week, so I'll
publish when that's through.

+#define UI_GFX_MAC_POINT_UTILS_H_
+
+#include "ui/gfx/gfx_export.h"
+
+typedef struct _NSRect NSRect;
+
+namespace gfx {
+
+class Rect;
+
+// Convert a gfx::Rect specified with the origin at the top left of the primary
+// display into AppKit secreen coordinates (origin at the bottom left).
+GFX_EXPORT NSRect ScreenRectToNSRect(const gfx::Rect& rect);
+
+// Convert an AppKit NSRect with origin in the bottom left of the primary
+// display into a gfx::Rect with origin at the top left of the primary display.
+GFX_EXPORT gfx::Rect ScreenRectFromNSRect(const NSRect& point);
+
+}  // namespace gfx
+
+#endif  // UI_GFX_MAC_POINT_UTILS_H_

[tapted, 2014/06/25 10:55:56]

note: not sure it's the best filename, but I've put this in gfx since it will
come up a bit (e.g. anonymous::ConvertCoordinateSystem() in ui/gfx/screen_mac.mm
can use this, but that's orthogonal).

[sky, 2014/06/25 16:52:22]

If you want this to be focused around screen conversions maybe it should be
screen_coordinate_conversions?

[tapted, 2014/06/26 08:44:31]

Done - went with coordinate_conversions.h/mm (discovered
ash/wm/coordinate_conversions, and this is performing a similar role).