Streamline the presentation of ImageBitmapRenderingContext

third_party/WebKit/Source/modules/imagebitmap/ImageBitmapRenderingContext.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 "modules/imagebitmap/ImageBitmapRenderingContext.h"
 
 #include "bindings/modules/v8/RenderingContext.h"
 #include "core/frame/ImageBitmap.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "platform/graphics/StaticBitmapImage.h"
+#include "platform/graphics/gpu/ImageLayerBridge.h"
 #include "third_party/skia/include/core/SkImage.h"
 #include "third_party/skia/include/core/SkSurface.h"
 
 namespace blink {
 
 ImageBitmapRenderingContext::ImageBitmapRenderingContext(
     HTMLCanvasElement* canvas,
     const CanvasContextCreationAttributes& attrs,
     Document& document)
-    : CanvasRenderingContext(canvas, nullptr, attrs) {}
+    : CanvasRenderingContext(canvas, nullptr, attrs),
+      m_imageLayerBridge(
+          new ImageLayerBridge(attrs.alpha() ? NonOpaque : Opaque)) {}
 
 ImageBitmapRenderingContext::~ImageBitmapRenderingContext() {}
 
 void ImageBitmapRenderingContext::setCanvasGetContextResult(
     RenderingContext& result) {
   result.setImageBitmapRenderingContext(this);
 }
 
 void ImageBitmapRenderingContext::transferFromImageBitmap(
     ImageBitmap* imageBitmap,
     ExceptionState& exceptionState) {
-  if (!imageBitmap) {
-    m_image.release();
-    return;
-  }
-
-  if (imageBitmap->isNeutered()) {
+  if (imageBitmap && imageBitmap->isNeutered()) {
     exceptionState.throwDOMException(InvalidStateError,
                                      "The input ImageBitmap has been detached");
     return;
   }
 
-  m_image = imageBitmap->bitmapImage();
-  if (!m_image)
-    return;
+  m_imageLayerBridge->setImage(imageBitmap ? imageBitmap->bitmapImage()
+                                           : nullptr);
 
-  // TODO(ccameron): Determine the correct color behavior here.

[xlai (Olivia), 2017/03/09 21:57:27]

I think you accidentally took away this TODO comment when you transfer part of
the code to the new imagelayerbridge.

[Justin Novosad, 2017/03/10 21:09:59]

Don't worry, ccameron removed this concurently in another CL that just landed.

-  // ImageBitmapRenderingContext.
-  // https://crbug.com/672306
-  sk_sp<SkImage> skImage =
-      m_image->imageForCurrentFrame(ColorBehavior::transformToGlobalTarget());
-  if (skImage->isTextureBacked()) {
-    // TODO(junov): crbug.com/585607 Eliminate this readback and use an
-    // ExternalTextureLayer
-    sk_sp<SkSurface> surface =
-        SkSurface::MakeRasterN32Premul(skImage->width(), skImage->height());
-    if (!surface) {
-      // silent failure
-      m_image.clear();
-      return;
-    }
-    surface->getCanvas()->drawImage(skImage, 0, 0);
-    m_image = StaticBitmapImage::create(surface->makeImageSnapshot());
-  }
-  didDraw(skImage->bounds());
-  imageBitmap->close();
-}
+  didDraw();
 
-bool ImageBitmapRenderingContext::paint(GraphicsContext& gc, const IntRect& r) {
-  if (!m_image)
-    return true;
-
-  // With impl-side painting, it is unsafe to use a gpu-backed SkImage
-  DCHECK(
-      !m_image->imageForCurrentFrame(ColorBehavior::transformToGlobalTarget())
-           ->isTextureBacked());
-  gc.drawImage(m_image.get(), r, nullptr, creationAttributes().alpha()
-                                              ? SkBlendMode::kSrcOver
-                                              : SkBlendMode::kSrc);
-
-  return true;
+  if (imageBitmap)
+    imageBitmap->close();
 }
 
 CanvasRenderingContext* ImageBitmapRenderingContext::Factory::create(
     HTMLCanvasElement* canvas,
     const CanvasContextCreationAttributes& attrs,
     Document& document) {
   if (!RuntimeEnabledFeatures::experimentalCanvasFeaturesEnabled())
     return nullptr;
   return new ImageBitmapRenderingContext(canvas, attrs, document);
 }
 
 void ImageBitmapRenderingContext::stop() {
-  m_image.clear();
+  m_imageLayerBridge->dispose();
+}
+
+PassRefPtr<Image> ImageBitmapRenderingContext::getImage(AccelerationHint,
+                                                        SnapshotReason) const {
+  return m_imageLayerBridge->image();
+}
+
+WebLayer* ImageBitmapRenderingContext::platformLayer() const {
+  return m_imageLayerBridge->platformLayer();
+}
+
+bool ImageBitmapRenderingContext::isPaintable() const {
+  return !!m_imageLayerBridge->image();
+}
+
+DEFINE_TRACE(ImageBitmapRenderingContext) {
+  visitor->trace(m_imageLayerBridge);
+  CanvasRenderingContext::trace(visitor);
+}
+
+bool ImageBitmapRenderingContext::isAccelerated() const {
+  return m_imageLayerBridge->isAccelerated();
 }
 
 }  // blink