Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Unified Diff: ui/views/widget/native_widget_aura.cc

Issue 11030017: Add context to gfx::Screen calls in support of simultaneous desktop+ash (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: win fix Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/widget/native_widget_aura.cc
diff --git a/ui/views/widget/native_widget_aura.cc b/ui/views/widget/native_widget_aura.cc
index ee2fe37ac28105684fc56c73d2a69c33fe6948b0..14119547ec127d960c5391105e45bf7079cd7938 100644
--- a/ui/views/widget/native_widget_aura.cc
+++ b/ui/views/widget/native_widget_aura.cc
@@ -128,11 +128,12 @@ class NativeWidgetAura::ActiveWindowObserver
////////////////////////////////////////////////////////////////////////////////
// NativeWidgetAura, public:
-NativeWidgetAura::NativeWidgetAura(internal::NativeWidgetDelegate* delegate)
+NativeWidgetAura::NativeWidgetAura(
+ internal::NativeWidgetDelegate* delegate, gfx::NativeView parent)
: delegate_(delegate),
ALLOW_THIS_IN_INITIALIZER_LIST(desktop_helper_(
ViewsDelegate::views_delegate ?
- ViewsDelegate::views_delegate->CreateNativeWidgetHelper(this) :
+ ViewsDelegate::views_delegate->CreateNativeWidgetHelper(this, parent):
NULL)),
ALLOW_THIS_IN_INITIALIZER_LIST(window_(new aura::Window(this))),
ownership_(Widget::InitParams::NATIVE_WIDGET_OWNS_WIDGET),
@@ -213,7 +214,8 @@ void NativeWidgetAura::InitNativeWidget(const Widget::InitParams& params) {
// If a parent is specified but no bounds are given,
// use the origin of the parent's display so that the widget
// will be added to the same display as the parent.
- gfx::Rect bounds = gfx::Screen::GetDisplayNearestWindow(parent).bounds();
+ gfx::Rect bounds = gfx::Screen::GetScreenFor(parent)->
+ GetDisplayNearestWindow(parent).bounds();
window_bounds.set_origin(bounds.origin());
}
window_->SetParent(parent);
@@ -361,8 +363,8 @@ void NativeWidgetAura::CenterWindow(const gfx::Size& size) {
// When centering window, we take the intersection of the host and
// the parent. We assume the root window represents the visible
// rect of a single screen.
- gfx::Rect work_area =
- gfx::Screen::GetDisplayNearestWindow(window_).work_area();
+ gfx::Rect work_area = gfx::Screen::GetScreenFor(window_)->
+ GetDisplayNearestWindow(window_).work_area();
aura::client::ScreenPositionClient* screen_position_client =
aura::client::GetScreenPositionClient(window_->GetRootWindow());
@@ -469,7 +471,8 @@ void NativeWidgetAura::SetBounds(const gfx::Rect& bounds) {
aura::client::ScreenPositionClient* screen_position_client =
aura::client::GetScreenPositionClient(root);
if (screen_position_client) {
- gfx::Display dst_display = gfx::Screen::GetDisplayMatching(bounds);
+ gfx::Display dst_display =
+ gfx::Screen::GetScreenFor(window_)->GetDisplayMatching(bounds);
screen_position_client->SetBounds(window_, bounds, dst_display);
return;
}
@@ -668,7 +671,8 @@ void NativeWidgetAura::ClearNativeFocus() {
}
gfx::Rect NativeWidgetAura::GetWorkAreaBoundsInScreen() const {
- return gfx::Screen::GetDisplayNearestWindow(GetNativeView()).work_area();
+ return gfx::Screen::GetScreenFor(GetNativeView())->
+ GetDisplayNearestWindow(GetNativeView()).work_area();
}
void NativeWidgetAura::SetInactiveRenderingDisabled(bool value) {
@@ -990,8 +994,9 @@ namespace internal {
// static
NativeWidgetPrivate* NativeWidgetPrivate::CreateNativeWidget(
- internal::NativeWidgetDelegate* delegate) {
- return new NativeWidgetAura(delegate);
+ internal::NativeWidgetDelegate* delegate,
+ gfx::NativeView parent) {
+ return new NativeWidgetAura(delegate, parent);
}
// static

Powered by Google App Engine
This is Rietveld 408576698