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

Side by Side Diff: services/ui/public/cpp/window_compositor_frame_sink.cc

Issue 2626023007: Passing FrameSinkId to WindowCompositorFrameSink (Closed)
Patch Set: c Created 3 years, 11 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
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/public/cpp/window_compositor_frame_sink.h" 5 #include "services/ui/public/cpp/window_compositor_frame_sink.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #include "cc/output/compositor_frame.h" 9 #include "cc/output/compositor_frame.h"
10 #include "cc/output/compositor_frame_sink_client.h" 10 #include "cc/output/compositor_frame_sink_client.h"
11 #include "gpu/ipc/client/gpu_channel_host.h" 11 #include "gpu/ipc/client/gpu_channel_host.h"
12 12
13 namespace ui { 13 namespace ui {
14 14
15 // static 15 // static
16 std::unique_ptr<WindowCompositorFrameSink> WindowCompositorFrameSink::Create( 16 std::unique_ptr<WindowCompositorFrameSink> WindowCompositorFrameSink::Create(
17 const cc::FrameSinkId& frame_sink_id,
17 scoped_refptr<cc::ContextProvider> context_provider, 18 scoped_refptr<cc::ContextProvider> context_provider,
18 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 19 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
19 std::unique_ptr<WindowCompositorFrameSinkBinding>* 20 std::unique_ptr<WindowCompositorFrameSinkBinding>*
20 compositor_frame_sink_binding) { 21 compositor_frame_sink_binding) {
21 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink; 22 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink;
22 cc::mojom::MojoCompositorFrameSinkClientPtr compositor_frame_sink_client; 23 cc::mojom::MojoCompositorFrameSinkClientPtr compositor_frame_sink_client;
23 cc::mojom::MojoCompositorFrameSinkClientRequest 24 cc::mojom::MojoCompositorFrameSinkClientRequest
24 compositor_frame_sink_client_request = 25 compositor_frame_sink_client_request =
25 MakeRequest(&compositor_frame_sink_client); 26 MakeRequest(&compositor_frame_sink_client);
26 27
27 compositor_frame_sink_binding->reset(new WindowCompositorFrameSinkBinding( 28 compositor_frame_sink_binding->reset(new WindowCompositorFrameSinkBinding(
28 MakeRequest(&compositor_frame_sink), 29 MakeRequest(&compositor_frame_sink),
29 compositor_frame_sink_client.PassInterface())); 30 compositor_frame_sink_client.PassInterface()));
30 return base::WrapUnique(new WindowCompositorFrameSink( 31 return base::WrapUnique(new WindowCompositorFrameSink(
31 std::move(context_provider), gpu_memory_buffer_manager, 32 frame_sink_id, std::move(context_provider), gpu_memory_buffer_manager,
32 compositor_frame_sink.PassInterface(), 33 compositor_frame_sink.PassInterface(),
33 std::move(compositor_frame_sink_client_request))); 34 std::move(compositor_frame_sink_client_request)));
34 } 35 }
35 36
36 WindowCompositorFrameSink::~WindowCompositorFrameSink() {} 37 WindowCompositorFrameSink::~WindowCompositorFrameSink() {}
37 38
38 bool WindowCompositorFrameSink::BindToClient( 39 bool WindowCompositorFrameSink::BindToClient(
39 cc::CompositorFrameSinkClient* client) { 40 cc::CompositorFrameSinkClient* client) {
40 if (!cc::CompositorFrameSink::BindToClient(client)) 41 if (!cc::CompositorFrameSink::BindToClient(client))
41 return false; 42 return false;
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 if (!local_frame_id_.is_valid() || frame_size != last_submitted_frame_size_) 74 if (!local_frame_id_.is_valid() || frame_size != last_submitted_frame_size_)
74 local_frame_id_ = id_allocator_.GenerateId(); 75 local_frame_id_ = id_allocator_.GenerateId();
75 76
76 compositor_frame_sink_->SubmitCompositorFrame(local_frame_id_, 77 compositor_frame_sink_->SubmitCompositorFrame(local_frame_id_,
77 std::move(frame)); 78 std::move(frame));
78 79
79 last_submitted_frame_size_ = frame_size; 80 last_submitted_frame_size_ = frame_size;
80 } 81 }
81 82
82 WindowCompositorFrameSink::WindowCompositorFrameSink( 83 WindowCompositorFrameSink::WindowCompositorFrameSink(
84 const cc::FrameSinkId& frame_sink_id,
83 scoped_refptr<cc::ContextProvider> context_provider, 85 scoped_refptr<cc::ContextProvider> context_provider,
84 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 86 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
85 mojo::InterfacePtrInfo<cc::mojom::MojoCompositorFrameSink> 87 mojo::InterfacePtrInfo<cc::mojom::MojoCompositorFrameSink>
86 compositor_frame_sink_info, 88 compositor_frame_sink_info,
87 cc::mojom::MojoCompositorFrameSinkClientRequest client_request) 89 cc::mojom::MojoCompositorFrameSinkClientRequest client_request)
88 : cc::CompositorFrameSink(std::move(context_provider), 90 : cc::CompositorFrameSink(std::move(context_provider),
89 nullptr, 91 nullptr,
90 gpu_memory_buffer_manager, 92 gpu_memory_buffer_manager,
91 nullptr), 93 nullptr),
92 compositor_frame_sink_info_(std::move(compositor_frame_sink_info)), 94 compositor_frame_sink_info_(std::move(compositor_frame_sink_info)),
93 client_request_(std::move(client_request)) {} 95 client_request_(std::move(client_request)),
96 frame_sink_id_(frame_sink_id) {}
94 97
95 void WindowCompositorFrameSink::DidReceiveCompositorFrameAck() { 98 void WindowCompositorFrameSink::DidReceiveCompositorFrameAck() {
96 DCHECK(thread_checker_); 99 DCHECK(thread_checker_);
97 DCHECK(thread_checker_->CalledOnValidThread()); 100 DCHECK(thread_checker_->CalledOnValidThread());
98 if (!client_) 101 if (!client_)
99 return; 102 return;
100 client_->DidReceiveCompositorFrameAck(); 103 client_->DidReceiveCompositorFrameAck();
101 } 104 }
102 105
103 void WindowCompositorFrameSink::OnBeginFrame( 106 void WindowCompositorFrameSink::OnBeginFrame(
(...skipping 21 matching lines...) Expand all
125 WindowCompositorFrameSinkBinding::~WindowCompositorFrameSinkBinding() {} 128 WindowCompositorFrameSinkBinding::~WindowCompositorFrameSinkBinding() {}
126 129
127 WindowCompositorFrameSinkBinding::WindowCompositorFrameSinkBinding( 130 WindowCompositorFrameSinkBinding::WindowCompositorFrameSinkBinding(
128 cc::mojom::MojoCompositorFrameSinkRequest compositor_frame_sink_request, 131 cc::mojom::MojoCompositorFrameSinkRequest compositor_frame_sink_request,
129 mojo::InterfacePtrInfo<cc::mojom::MojoCompositorFrameSinkClient> 132 mojo::InterfacePtrInfo<cc::mojom::MojoCompositorFrameSinkClient>
130 compositor_frame_sink_client) 133 compositor_frame_sink_client)
131 : compositor_frame_sink_request_(std::move(compositor_frame_sink_request)), 134 : compositor_frame_sink_request_(std::move(compositor_frame_sink_request)),
132 compositor_frame_sink_client_(std::move(compositor_frame_sink_client)) {} 135 compositor_frame_sink_client_(std::move(compositor_frame_sink_client)) {}
133 136
134 } // namespace ui 137 } // namespace ui
OLDNEW
« no previous file with comments | « services/ui/public/cpp/window_compositor_frame_sink.h ('k') | ui/aura/mus/window_compositor_frame_sink.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698