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

Unified Diff: chrome/browser/ui/window_sizer/window_sizer.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: fix new addition 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: chrome/browser/ui/window_sizer/window_sizer.cc
diff --git a/chrome/browser/ui/window_sizer/window_sizer.cc b/chrome/browser/ui/window_sizer/window_sizer.cc
index 57d0426147fed45c8d9cf955010bb3284c397fbb..4cee7597f2d64d6c64fd956e9be5d6f7bc257d35 100644
--- a/chrome/browser/ui/window_sizer/window_sizer.cc
+++ b/chrome/browser/ui/window_sizer/window_sizer.cc
@@ -25,17 +25,22 @@ const int kMinVisibleWidth = 30;
class DefaultMonitorInfoProvider : public MonitorInfoProvider {
public:
+ explicit DefaultMonitorInfoProvider(const gfx::Screen* screen)
+ : screen_(screen) {}
// Overridden from MonitorInfoProvider:
virtual gfx::Rect GetPrimaryDisplayWorkArea() const OVERRIDE {
- return gfx::Screen::GetPrimaryDisplay().work_area();
+ return screen_->GetPrimaryDisplay().work_area();
}
virtual gfx::Rect GetPrimaryDisplayBounds() const OVERRIDE {
- return gfx::Screen::GetPrimaryDisplay().bounds();
+ return screen_->GetPrimaryDisplay().bounds();
}
virtual gfx::Rect GetMonitorWorkAreaMatching(
const gfx::Rect& match_rect) const OVERRIDE {
- return gfx::Screen::GetDisplayMatching(match_rect).work_area();
+ return screen_->GetDisplayMatching(match_rect).work_area();
}
+ private:
+ const gfx::Screen* screen_;
+ DISALLOW_COPY_AND_ASSIGN(DefaultMonitorInfoProvider);
};
///////////////////////////////////////////////////////////////////////////////
@@ -150,7 +155,9 @@ const int WindowSizer::kMaximumWindowWidth = 1100;
WindowSizer::WindowSizer(StateProvider* state_provider, const Browser* browser)
: state_provider_(state_provider),
- monitor_info_provider_(new DefaultMonitorInfoProvider),
+ monitor_info_provider_(new DefaultMonitorInfoProvider(
+ // TODO(scottmg): NativeScreen is wrong. http://crbug.com/133312
+ gfx::Screen::GetNativeScreen())),
browser_(browser) {
}

Powered by Google App Engine
This is Rietveld 408576698