Revamp OffscreenCanvas commit flow

third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp

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 1d001444ce2e754ec7ec735c6ddbb044172acd73..59b2f098b03c9446918658a79e243876f45299f8 100644
--- a/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
+++ b/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
@@ -30,16 +30,12 @@ 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_surfaceIdAllocator(new cc::SurfaceIdAllocator()),

[Fady Samuel, 2016/12/15 19:54:52]

You don't need this on the heap, just make it inplace.

[xlai (Olivia), 2016/12/15 22:20:12]

Done.

+      m_frameSinkId(cc::FrameSinkId(clientId, sinkId)),
+      m_currentLocalFrameId(m_surfaceIdAllocator->GenerateId()),
       m_width(width),
       m_height(height),
       m_nextResourceId(1u),