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

Side by Side Diff: ui/aura/mus/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 "ui/aura/mus/window_compositor_frame_sink.h" 5 #include "ui/aura/mus/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_sink_client.h" 9 #include "cc/output/compositor_frame_sink_client.h"
10 #include "gpu/ipc/client/gpu_channel_host.h" 10 #include "gpu/ipc/client/gpu_channel_host.h"
11 11
12 namespace aura { 12 namespace aura {
13 13
14 // static 14 // static
15 std::unique_ptr<WindowCompositorFrameSink> WindowCompositorFrameSink::Create( 15 std::unique_ptr<WindowCompositorFrameSink> WindowCompositorFrameSink::Create(
16 cc::FrameSinkId frame_sink_id,
Fady Samuel 2017/01/13 20:07:06 const cc::FrameSinkId&
16 scoped_refptr<cc::ContextProvider> context_provider, 17 scoped_refptr<cc::ContextProvider> context_provider,
17 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 18 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
18 std::unique_ptr<WindowCompositorFrameSinkBinding>* 19 std::unique_ptr<WindowCompositorFrameSinkBinding>*
19 compositor_frame_sink_binding) { 20 compositor_frame_sink_binding) {
20 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink; 21 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink;
21 cc::mojom::MojoCompositorFrameSinkClientPtr compositor_frame_sink_client; 22 cc::mojom::MojoCompositorFrameSinkClientPtr compositor_frame_sink_client;
22 cc::mojom::MojoCompositorFrameSinkClientRequest 23 cc::mojom::MojoCompositorFrameSinkClientRequest
23 compositor_frame_sink_client_request = 24 compositor_frame_sink_client_request =
24 MakeRequest(&compositor_frame_sink_client); 25 MakeRequest(&compositor_frame_sink_client);
25 26
26 compositor_frame_sink_binding->reset(new WindowCompositorFrameSinkBinding( 27 compositor_frame_sink_binding->reset(new WindowCompositorFrameSinkBinding(
27 MakeRequest(&compositor_frame_sink), 28 MakeRequest(&compositor_frame_sink),
28 compositor_frame_sink_client.PassInterface())); 29 compositor_frame_sink_client.PassInterface()));
29 return base::WrapUnique(new WindowCompositorFrameSink( 30 return base::WrapUnique(new WindowCompositorFrameSink(
30 std::move(context_provider), gpu_memory_buffer_manager, 31 frame_sink_id, std::move(context_provider), gpu_memory_buffer_manager,
31 compositor_frame_sink.PassInterface(), 32 compositor_frame_sink.PassInterface(),
32 std::move(compositor_frame_sink_client_request))); 33 std::move(compositor_frame_sink_client_request)));
33 } 34 }
34 35
35 WindowCompositorFrameSink::~WindowCompositorFrameSink() {} 36 WindowCompositorFrameSink::~WindowCompositorFrameSink() {}
36 37
37 bool WindowCompositorFrameSink::BindToClient( 38 bool WindowCompositorFrameSink::BindToClient(
38 cc::CompositorFrameSinkClient* client) { 39 cc::CompositorFrameSinkClient* client) {
39 if (!cc::CompositorFrameSink::BindToClient(client)) 40 if (!cc::CompositorFrameSink::BindToClient(client))
40 return false; 41 return false;
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 if (!local_frame_id_.is_valid() || frame_size != last_submitted_frame_size_) 73 if (!local_frame_id_.is_valid() || frame_size != last_submitted_frame_size_)
73 local_frame_id_ = id_allocator_.GenerateId(); 74 local_frame_id_ = id_allocator_.GenerateId();
74 75
75 compositor_frame_sink_->SubmitCompositorFrame(local_frame_id_, 76 compositor_frame_sink_->SubmitCompositorFrame(local_frame_id_,
76 std::move(frame)); 77 std::move(frame));
77 78
78 last_submitted_frame_size_ = frame_size; 79 last_submitted_frame_size_ = frame_size;
79 } 80 }
80 81
81 WindowCompositorFrameSink::WindowCompositorFrameSink( 82 WindowCompositorFrameSink::WindowCompositorFrameSink(
83 cc::FrameSinkId frame_sink_id,
Fady Samuel 2017/01/13 20:07:06 const cc::FrameSinkId&
82 scoped_refptr<cc::ContextProvider> context_provider, 84 scoped_refptr<cc::ContextProvider> context_provider,
83 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 85 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
84 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info, 86 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info,
85 cc::mojom::MojoCompositorFrameSinkClientRequest client_request) 87 cc::mojom::MojoCompositorFrameSinkClientRequest client_request)
86 : cc::CompositorFrameSink(std::move(context_provider), 88 : cc::CompositorFrameSink(std::move(context_provider),
87 nullptr, 89 nullptr,
88 gpu_memory_buffer_manager, 90 gpu_memory_buffer_manager,
89 nullptr), 91 nullptr),
90 compositor_frame_sink_info_(std::move(compositor_frame_sink_info)), 92 compositor_frame_sink_info_(std::move(compositor_frame_sink_info)),
91 client_request_(std::move(client_request)) {} 93 client_request_(std::move(client_request)),
94 frame_sink_id_(frame_sink_id) {}
92 95
93 void WindowCompositorFrameSink::DidReceiveCompositorFrameAck() { 96 void WindowCompositorFrameSink::DidReceiveCompositorFrameAck() {
94 DCHECK(thread_checker_); 97 DCHECK(thread_checker_);
95 DCHECK(thread_checker_->CalledOnValidThread()); 98 DCHECK(thread_checker_->CalledOnValidThread());
96 if (!client_) 99 if (!client_)
97 return; 100 return;
98 client_->DidReceiveCompositorFrameAck(); 101 client_->DidReceiveCompositorFrameAck();
99 } 102 }
100 103
101 void WindowCompositorFrameSink::OnBeginFrame( 104 void WindowCompositorFrameSink::OnBeginFrame(
(...skipping 21 matching lines...) Expand all
123 WindowCompositorFrameSinkBinding::~WindowCompositorFrameSinkBinding() {} 126 WindowCompositorFrameSinkBinding::~WindowCompositorFrameSinkBinding() {}
124 127
125 WindowCompositorFrameSinkBinding::WindowCompositorFrameSinkBinding( 128 WindowCompositorFrameSinkBinding::WindowCompositorFrameSinkBinding(
126 cc::mojom::MojoCompositorFrameSinkRequest compositor_frame_sink_request, 129 cc::mojom::MojoCompositorFrameSinkRequest compositor_frame_sink_request,
127 mojo::InterfacePtrInfo<cc::mojom::MojoCompositorFrameSinkClient> 130 mojo::InterfacePtrInfo<cc::mojom::MojoCompositorFrameSinkClient>
128 compositor_frame_sink_client) 131 compositor_frame_sink_client)
129 : compositor_frame_sink_request_(std::move(compositor_frame_sink_request)), 132 : compositor_frame_sink_request_(std::move(compositor_frame_sink_request)),
130 compositor_frame_sink_client_(std::move(compositor_frame_sink_client)) {} 133 compositor_frame_sink_client_(std::move(compositor_frame_sink_client)) {}
131 134
132 } // namespace aura 135 } // namespace aura
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698