Revamp OffscreenCanvas commit flow

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

 // 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 "platform/graphics/CanvasSurfaceLayerBridge.h"
 
+#include "cc/layers/solid_color_layer.h"
 #include "cc/layers/surface_layer.h"
 #include "cc/surfaces/surface_id.h"
 #include "cc/surfaces/surface_sequence.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/mojo/MojoHelper.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebCompositorSupport.h"
 #include "public/platform/WebLayer.h"
 #include "ui/gfx/geometry/size.h"
 #include "wtf/Functional.h"
 
 namespace blink {
 
 CanvasSurfaceLayerBridge::CanvasSurfaceLayerBridge(
-    mojom::blink::OffscreenCanvasSurfacePtr service)
-    : m_service(std::move(service)) {}
+    mojom::blink::OffscreenCanvasSurfacePtr service,
+    CanvasSurfaceLayerBridgeObserver* observer)
+    : m_observer(observer),
+      m_service(std::move(service)),
+      m_binding(this),
+      m_frameSinkId(Platform::current()->generateFrameSinkId()) {
+  // This FrameSinkId will be used throughout the lifecycle of this canvas.
+  m_service->SetClient(m_binding.CreateInterfacePtrAndBind(), m_frameSinkId);
+}
 
-CanvasSurfaceLayerBridge::~CanvasSurfaceLayerBridge() {}
+CanvasSurfaceLayerBridge::~CanvasSurfaceLayerBridge() {
+  m_observer = nullptr;
+}
 
-bool CanvasSurfaceLayerBridge::createSurfaceLayer(int canvasWidth,
-                                                  int canvasHeight) {
-  if (!m_service->GetSurfaceId(&m_surfaceId))
-    return false;
+void CanvasSurfaceLayerBridge::createSolidColorLayer() {
+  m_solidColorLayer = cc::SolidColorLayer::Create();
+  m_solidColorLayer->SetBackgroundColor(SK_ColorTRANSPARENT);
+  m_webLayer = Platform::current()->compositorSupport()->createLayerFromCCLayer(
+      m_solidColorLayer.get());
+  GraphicsLayer::registerContentsLayer(m_webLayer.get());
+}
 
-  cc::SurfaceLayer::SatisfyCallback satisfyCallback =
-      convertToBaseCallback(WTF::bind(
-          &CanvasSurfaceLayerBridge::satisfyCallback, WTF::unretained(this)));
-  cc::SurfaceLayer::RequireCallback requireCallback =
-      convertToBaseCallback(WTF::bind(
-          &CanvasSurfaceLayerBridge::requireCallback, WTF::unretained(this)));
-  m_surfaceLayer = cc::SurfaceLayer::Create(std::move(satisfyCallback),
-                                            std::move(requireCallback));
-  m_surfaceLayer->SetSurfaceId(m_surfaceId, 1.f,
-                               gfx::Size(canvasWidth, canvasHeight));
+void CanvasSurfaceLayerBridge::OnSurfaceCreated(const cc::SurfaceId& surfaceId,
+                                                int32_t width,
+                                                int32_t height,
+                                                float deviceScaleFactor) {
+  if (!m_currentSurfaceId.is_valid() || surfaceId != m_currentSurfaceId) {
+    m_currentSurfaceId = surfaceId;
 
-  m_webLayer = Platform::current()->compositorSupport()->createLayerFromCCLayer(
-      m_surfaceLayer.get());
-  GraphicsLayer::registerContentsLayer(m_webLayer.get());
-  return true;
+    cc::SurfaceLayer::SatisfyCallback satisfyCallback =
+        convertToBaseCallback(WTF::bind(
+            &CanvasSurfaceLayerBridge::satisfyCallback, WTF::unretained(this)));
+    cc::SurfaceLayer::RequireCallback requireCallback =
+        convertToBaseCallback(WTF::bind(
+            &CanvasSurfaceLayerBridge::requireCallback, WTF::unretained(this)));
+    m_surfaceLayer = cc::SurfaceLayer::Create(std::move(satisfyCallback),
+                                              std::move(requireCallback));
+    m_surfaceLayer->SetSurfaceId(m_currentSurfaceId, deviceScaleFactor,
+                                 gfx::Size(width, height));
+
+    GraphicsLayer::unregisterContentsLayer(m_webLayer.get());
+    m_webLayer->removeFromParent();
+
+    m_webLayer =

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

You don't need to create a new layer every time the surfaceId changes. Only when
it was invalid and now is valid.

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

Done. But I'm wondering, when SurfaceId, and possibly dsf, size is changed, do I
just need to call SetSurfaceId function to replace the attribute values on the
existing SurfaceLayer? <== this is a question about resizing functionality not
in this patch.

[Fady Samuel, 2016/12/15 22:26:01]

Yes, you just call SetSurfaceId

+        Platform::current()->compositorSupport()->createLayerFromCCLayer(
+            m_surfaceLayer.get());
+    GraphicsLayer::registerContentsLayer(m_webLayer.get());
+
+    m_observer->OnWebLayerReplaced();
+  }
 }
 
 void CanvasSurfaceLayerBridge::satisfyCallback(
     const cc::SurfaceSequence& sequence) {
   m_service->Satisfy(sequence);
 }
 
 void CanvasSurfaceLayerBridge::requireCallback(
     const cc::SurfaceId& surfaceId,
     const cc::SurfaceSequence& sequence) {
   m_service->Require(surfaceId, sequence);
 }
 
 }  // namespace blink