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

Side by Side Diff: ui/views/controls/menu/menu_message_loop_aura.cc

Issue 1138523006: Enable keyboard accelerators while a menu is open (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ui/views/controls/menu/menu_message_loop_aura.h" 5 #include "ui/views/controls/menu/menu_message_loop_aura.h"
6 6
7 #if defined(OS_WIN) 7 #if defined(OS_WIN)
8 #include <windowsx.h> 8 #include <windowsx.h>
9 #endif 9 #endif
10 10
11 #include "base/run_loop.h" 11 #include "base/run_loop.h"
12 #include "ui/aura/client/screen_position_client.h" 12 #include "ui/aura/client/screen_position_client.h"
13 #include "ui/aura/env.h" 13 #include "ui/aura/env.h"
14 #include "ui/aura/window.h" 14 #include "ui/aura/window.h"
15 #include "ui/aura/window_event_dispatcher.h" 15 #include "ui/aura/window_event_dispatcher.h"
16 #include "ui/aura/window_tree_host.h" 16 #include "ui/aura/window_tree_host.h"
17 #include "ui/events/event.h" 17 #include "ui/events/event.h"
18 #include "ui/events/platform/platform_event_source.h" 18 #include "ui/events/platform/platform_event_source.h"
19 #include "ui/events/platform/scoped_event_dispatcher.h" 19 #include "ui/events/platform/scoped_event_dispatcher.h"
20 #include "ui/views/controls/menu/menu_controller.h" 20 #include "ui/views/controls/menu/menu_controller.h"
21 #include "ui/views/controls/menu/menu_event_filter.h"
21 #include "ui/views/widget/widget.h" 22 #include "ui/views/widget/widget.h"
22 #include "ui/wm/public/activation_change_observer.h" 23 #include "ui/wm/public/activation_change_observer.h"
23 #include "ui/wm/public/activation_client.h" 24 #include "ui/wm/public/activation_client.h"
24 #include "ui/wm/public/dispatcher_client.h" 25 #include "ui/wm/public/dispatcher_client.h"
25 #include "ui/wm/public/drag_drop_client.h" 26 #include "ui/wm/public/drag_drop_client.h"
26 27
27 #if defined(OS_WIN) 28 #if defined(OS_WIN)
28 #include "ui/base/win/internal_constants.h" 29 #include "ui/base/win/internal_constants.h"
29 #include "ui/views/controls/menu/menu_message_pump_dispatcher_win.h" 30 #include "ui/views/controls/menu/menu_message_pump_dispatcher_win.h"
30 #include "ui/views/win/hwnd_util.h" 31 #include "ui/views/win/hwnd_util.h"
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 DISALLOW_COPY_AND_ASSIGN(ActivationChangeObserverImpl); 97 DISALLOW_COPY_AND_ASSIGN(ActivationChangeObserverImpl);
97 }; 98 };
98 99
99 } // namespace 100 } // namespace
100 101
101 // static 102 // static
102 MenuMessageLoop* MenuMessageLoop::Create() { 103 MenuMessageLoop* MenuMessageLoop::Create() {
103 return new MenuMessageLoopAura; 104 return new MenuMessageLoopAura;
104 } 105 }
105 106
106 MenuMessageLoopAura::MenuMessageLoopAura() : owner_(NULL) { 107 MenuMessageLoopAura::MenuMessageLoopAura() : owner_(nullptr) {
107 } 108 }
108 109
109 MenuMessageLoopAura::~MenuMessageLoopAura() { 110 MenuMessageLoopAura::~MenuMessageLoopAura() {
110 } 111 }
111 112
112 void MenuMessageLoopAura::RepostEventToWindow(const ui::LocatedEvent& event, 113 void MenuMessageLoopAura::RepostEventToWindow(const ui::LocatedEvent& event,
113 gfx::NativeWindow window, 114 gfx::NativeWindow window,
114 const gfx::Point& screen_loc) { 115 const gfx::Point& screen_loc) {
115 aura::Window* root = window->GetRootWindow(); 116 aura::Window* root = window->GetRootWindow();
116 ScreenPositionClient* spc = aura::client::GetScreenPositionClient(root); 117 ScreenPositionClient* spc = aura::client::GetScreenPositionClient(root);
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 message_loop_quit_ = run_loop.QuitClosure(); 151 message_loop_quit_ = run_loop.QuitClosure();
151 run_loop.Run(); 152 run_loop.Run();
152 } else { 153 } else {
153 base::MessageLoopForUI* loop = base::MessageLoopForUI::current(); 154 base::MessageLoopForUI* loop = base::MessageLoopForUI::current();
154 base::MessageLoop::ScopedNestableTaskAllower allow(loop); 155 base::MessageLoop::ScopedNestableTaskAllower allow(loop);
155 base::RunLoop run_loop(&nested_dispatcher); 156 base::RunLoop run_loop(&nested_dispatcher);
156 message_loop_quit_ = run_loop.QuitClosure(); 157 message_loop_quit_ = run_loop.QuitClosure();
157 run_loop.Run(); 158 run_loop.Run();
158 } 159 }
159 #else 160 #else
160 internal::MenuEventDispatcher event_dispatcher(controller); 161 scoped_ptr<ActivationChangeObserverImpl> observer;
161 scoped_ptr<ui::ScopedEventDispatcher> dispatcher_override;
162 if (ui::PlatformEventSource::GetInstance()) {
163 dispatcher_override =
164 ui::PlatformEventSource::GetInstance()->OverrideDispatcher(
165 &event_dispatcher);
166 }
167 if (root) { 162 if (root) {
168 scoped_ptr<ActivationChangeObserverImpl> observer;
169 if (!nested_menu) 163 if (!nested_menu)
170 observer.reset(new ActivationChangeObserverImpl(controller, root)); 164 observer.reset(new ActivationChangeObserverImpl(controller, root));
171 aura::client::DispatcherRunLoop run_loop(
172 aura::client::GetDispatcherClient(root), NULL);
173 message_loop_quit_ = run_loop.QuitClosure();
174 run_loop.Run();
175 } else {
176 base::MessageLoopForUI* loop = base::MessageLoopForUI::current();
177 base::MessageLoop::ScopedNestableTaskAllower allow(loop);
178 base::RunLoop run_loop;
179 message_loop_quit_ = run_loop.QuitClosure();
180 run_loop.Run();
181 } 165 }
182 #endif 166
167 scoped_ptr<MenuEventFilter> menu_event_filter;
168 if (!nested_menu) {
169 // No need to recreate the MenuEventFilter for every nested menu.
170 menu_event_filter.reset(new MenuEventFilter);
171 aura::Env::GetInstanceDontCreate()->PrependPreTargetHandler(
172 menu_event_filter.get());
oshima 2015/09/08 21:43:08 You can add/remove the filter in MenuEventFilter's
afakhry 2015/09/08 23:34:38 Done.
173 }
174
175 base::MessageLoopForUI* loop = base::MessageLoopForUI::current();
176 base::MessageLoop::ScopedNestableTaskAllower allow(loop);
177 base::RunLoop run_loop;
178 message_loop_quit_ = run_loop.QuitClosure();
179
180 run_loop.Run();
181
182 if (menu_event_filter.get()) {
183 aura::Env::GetInstanceDontCreate()->RemovePreTargetHandler(
184 menu_event_filter.get());
185 }
186 #endif // defined(OS_WIN)
183 } 187 }
184 188
185 void MenuMessageLoopAura::QuitNow() { 189 void MenuMessageLoopAura::QuitNow() {
186 CHECK(!message_loop_quit_.is_null()); 190 CHECK(!message_loop_quit_.is_null());
187 message_loop_quit_.Run(); 191 message_loop_quit_.Run();
188 192
189 #if !defined(OS_WIN) 193 #if !defined(OS_WIN)
190 // Ask PlatformEventSource to stop dispatching events in this message loop 194 // Ask PlatformEventSource to stop dispatching events in this message loop
191 // iteration. We want our menu's loop to return before the next event. 195 // iteration. We want our menu's loop to return before the next event.
192 if (ui::PlatformEventSource::GetInstance()) 196 if (ui::PlatformEventSource::GetInstance())
193 ui::PlatformEventSource::GetInstance()->StopCurrentEventStream(); 197 ui::PlatformEventSource::GetInstance()->StopCurrentEventStream();
194 #endif 198 #endif
195 } 199 }
196 200
197 void MenuMessageLoopAura::ClearOwner() { 201 void MenuMessageLoopAura::ClearOwner() {
198 owner_ = NULL; 202 owner_ = NULL;
199 } 203 }
200 204
201 } // namespace views 205 } // namespace views
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698