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

Unified Diff: content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc

Issue 2789753002: Convert offscreen canvas to use FrameSinkManagerHost. (Closed)
Patch Set: Rebase. 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc
diff --git a/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc b/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc
deleted file mode 100644
index 97f1fcfb9acbd3ab6f6d781c98dc9f56f7980a38..0000000000000000000000000000000000000000
--- a/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc
+++ /dev/null
@@ -1,88 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h"
-
-#include "base/memory/ptr_util.h"
-#include "cc/surfaces/surface.h"
-#include "cc/surfaces/surface_manager.h"
-#include "content/browser/renderer_host/offscreen_canvas_compositor_frame_sink_provider_impl.h"
-#include "mojo/public/cpp/bindings/strong_binding.h"
-
-namespace content {
-
-OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink(
- OffscreenCanvasCompositorFrameSinkProviderImpl* provider,
- const cc::FrameSinkId& frame_sink_id,
- cc::mojom::MojoCompositorFrameSinkRequest request,
- cc::mojom::MojoCompositorFrameSinkClientPtr client)
- : provider_(provider),
- support_(cc::CompositorFrameSinkSupport::Create(
- this,
- provider->GetSurfaceManager(),
- frame_sink_id,
- false /* is_root */,
- true /* handles_frame_sink_id_invalidation */,
- true /* needs_sync_points */)),
- client_(std::move(client)),
- binding_(this, std::move(request)) {
- binding_.set_connection_error_handler(
- base::Bind(&OffscreenCanvasCompositorFrameSink::OnClientConnectionLost,
- base::Unretained(this)));
-}
-
-OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() {
- provider_->OnCompositorFrameSinkClientDestroyed(support_->frame_sink_id());
-}
-
-void OffscreenCanvasCompositorFrameSink::SetNeedsBeginFrame(
- bool needs_begin_frame) {
- support_->SetNeedsBeginFrame(needs_begin_frame);
-}
-
-void OffscreenCanvasCompositorFrameSink::SubmitCompositorFrame(
- const cc::LocalSurfaceId& local_surface_id,
- cc::CompositorFrame frame) {
- // TODO(samans): This will need to do something similar to
- // GpuCompositorFrameSink.
- support_->SubmitCompositorFrame(local_surface_id, std::move(frame));
-}
-
-void OffscreenCanvasCompositorFrameSink::BeginFrameDidNotSwap(
- const cc::BeginFrameAck& begin_frame_ack) {
- support_->BeginFrameDidNotSwap(begin_frame_ack);
-}
-
-void OffscreenCanvasCompositorFrameSink::EvictFrame() {
- support_->EvictFrame();
-}
-
-void OffscreenCanvasCompositorFrameSink::DidReceiveCompositorFrameAck(
- const cc::ReturnedResourceArray& resources) {
- if (client_)
- client_->DidReceiveCompositorFrameAck(resources);
-}
-
-void OffscreenCanvasCompositorFrameSink::OnBeginFrame(
- const cc::BeginFrameArgs& args) {
- if (client_)
- client_->OnBeginFrame(args);
-}
-
-void OffscreenCanvasCompositorFrameSink::ReclaimResources(
- const cc::ReturnedResourceArray& resources) {
- if (client_)
- client_->ReclaimResources(resources);
-}
-
-void OffscreenCanvasCompositorFrameSink::WillDrawSurface(
- const cc::LocalSurfaceId& local_surface_id,
- const gfx::Rect& damage_rect) {}
-
-void OffscreenCanvasCompositorFrameSink::OnClientConnectionLost() {
- provider_->OnCompositorFrameSinkClientConnectionLost(
- support_->frame_sink_id());
-}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698