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

Unified Diff: ash/root_window_controller.cc

Issue 10911256: Change the way the ToplevelEventHandler is hooked up. It is now both a pre- and post-target event h… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 3 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
« no previous file with comments | « no previous file | ash/wm/toplevel_window_event_handler.h » ('j') | ash/wm/toplevel_window_event_handler.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/root_window_controller.cc
===================================================================
--- ash/root_window_controller.cc (revision 156300)
+++ ash/root_window_controller.cc (working copy)
@@ -295,7 +295,6 @@
if (!internal::WorkspaceController::IsWorkspace2Enabled()) {
default_container_handler_.reset(
new ToplevelWindowEventHandler(default_container));
- default_container->AddPreTargetHandler(default_container_handler_.get());
}
SetChildWindowVisibilityChangesAnimated(default_container);
SetUsesScreenCoordinates(default_container);
@@ -306,8 +305,6 @@
non_lock_screen_containers);
always_on_top_container_handler_.reset(
new ToplevelWindowEventHandler(always_on_top_container));
- always_on_top_container->AddPreTargetHandler(
- always_on_top_container_handler_.get());
SetChildWindowVisibilityChangesAnimated(always_on_top_container);
SetUsesScreenCoordinates(always_on_top_container);
@@ -333,7 +330,6 @@
non_lock_screen_containers);
modal_container_handler_.reset(
new ToplevelWindowEventHandler(modal_container));
- modal_container->AddPreTargetHandler(modal_container_handler_.get());
modal_container->SetLayoutManager(
new internal::SystemModalContainerLayoutManager(modal_container));
SetChildWindowVisibilityChangesAnimated(modal_container);
@@ -362,8 +358,6 @@
lock_screen_containers);
lock_modal_container_handler_.reset(
new ToplevelWindowEventHandler(lock_modal_container));
- lock_modal_container->AddPreTargetHandler(
- lock_modal_container_handler_.get());
lock_modal_container->SetLayoutManager(
new internal::SystemModalContainerLayoutManager(lock_modal_container));
SetChildWindowVisibilityChangesAnimated(lock_modal_container);
« no previous file with comments | « no previous file | ash/wm/toplevel_window_event_handler.h » ('j') | ash/wm/toplevel_window_event_handler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698