OLD | NEW |
---|---|
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 "cc/surfaces/surface.h" | 7 #include "cc/surfaces/surface.h" |
8 #include "cc/surfaces/surface_manager.h" | 8 #include "cc/surfaces/surface_manager.h" |
9 #include "content/browser/compositor/surface_utils.h" | 9 #include "content/browser/compositor/surface_utils.h" |
10 #include "content/browser/renderer_host/offscreen_canvas_surface_manager.h" | |
10 #include "mojo/public/cpp/bindings/strong_binding.h" | 11 #include "mojo/public/cpp/bindings/strong_binding.h" |
11 | 12 |
12 namespace content { | 13 namespace content { |
13 | 14 |
14 OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink( | 15 OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink( |
15 const cc::SurfaceId& surface_id, | 16 const cc::SurfaceId& surface_id, |
16 cc::mojom::MojoCompositorFrameSinkClientPtr client) | 17 cc::mojom::MojoCompositorFrameSinkClientPtr client) |
17 : surface_id_(surface_id), client_(std::move(client)) { | 18 : surface_id_(surface_id), client_(std::move(client)), weak_factory_(this) { |
18 cc::SurfaceManager* manager = GetSurfaceManager(); | 19 cc::SurfaceManager* manager = GetSurfaceManager(); |
19 surface_factory_ = base::MakeUnique<cc::SurfaceFactory>( | 20 surface_factory_ = base::MakeUnique<cc::SurfaceFactory>( |
20 surface_id_.frame_sink_id(), manager, this); | 21 surface_id_.frame_sink_id(), manager, this); |
21 manager->RegisterFrameSinkId(surface_id_.frame_sink_id()); | 22 manager->RegisterFrameSinkId(surface_id_.frame_sink_id()); |
22 surface_factory_->Create(surface_id_.local_frame_id()); | 23 surface_factory_->Create(surface_id_.local_frame_id()); |
24 | |
25 DCHECK(OffscreenCanvasSurfaceManager::GetInstance()->GetSurfaceInstance( | |
Fady Samuel
2016/11/14 21:15:40
Can we get rid of this?
xlai (Olivia)
2016/11/15 17:16:18
Done.
| |
26 surface_id_.frame_sink_id())); | |
dcheng
2016/11/15 07:28:29
Do we need to check somewhere higher in the stack
xlai (Olivia)
2016/11/15 17:16:18
I'm removing it now. I think this DCHECK is better
| |
23 } | 27 } |
24 | 28 |
25 OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() { | 29 OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() { |
26 cc::SurfaceManager* manager = GetSurfaceManager(); | 30 cc::SurfaceManager* manager = GetSurfaceManager(); |
27 if (!manager) { | 31 if (!manager) { |
28 // Inform SurfaceFactory that SurfaceManager's no longer alive to | 32 // Inform SurfaceFactory that SurfaceManager's no longer alive to |
29 // avoid its destruction error. | 33 // avoid its destruction error. |
30 surface_factory_->DidDestroySurfaceManager(); | 34 surface_factory_->DidDestroySurfaceManager(); |
31 } else { | 35 } else { |
32 manager->InvalidateFrameSinkId(surface_id_.frame_sink_id()); | 36 manager->InvalidateFrameSinkId(surface_id_.frame_sink_id()); |
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
86 client_->DidReceiveCompositorFrameAck(); | 90 client_->DidReceiveCompositorFrameAck(); |
87 DCHECK_GT(ack_pending_count_, 0); | 91 DCHECK_GT(ack_pending_count_, 0); |
88 if (!surface_returned_resources_.empty()) { | 92 if (!surface_returned_resources_.empty()) { |
89 client_->ReclaimResources(surface_returned_resources_); | 93 client_->ReclaimResources(surface_returned_resources_); |
90 surface_returned_resources_.clear(); | 94 surface_returned_resources_.clear(); |
91 } | 95 } |
92 ack_pending_count_--; | 96 ack_pending_count_--; |
93 } | 97 } |
94 | 98 |
95 } // namespace content | 99 } // namespace content |
OLD | NEW |