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

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

Issue 2612083002: DirectCompositorFrameSink Uses CompositorFrameSinkSupport (Closed)
Patch Set: Use MakeUnique 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 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
index c0409911d5be6d7280b862d340d1f7044089a8ae..102617f14a7051cdeb7f4893d712a19f0194562a 100644
--- a/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc
+++ b/content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.cc
@@ -21,7 +21,6 @@ OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink(
support_(this,
provider->GetSurfaceManager(),
frame_sink_id,
- nullptr,
nullptr),
client_(std::move(client)),
binding_(this, std::move(request)) {
@@ -30,7 +29,9 @@ OffscreenCanvasCompositorFrameSink::OffscreenCanvasCompositorFrameSink(
base::Unretained(this)));
}
-OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() {}
+OffscreenCanvasCompositorFrameSink::~OffscreenCanvasCompositorFrameSink() {
+ support_.InvalidateFrameSinkId();
+}
void OffscreenCanvasCompositorFrameSink::SetNeedsBeginFrame(
bool needs_begin_frame) {

Powered by Google App Engine
This is Rietveld 408576698