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

Side by Side Diff: ui/views/mus/pointer_watcher_event_router.cc

Issue 2626013005: Change CaptureSynchronizer and PointerWatcherEventRouter to support multiple CaptureClients. (Closed)
Patch Set: separate out changes for removing GetCaptureClient() Created 3 years, 10 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/mus/pointer_watcher_event_router.h" 5 #include "ui/views/mus/pointer_watcher_event_router.h"
6 6
7 #include "ui/aura/client/capture_client.h" 7 #include "ui/aura/client/capture_client.h"
8 #include "ui/aura/mus/window_tree_client.h" 8 #include "ui/aura/mus/window_tree_client.h"
9 #include "ui/aura/window.h" 9 #include "ui/aura/window.h"
10 #include "ui/display/screen.h" 10 #include "ui/display/screen.h"
11 #include "ui/events/base_event_utils.h" 11 #include "ui/events/base_event_utils.h"
12 #include "ui/events/event.h" 12 #include "ui/events/event.h"
13 #include "ui/views/pointer_watcher.h" 13 #include "ui/views/pointer_watcher.h"
14 #include "ui/views/widget/desktop_aura/desktop_native_widget_aura.h" 14 #include "ui/views/widget/desktop_aura/desktop_native_widget_aura.h"
15 15
16 namespace views { 16 namespace views {
17 namespace { 17 namespace {
18 18
19 bool HasPointerWatcher( 19 bool HasPointerWatcher(
20 base::ObserverList<views::PointerWatcher, true>* observer_list) { 20 base::ObserverList<views::PointerWatcher, true>* observer_list) {
21 return observer_list->begin() != observer_list->end(); 21 return observer_list->begin() != observer_list->end();
22 } 22 }
23 23
24 } // namespace 24 } // namespace
25 25
26 PointerWatcherEventRouter::PointerWatcherEventRouter( 26 PointerWatcherEventRouter::PointerWatcherEventRouter(
27 aura::WindowTreeClient* window_tree_client) 27 aura::WindowTreeClient* window_tree_client)
28 : window_tree_client_(window_tree_client) { 28 : window_tree_client_(window_tree_client) {
29 window_tree_client->AddObserver(this); 29 window_tree_client->AddObserver(this);
30 window_tree_client_->GetCaptureClient()->AddObserver(this);
31 } 30 }
32 31
33 PointerWatcherEventRouter::~PointerWatcherEventRouter() { 32 PointerWatcherEventRouter::~PointerWatcherEventRouter() {
34 if (window_tree_client_) { 33 if (window_tree_client_)
35 window_tree_client_->RemoveObserver(this); 34 window_tree_client_->RemoveObserver(this);
36 window_tree_client_->GetCaptureClient()->RemoveObserver(this);
37 }
38 } 35 }
39 36
40 void PointerWatcherEventRouter::AddPointerWatcher(PointerWatcher* watcher, 37 void PointerWatcherEventRouter::AddPointerWatcher(PointerWatcher* watcher,
41 bool wants_moves) { 38 bool wants_moves) {
42 // Pointer watchers cannot be added multiple times. 39 // Pointer watchers cannot be added multiple times.
43 DCHECK(!move_watchers_.HasObserver(watcher)); 40 DCHECK(!move_watchers_.HasObserver(watcher));
44 DCHECK(!non_move_watchers_.HasObserver(watcher)); 41 DCHECK(!non_move_watchers_.HasObserver(watcher));
45 if (wants_moves) { 42 if (wants_moves) {
46 move_watchers_.AddObserver(watcher); 43 move_watchers_.AddObserver(watcher);
47 if (event_types_ != EventTypes::MOVE_EVENTS) { 44 if (event_types_ != EventTypes::MOVE_EVENTS) {
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
154 observer.OnPointerEventObserved(event, location_in_screen, nullptr); 151 observer.OnPointerEventObserved(event, location_in_screen, nullptr);
155 for (PointerWatcher& observer : non_move_watchers_) 152 for (PointerWatcher& observer : non_move_watchers_)
156 observer.OnPointerEventObserved(event, location_in_screen, nullptr); 153 observer.OnPointerEventObserved(event, location_in_screen, nullptr);
157 } 154 }
158 155
159 void PointerWatcherEventRouter::OnDidDestroyClient( 156 void PointerWatcherEventRouter::OnDidDestroyClient(
160 aura::WindowTreeClient* client) { 157 aura::WindowTreeClient* client) {
161 // We expect that all observers have been removed by this time. 158 // We expect that all observers have been removed by this time.
162 DCHECK_EQ(event_types_, EventTypes::NONE); 159 DCHECK_EQ(event_types_, EventTypes::NONE);
163 DCHECK_EQ(client, window_tree_client_); 160 DCHECK_EQ(client, window_tree_client_);
164 window_tree_client_->GetCaptureClient()->RemoveObserver(this);
165 window_tree_client_->RemoveObserver(this); 161 window_tree_client_->RemoveObserver(this);
166 window_tree_client_ = nullptr; 162 window_tree_client_ = nullptr;
167 } 163 }
168 164
169 } // namespace views 165 } // namespace views
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698