Ash/aura split: NativeWidgetAura::GetWindowScreenBounds() should return screen bounds.

ui/aura/window.h

 // Copyright (c) 2012 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_AURA_WINDOW_H_
 #define UI_AURA_WINDOW_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 118 matching lines @@
   // Changes the visibility of the window.
   void Show();
   void Hide();
   // Returns true if this window and all its ancestors are visible.
   bool IsVisible() const;
   // Returns the visibility requested by this window. IsVisible() takes into
   // account the visibility of the layer and ancestors, where as this tracks
   // whether Show() without a Hide() has been invoked.
   bool TargetVisibility() const { return visible_; }
 
   // Returns the window's bounds in screen coordinates.

[sky, 2012/04/24 22:42:28]

Update the comment. In particular mention this is effectively screen coordinates
for ash, but not for non-ash.

[oshima, 2012/04/24 23:04:15]

Please add TODO(oshima): Fix this to return screen's coordinate for multi
monitor support.

-  gfx::Rect GetScreenBounds() const;
+  gfx::Rect GetBoundsInRootWindow() const;
 
   virtual void SetTransform(const ui::Transform& transform);
 
   // Assigns a LayoutManager to size and place child windows.
   // The Window takes ownership of the LayoutManager.
   void SetLayoutManager(LayoutManager* layout_manager);
   LayoutManager* layout_manager() { return layout_manager_.get(); }
 
   // Changes the bounds of the window. If present, the window's parent's
   // LayoutManager may adjust the bounds.
@@ skipping 311 matching lines @@
   };
 
   std::map<const void*, Value> prop_map_;
 
   DISALLOW_COPY_AND_ASSIGN(Window);
 };
 
 }  // namespace aura
 
 #endif  // UI_AURA_WINDOW_H_