Chromium Code Reviews| Index: services/ui/public/cpp/window_tree_client.cc |
| diff --git a/services/ui/public/cpp/window_tree_client.cc b/services/ui/public/cpp/window_tree_client.cc |
| index b37d6bc846350de1af9ee0f288cc42a04fb49bbc..b12265479eacccd3882be3f8f87430e0ee597f42 100644 |
| --- a/services/ui/public/cpp/window_tree_client.cc |
| +++ b/services/ui/public/cpp/window_tree_client.cc |
| @@ -632,10 +632,7 @@ void WindowTreeClient::StartPointerWatcher(bool want_moves) { |
| if (has_pointer_watcher_) |
| StopPointerWatcher(); |
| has_pointer_watcher_ = true; |
| - pointer_watcher_id_++; |
| - if (pointer_watcher_id_ == 0) |
| - pointer_watcher_id_++; |
| - tree_->StartPointerWatcher(want_moves, pointer_watcher_id_); |
| + tree_->StartPointerWatcher(want_moves); |
| } |
| void WindowTreeClient::StopPointerWatcher() { |
| @@ -981,14 +978,13 @@ void WindowTreeClient::OnWindowSharedPropertyChanged( |
| void WindowTreeClient::OnWindowInputEvent(uint32_t event_id, |
| Id window_id, |
| std::unique_ptr<ui::Event> event, |
| - uint32_t pointer_watcher_id) { |
| + bool matches_pointer_watcher) { |
| DCHECK(event); |
| Window* window = GetWindowByServerId(window_id); // May be null. |
| // Non-zero pointer_watcher_id means it matched a pointer watcher on the |
|
msw
2016/08/18 22:45:23
nit: update comment
sky
2016/08/18 23:17:14
I just removed the comment as the code isn't parti
|
| // server. |
| - if (pointer_watcher_id_ != 0 && has_pointer_watcher_ && |
| - pointer_watcher_id == pointer_watcher_id_) { |
| + if (matches_pointer_watcher && has_pointer_watcher_) { |
| DCHECK(event->IsPointerEvent()); |
| delegate_->OnPointerEventObserved(*event->AsPointerEvent(), window); |
| } |
| @@ -1023,11 +1019,10 @@ void WindowTreeClient::OnWindowInputEvent(uint32_t event_id, |
| } |
| void WindowTreeClient::OnPointerEventObserved(std::unique_ptr<ui::Event> event, |
| - uint32_t pointer_watcher_id, |
| uint32_t window_id) { |
| DCHECK(event); |
| DCHECK(event->IsPointerEvent()); |
| - if (has_pointer_watcher_ && pointer_watcher_id == pointer_watcher_id_) { |
| + if (has_pointer_watcher_) { |
| Window* target_window = GetWindowByServerId(window_id); |
| delegate_->OnPointerEventObserved(*event->AsPointerEvent(), target_window); |
| } |