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

Side by Side Diff: components/mus/ws/window_tree_client_unittest.cc

Issue 1891413002: mash: Convert OnWindowInputEventAck to use an enum for handled status (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: missed a file Created 4 years, 8 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
« no previous file with comments | « components/mus/ws/window_tree.cc ('k') | components/mus/ws/window_tree_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
(...skipping 343 matching lines...) Expand 10 before | Expand all | Expand 10 after
354 } 354 }
355 void OnWindowParentDrawnStateChanged(uint32_t window, bool drawn) override { 355 void OnWindowParentDrawnStateChanged(uint32_t window, bool drawn) override {
356 tracker()->OnWindowParentDrawnStateChanged(window, drawn); 356 tracker()->OnWindowParentDrawnStateChanged(window, drawn);
357 } 357 }
358 void OnWindowInputEvent(uint32_t event_id, 358 void OnWindowInputEvent(uint32_t event_id,
359 Id window_id, 359 Id window_id,
360 EventPtr event) override { 360 EventPtr event) override {
361 // Ack input events to clear the state on the server. These can be received 361 // Ack input events to clear the state on the server. These can be received
362 // during test startup. X11Window::DispatchEvent sends a synthetic move 362 // during test startup. X11Window::DispatchEvent sends a synthetic move
363 // event to notify of entry. 363 // event to notify of entry.
364 tree()->OnWindowInputEventAck(event_id, true); 364 tree()->OnWindowInputEventAck(event_id, mojom::EventResult::HANDLED);
365 // Don't log input events as none of the tests care about them and they 365 // Don't log input events as none of the tests care about them and they
366 // may come in at random points. 366 // may come in at random points.
367 } 367 }
368 void OnWindowSharedPropertyChanged(uint32_t window, 368 void OnWindowSharedPropertyChanged(uint32_t window,
369 const String& name, 369 const String& name,
370 Array<uint8_t> new_data) override { 370 Array<uint8_t> new_data) override {
371 tracker_.OnWindowSharedPropertyChanged(window, name, std::move(new_data)); 371 tracker_.OnWindowSharedPropertyChanged(window, name, std::move(new_data));
372 } 372 }
373 // TODO(sky): add testing coverage. 373 // TODO(sky): add testing coverage.
374 void OnWindowFocused(uint32_t focused_window_id) override {} 374 void OnWindowFocused(uint32_t focused_window_id) override {}
(...skipping 1658 matching lines...) Expand 10 before | Expand all | Expand 10 after
2033 2033
2034 // TODO(sky): make sure coverage of what was 2034 // TODO(sky): make sure coverage of what was
2035 // WindowManagerTest.SecondEmbedRoot_InitService and 2035 // WindowManagerTest.SecondEmbedRoot_InitService and
2036 // WindowManagerTest.MultipleEmbedRootsBeforeWTHReady gets added to window 2036 // WindowManagerTest.MultipleEmbedRootsBeforeWTHReady gets added to window
2037 // manager 2037 // manager
2038 // tests. 2038 // tests.
2039 2039
2040 } // namespace test 2040 } // namespace test
2041 } // namespace ws 2041 } // namespace ws
2042 } // namespace mus 2042 } // namespace mus
OLDNEW
« no previous file with comments | « components/mus/ws/window_tree.cc ('k') | components/mus/ws/window_tree_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698