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

Side by Side Diff: content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc

Issue 2807653003: Move Work From CompositorFrameSinkSupport() To Init() (Closed)
Patch Set: Add CompositorFrameSinkSupport::Create Created 3 years, 8 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 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 #include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h " 5 #include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h "
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "cc/surfaces/surface.h" 8 #include "cc/surfaces/surface.h"
9 #include "cc/surfaces/surface_manager.h" 9 #include "cc/surfaces/surface_manager.h"
10 #include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_p rovider_impl.h" 10 #include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_p rovider_impl.h"
11 #include "mojo/public/cpp/bindings/strong_binding.h" 11 #include "mojo/public/cpp/bindings/strong_binding.h"
12 12
13 namespace content { 13 namespace content {
14 14
15 OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink( 15 OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink(
16 OffscreenCanvasCompositorFrameSinkProviderImpl* provider, 16 OffscreenCanvasCompositorFrameSinkProviderImpl* provider,
17 const cc::FrameSinkId& frame_sink_id, 17 const cc::FrameSinkId& frame_sink_id,
18 cc::mojom::MojoCompositorFrameSinkRequest request, 18 cc::mojom::MojoCompositorFrameSinkRequest request,
19 cc::mojom::MojoCompositorFrameSinkClientPtr client) 19 cc::mojom::MojoCompositorFrameSinkClientPtr client)
20 : provider_(provider), 20 : provider_(provider),
21 support_(this, 21 support_(cc::CompositorFrameSinkSupport::Create(
22 provider->GetSurfaceManager(), 22 this,
23 frame_sink_id, 23 provider->GetSurfaceManager(),
24 false /* is_root */, 24 frame_sink_id,
25 true /* handles_frame_sink_id_invalidation */, 25 false /* is_root */,
26 true /* needs_sync_points */), 26 true /* handles_frame_sink_id_invalidation */,
27 true /* needs_sync_points */)),
27 client_(std::move(client)), 28 client_(std::move(client)),
28 binding_(this, std::move(request)) { 29 binding_(this, std::move(request)) {
29 binding_.set_connection_error_handler( 30 binding_.set_connection_error_handler(
30 base::Bind(&OffscreenCanvasCompositorFrameSink::OnClientConnectionLost, 31 base::Bind(&OffscreenCanvasCompositorFrameSink::OnClientConnectionLost,
31 base::Unretained(this))); 32 base::Unretained(this)));
32 } 33 }
33 34
34 OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() { 35 OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() {
35 provider_->OnCompositorFrameSinkClientDestroyed(support_.frame_sink_id()); 36 provider_->OnCompositorFrameSinkClientDestroyed(support_->frame_sink_id());
36 } 37 }
37 38
38 void OffscreenCanvasCompositorFrameSink::SetNeedsBeginFrame( 39 void OffscreenCanvasCompositorFrameSink::SetNeedsBeginFrame(
39 bool needs_begin_frame) { 40 bool needs_begin_frame) {
40 support_.SetNeedsBeginFrame(needs_begin_frame); 41 support_->SetNeedsBeginFrame(needs_begin_frame);
41 } 42 }
42 43
43 void OffscreenCanvasCompositorFrameSink::SubmitCompositorFrame( 44 void OffscreenCanvasCompositorFrameSink::SubmitCompositorFrame(
44 const cc::LocalSurfaceId& local_surface_id, 45 const cc::LocalSurfaceId& local_surface_id,
45 cc::CompositorFrame frame) { 46 cc::CompositorFrame frame) {
46 // TODO(samans): This will need to do something similar to 47 // TODO(samans): This will need to do something similar to
47 // GpuCompositorFrameSink. 48 // GpuCompositorFrameSink.
48 support_.SubmitCompositorFrame(local_surface_id, std::move(frame)); 49 support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
49 } 50 }
50 51
51 void OffscreenCanvasCompositorFrameSink::BeginFrameDidNotSwap( 52 void OffscreenCanvasCompositorFrameSink::BeginFrameDidNotSwap(
52 const cc::BeginFrameAck& begin_frame_ack) { 53 const cc::BeginFrameAck& begin_frame_ack) {
53 support_.BeginFrameDidNotSwap(begin_frame_ack); 54 support_->BeginFrameDidNotSwap(begin_frame_ack);
54 } 55 }
55 56
56 void OffscreenCanvasCompositorFrameSink::EvictFrame() { 57 void OffscreenCanvasCompositorFrameSink::EvictFrame() {
57 support_.EvictFrame(); 58 support_->EvictFrame();
58 } 59 }
59 60
60 void OffscreenCanvasCompositorFrameSink::DidReceiveCompositorFrameAck() { 61 void OffscreenCanvasCompositorFrameSink::DidReceiveCompositorFrameAck() {
61 if (client_) 62 if (client_)
62 client_->DidReceiveCompositorFrameAck(); 63 client_->DidReceiveCompositorFrameAck();
63 } 64 }
64 65
65 void OffscreenCanvasCompositorFrameSink::OnBeginFrame( 66 void OffscreenCanvasCompositorFrameSink::OnBeginFrame(
66 const cc::BeginFrameArgs& args) { 67 const cc::BeginFrameArgs& args) {
67 if (client_) 68 if (client_)
68 client_->OnBeginFrame(args); 69 client_->OnBeginFrame(args);
69 } 70 }
70 71
71 void OffscreenCanvasCompositorFrameSink::ReclaimResources( 72 void OffscreenCanvasCompositorFrameSink::ReclaimResources(
72 const cc::ReturnedResourceArray& resources) { 73 const cc::ReturnedResourceArray& resources) {
73 if (client_) 74 if (client_)
74 client_->ReclaimResources(resources); 75 client_->ReclaimResources(resources);
75 } 76 }
76 77
77 void OffscreenCanvasCompositorFrameSink::WillDrawSurface( 78 void OffscreenCanvasCompositorFrameSink::WillDrawSurface(
78 const cc::LocalSurfaceId& local_surface_id, 79 const cc::LocalSurfaceId& local_surface_id,
79 const gfx::Rect& damage_rect) {} 80 const gfx::Rect& damage_rect) {}
80 81
81 void OffscreenCanvasCompositorFrameSink::OnClientConnectionLost() { 82 void OffscreenCanvasCompositorFrameSink::OnClientConnectionLost() {
82 provider_->OnCompositorFrameSinkClientConnectionLost( 83 provider_->OnCompositorFrameSinkClientConnectionLost(
83 support_.frame_sink_id()); 84 support_->frame_sink_id());
84 } 85 }
85 86
86 } // namespace content 87 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698