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

Unified Diff: third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp

Issue 2584643002: Revamp OffscreenCanvas commit flow (Closed)
Patch Set: rebase and fix conflict Created 4 years 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: third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp b/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
index 9be6d9b98f857c255187bec8021c143920a89ad0..f1c74dcbd8e1291dd2d048516575a42535baae9a 100644
--- a/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
+++ b/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
@@ -30,27 +30,21 @@ namespace blink {
OffscreenCanvasFrameDispatcherImpl::OffscreenCanvasFrameDispatcherImpl(
uint32_t clientId,
uint32_t sinkId,
- uint32_t localId,
- uint64_t nonceHigh,
- uint64_t nonceLow,
int canvasId,
int width,
int height)
: m_frameSinkId(cc::FrameSinkId(clientId, sinkId)),
- m_currentLocalFrameId(cc::LocalFrameId(
- localId,
- base::UnguessableToken::Deserialize(nonceHigh, nonceLow))),
m_width(width),
m_height(height),
m_nextResourceId(1u),
m_binding(this),
m_placeholderCanvasId(canvasId) {
+ m_currentLocalFrameId = m_surfaceIdAllocator.GenerateId();
DCHECK(!m_sink.is_bound());
mojom::blink::OffscreenCanvasCompositorFrameSinkProviderPtr provider;
Platform::current()->interfaceProvider()->getInterface(
mojo::GetProxy(&provider));
- cc::SurfaceId surfaceId(m_frameSinkId, m_currentLocalFrameId);
- provider->CreateCompositorFrameSink(surfaceId,
+ provider->CreateCompositorFrameSink(m_frameSinkId,
m_binding.CreateInterfacePtrAndBind(),
mojo::GetProxy(&m_sink));
}

Powered by Google App Engine
This is Rietveld 408576698