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

Side by Side Diff: services/ui/ws/window_tree_client_unittest.cc

Issue 2346253004: Makes display roots be destroyed by client (Closed)
Patch Set: cleanup Created 4 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
« no previous file with comments | « services/ui/ws/window_tree.cc ('k') | no next file » | 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 406 matching lines...) Expand 10 before | Expand all | Expand 10 after
417 GetProxy(&window_manager_client_, tree_.associated_group())); 417 GetProxy(&window_manager_client_, tree_.associated_group()));
418 } 418 }
419 419
420 // mojom::WindowManager: 420 // mojom::WindowManager:
421 void OnConnect(uint16_t client_id) override {} 421 void OnConnect(uint16_t client_id) override {}
422 void WmNewDisplayAdded(const display::Display& display, 422 void WmNewDisplayAdded(const display::Display& display,
423 mojom::WindowDataPtr root_data, 423 mojom::WindowDataPtr root_data,
424 bool drawn) override { 424 bool drawn) override {
425 NOTIMPLEMENTED(); 425 NOTIMPLEMENTED();
426 } 426 }
427 void WmDisplayRemoved(int64_t display_id) override { NOTIMPLEMENTED(); }
427 void WmSetBounds(uint32_t change_id, 428 void WmSetBounds(uint32_t change_id,
428 uint32_t window_id, 429 uint32_t window_id,
429 const gfx::Rect& bounds) override { 430 const gfx::Rect& bounds) override {
430 window_manager_client_->WmResponse(change_id, false); 431 window_manager_client_->WmResponse(change_id, false);
431 } 432 }
432 void WmSetProperty(uint32_t change_id, 433 void WmSetProperty(uint32_t change_id,
433 uint32_t window_id, 434 uint32_t window_id,
434 const mojo::String& name, 435 const mojo::String& name,
435 mojo::Array<uint8_t> value) override { 436 mojo::Array<uint8_t> value) override {
436 window_manager_client_->WmResponse(change_id, false); 437 window_manager_client_->WmResponse(change_id, false);
(...skipping 1673 matching lines...) Expand 10 before | Expand all | Expand 10 after
2110 2111
2111 // TODO(sky): make sure coverage of what was 2112 // TODO(sky): make sure coverage of what was
2112 // WindowManagerTest.SecondEmbedRoot_InitService and 2113 // WindowManagerTest.SecondEmbedRoot_InitService and
2113 // WindowManagerTest.MultipleEmbedRootsBeforeWTHReady gets added to window 2114 // WindowManagerTest.MultipleEmbedRootsBeforeWTHReady gets added to window
2114 // manager 2115 // manager
2115 // tests. 2116 // tests.
2116 2117
2117 } // namespace test 2118 } // namespace test
2118 } // namespace ws 2119 } // namespace ws
2119 } // namespace ui 2120 } // namespace ui
OLDNEW
« no previous file with comments | « services/ui/ws/window_tree.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698