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

Side by Side Diff: services/ui/ws/test_utils.h

Issue 2886873002: Only send the FrameSinkId to client when it is necessary (Closed)
Patch Set: Address review issues. Created 3 years, 7 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/test_change_tracker.cc ('k') | services/ui/ws/test_utils.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 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 #ifndef SERVICES_UI_WS_TEST_UTILS_H_ 5 #ifndef SERVICES_UI_WS_TEST_UTILS_H_
6 #define SERVICES_UI_WS_TEST_UTILS_H_ 6 #define SERVICES_UI_WS_TEST_UTILS_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 332 matching lines...) Expand 10 before | Expand all | Expand 10 after
343 int connect_count() const { return connect_count_; } 343 int connect_count() const { return connect_count_; }
344 int display_added_count() const { return display_added_count_; } 344 int display_added_count() const { return display_added_count_; }
345 345
346 private: 346 private:
347 // WindowManager: 347 // WindowManager:
348 void OnConnect(uint16_t client_id) override; 348 void OnConnect(uint16_t client_id) override;
349 void WmNewDisplayAdded( 349 void WmNewDisplayAdded(
350 const display::Display& display, 350 const display::Display& display,
351 ui::mojom::WindowDataPtr root, 351 ui::mojom::WindowDataPtr root,
352 bool drawn, 352 bool drawn,
353 const cc::FrameSinkId& frame_sink_id,
354 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override; 353 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override;
355 void WmDisplayRemoved(int64_t display_id) override; 354 void WmDisplayRemoved(int64_t display_id) override;
356 void WmDisplayModified(const display::Display& display) override {} 355 void WmDisplayModified(const display::Display& display) override {}
357 void WmSetBounds(uint32_t change_id, 356 void WmSetBounds(uint32_t change_id,
358 uint32_t window_id, 357 uint32_t window_id,
359 const gfx::Rect& bounds) override {} 358 const gfx::Rect& bounds) override {}
360 void WmSetProperty( 359 void WmSetProperty(
361 uint32_t change_id, 360 uint32_t change_id,
362 uint32_t window_id, 361 uint32_t window_id,
363 const std::string& name, 362 const std::string& name,
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
427 426
428 private: 427 private:
429 // WindowTreeClient: 428 // WindowTreeClient:
430 void OnEmbed( 429 void OnEmbed(
431 uint16_t client_id, 430 uint16_t client_id,
432 mojom::WindowDataPtr root, 431 mojom::WindowDataPtr root,
433 ui::mojom::WindowTreePtr tree, 432 ui::mojom::WindowTreePtr tree,
434 int64_t display_id, 433 int64_t display_id,
435 Id focused_window_id, 434 Id focused_window_id,
436 bool drawn, 435 bool drawn,
437 const cc::FrameSinkId& frame_sink_id,
438 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override; 436 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override;
439 void OnEmbeddedAppDisconnected(uint32_t window) override; 437 void OnEmbeddedAppDisconnected(uint32_t window) override;
440 void OnUnembed(Id window_id) override; 438 void OnUnembed(Id window_id) override;
441 void OnCaptureChanged(Id new_capture_window_id, 439 void OnCaptureChanged(Id new_capture_window_id,
442 Id old_capture_window_id) override; 440 Id old_capture_window_id) override;
443 void OnFrameSinkIdAllocated(Id window_id, 441 void OnFrameSinkIdAllocated(Id window_id,
444 const cc::FrameSinkId& frame_sink_id) override; 442 const cc::FrameSinkId& frame_sink_id) override;
445 void OnTopLevelCreated( 443 void OnTopLevelCreated(
446 uint32_t change_id, 444 uint32_t change_id,
447 mojom::WindowDataPtr data, 445 mojom::WindowDataPtr data,
448 int64_t display_id, 446 int64_t display_id,
449 bool drawn, 447 bool drawn,
450 const cc::FrameSinkId& frame_sink_id,
451 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override; 448 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override;
452 void OnWindowBoundsChanged( 449 void OnWindowBoundsChanged(
453 uint32_t window, 450 uint32_t window,
454 const gfx::Rect& old_bounds, 451 const gfx::Rect& old_bounds,
455 const gfx::Rect& new_bounds, 452 const gfx::Rect& new_bounds,
456 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override; 453 const base::Optional<cc::LocalSurfaceId>& local_surface_id) override;
457 void OnClientAreaChanged( 454 void OnClientAreaChanged(
458 uint32_t window_id, 455 uint32_t window_id,
459 const gfx::Insets& new_client_area, 456 const gfx::Insets& new_client_area,
460 const std::vector<gfx::Rect>& new_additional_client_areas) override; 457 const std::vector<gfx::Rect>& new_additional_client_areas) override;
(...skipping 265 matching lines...) Expand 10 before | Expand all | Expand 10 after
726 ClientWindowId* client_id = nullptr); 723 ClientWindowId* client_id = nullptr);
727 ServerWindow* NewWindowInTreeWithParent(WindowTree* tree, 724 ServerWindow* NewWindowInTreeWithParent(WindowTree* tree,
728 ServerWindow* parent, 725 ServerWindow* parent,
729 ClientWindowId* client_id = nullptr); 726 ClientWindowId* client_id = nullptr);
730 727
731 } // namespace test 728 } // namespace test
732 } // namespace ws 729 } // namespace ws
733 } // namespace ui 730 } // namespace ui
734 731
735 #endif // SERVICES_UI_WS_TEST_UTILS_H_ 732 #endif // SERVICES_UI_WS_TEST_UTILS_H_
OLDNEW
« no previous file with comments | « services/ui/ws/test_change_tracker.cc ('k') | services/ui/ws/test_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698