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

Unified Diff: chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.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: non-aura 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: chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
diff --git a/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc b/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
index 7f05865a54c54ac9c29cf84fbb3b09c5516566ec..e54208a1cf7e24b04a05545d102b4134c33b1f6f 100644
--- a/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
+++ b/chrome/browser/ui/views/ash/chrome_browser_main_extra_parts_ash.cc
@@ -7,17 +7,33 @@
#include "chrome/browser/chrome_browser_main.h"
#include "chrome/browser/toolkit_extra_parts.h"
#include "chrome/browser/ui/ash/ash_init.h"
+#include "chrome/browser/ui/ash/ash_util.h"
#include "ui/aura/desktop/desktop_screen.h"
#include "ui/aura/desktop/desktop_stacking_client.h"
#include "ui/aura/env.h"
#include "ui/aura/single_display_manager.h"
#include "ui/gfx/screen.h"
+#include "ui/gfx/screen_type_delegate.h"
#if defined(FILE_MANAGER_EXTENSION)
#include "chrome/browser/ui/views/select_file_dialog_extension.h"
#include "chrome/browser/ui/views/select_file_dialog_extension_factory.h"
#endif
+#if defined(OS_WIN)
Ben Goodger (Google) 2012/10/10 22:52:09 if !defined(OS_CHROMEOS) ?
scottmg 2012/10/11 00:22:28 Done.
+class ScreenTypeDelegateWin : public gfx::ScreenTypeDelegate {
+ public:
+ ScreenTypeDelegateWin() {}
+ gfx::ScreenType GetScreenTypeForNativeView(gfx::NativeView view) {
Ben Goodger (Google) 2012/10/10 22:52:09 virtual blablah OVERRIDE {
scottmg 2012/10/11 00:22:28 Done.
+ return chrome::IsNativeViewInAsh(view) ?
+ gfx::SCREEN_TYPE_ALTERNATE :
+ gfx::SCREEN_TYPE_NATIVE;
+ }
+ private:
+ DISALLOW_COPY_AND_ASSIGN(ScreenTypeDelegateWin);
+};
+#endif
+
ChromeBrowserMainExtraPartsAsh::ChromeBrowserMainExtraPartsAsh() {
}
@@ -30,7 +46,11 @@ void ChromeBrowserMainExtraPartsAsh::PreProfileInit() {
} else {
aura::Env::GetInstance()->SetDisplayManager(new aura::SingleDisplayManager);
stacking_client_.reset(new aura::DesktopStackingClient);
- gfx::Screen::SetInstance(aura::CreateDesktopScreen());
+ gfx::Screen::SetScreenInstance(
+ gfx::SCREEN_TYPE_NATIVE, aura::CreateDesktopScreen());
+#if defined(OS_WIN)
Ben Goodger (Google) 2012/10/10 22:52:09 again if !defined(OS_CHROMEOS)?
scottmg 2012/10/11 00:22:28 Done.
+ gfx::Screen::SetScreenTypeDelegate(new ScreenTypeDelegateWin);
+#endif
}
#if defined(FILE_MANAGER_EXTENSION)
« no previous file with comments | « chrome/browser/ui/views/app_list/app_list_controller_win.cc ('k') | chrome/browser/ui/views/bookmarks/bookmark_bar_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698