Resolving drawImage issue with canvases that contain bitmap images

third_party/WebKit/Source/core/html/HTMLCanvasElement.cpp

 /*
  * Copyright (C) 2004, 2006, 2007 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  * Copyright (C) 2010 Torch Mobile (Beijing) Co. Ltd. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 1083 matching lines @@
 
     if (!m_context) {
         *status = NormalSourceImageStatus;
         return createTransparentImage(size());
     }
 
     if (m_context->is3d()) {
         m_context->paintRenderingResultsToCanvas(BackBuffer);
     }
 
-    RefPtr<SkImage> image = buffer()->newSkImageSnapshot(hint, reason);
+    RefPtr<SkImage> image;
+    if (renderingContext()->getContextType() == CanvasRenderingContext::ContextImageBitmap)

[Justin Novosad, 2016/08/22 19:42:25]

Avoid explicit type checks when possible. In this case, you made getImage
virtual, so the type check is not necessary.
Should be:
RefPtr<SkImage> skImage;
RefPtr<Blink::Image> image = renderingContext()->getImage();
if (image)
   skImage = image->imageForCurrentFrame();
else
...

[zakerinasab, 2016/08/22 19:55:50]

Done.

+        image = renderingContext()->getImage()->imageForCurrentFrame();
+    else
+        image = buffer()->newSkImageSnapshot(hint, reason);
+
     if (image) {
         *status = NormalSourceImageStatus;
         return StaticBitmapImage::create(image.release());
     }
 
     *status = InvalidSourceImageStatus;
     return nullptr;
 }
 
 bool HTMLCanvasElement::wouldTaintOrigin(SecurityOrigin*) const
@@ skipping 101 matching lines @@
 
 bool HTMLCanvasElement::createSurfaceLayer()
 {
     DCHECK(!m_surfaceLayerBridge);
     std::unique_ptr<CanvasSurfaceLayerBridgeClient> bridgeClient = wrapUnique(new CanvasSurfaceLayerBridgeClientImpl());
     m_surfaceLayerBridge = wrapUnique(new CanvasSurfaceLayerBridge(std::move(bridgeClient)));
     return m_surfaceLayerBridge->createSurfaceLayer(this->width(), this->height());
 }
 
 } // namespace blink