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

Issue 11087014: Revert 160696 - Decouple EventClientImpl and root window. Check containers on the same root window … (Closed)

Created:
8 years, 2 months ago by Ryan Hamilton
Modified:
8 years, 2 months ago
Reviewers:
oshima
CC:
chromium-reviews, mazda+watch_chromium.org, sadrul, yusukes+watch_chromium.org, derat+watch_chromium.org, ben+watch_chromium.org
Visibility:
Public.

Description

Revert 160696 - Decouple EventClientImpl and root window. Check containers on the same root window as focused window when testing if the focused window can receive events. This was causing lock screen to lose focus when mouse is clicked on another root window. This Cl also adds a debug build/linux desktop only shortcut (F11) to lock/shutdown. BUG=152982 TEST=covered by test. Review URL: https://chromiumcodereview.appspot.com/11047030 TBR=oshima@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=160707

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -155 lines) Patch
M ash/accelerators/accelerator_controller.cc View 3 chunks +4 lines, -31 lines 0 comments Download
M ash/accelerators/accelerator_table.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ash/accelerators/accelerator_table.cc View 4 chunks +0 lines, -16 lines 0 comments Download
M ash/extended_desktop_unittest.cc View 4 chunks +0 lines, -70 lines 0 comments Download
M ash/root_window_controller.h View 2 chunks +3 lines, -0 lines 0 comments Download
M ash/root_window_controller.cc View 3 chunks +4 lines, -0 lines 0 comments Download
M ash/shell.h View 3 chunks +0 lines, -4 lines 0 comments Download
M ash/shell.cc View 4 chunks +0 lines, -10 lines 0 comments Download
M ash/wm/event_client_impl.h View 2 chunks +3 lines, -1 line 0 comments Download
M ash/wm/event_client_impl.cc View 1 chunk +12 lines, -10 lines 0 comments Download
M ash/wm/power_button_controller.cc View 2 chunks +0 lines, -11 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Ryan Hamilton
8 years, 2 months ago (2012-10-08 21:24:15 UTC) #1
oshima
8 years, 2 months ago (2012-10-08 21:34:59 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698