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

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

Issue 2471503002: Mus+Ash: Unify CompositorFrameSinks (Closed)
Patch Set: Added comment Created 4 years, 1 month 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 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 "services/ui/ws/server_window.h" 5 #include "services/ui/ws/server_window.h"
6 6
7 #include <inttypes.h> 7 #include <inttypes.h>
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include "base/strings/stringprintf.h" 10 #include "base/strings/stringprintf.h"
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 DCHECK(observers_.HasObserver(observer)); 78 DCHECK(observers_.HasObserver(observer));
79 observers_.RemoveObserver(observer); 79 observers_.RemoveObserver(observer);
80 } 80 }
81 81
82 bool ServerWindow::HasObserver(ServerWindowObserver* observer) { 82 bool ServerWindow::HasObserver(ServerWindowObserver* observer) {
83 return observers_.HasObserver(observer); 83 return observers_.HasObserver(observer);
84 } 84 }
85 85
86 void ServerWindow::CreateCompositorFrameSink( 86 void ServerWindow::CreateCompositorFrameSink(
87 mojom::CompositorFrameSinkType compositor_frame_sink_type, 87 mojom::CompositorFrameSinkType compositor_frame_sink_type,
88 gfx::AcceleratedWidget widget,
89 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
90 scoped_refptr<SurfacesContextProvider> context_provider,
88 cc::mojom::MojoCompositorFrameSinkRequest request, 91 cc::mojom::MojoCompositorFrameSinkRequest request,
89 cc::mojom::MojoCompositorFrameSinkClientPtr client) { 92 cc::mojom::MojoCompositorFrameSinkClientPtr client) {
90 GetOrCreateCompositorFrameSinkManager()->CreateCompositorFrameSink( 93 GetOrCreateCompositorFrameSinkManager()->CreateCompositorFrameSink(
91 compositor_frame_sink_type, std::move(request), std::move(client)); 94 compositor_frame_sink_type, widget, gpu_memory_buffer_manager,
95 std::move(context_provider), std::move(request), std::move(client));
92 } 96 }
93 97
94 void ServerWindow::Add(ServerWindow* child) { 98 void ServerWindow::Add(ServerWindow* child) {
95 // We assume validation checks happened already. 99 // We assume validation checks happened already.
96 DCHECK(child); 100 DCHECK(child);
97 DCHECK(child != this); 101 DCHECK(child != this);
98 DCHECK(!child->Contains(this)); 102 DCHECK(!child->Contains(this));
99 if (child->parent() == this) { 103 if (child->parent() == this) {
100 if (children_.size() == 1) 104 if (children_.size() == 1)
101 return; // Already in the right position. 105 return; // Already in the right position.
(...skipping 355 matching lines...) Expand 10 before | Expand all | Expand 10 after
457 } 461 }
458 462
459 // static 463 // static
460 ServerWindow** ServerWindow::GetStackingTarget(ServerWindow* window) { 464 ServerWindow** ServerWindow::GetStackingTarget(ServerWindow* window) {
461 return &window->stacking_target_; 465 return &window->stacking_target_;
462 } 466 }
463 467
464 } // namespace ws 468 } // namespace ws
465 469
466 } // namespace ui 470 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698