Adds SVGImageElement as a CanvasImageSource

third_party/WebKit/Source/core/svg/SVGImageElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Rob Buis <buis@kde.org>
  * Copyright (C) 2006 Alexander Kellett <lypanov@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 186 matching lines @@
 
 const AtomicString SVGImageElement::imageSourceURL() const {
   return AtomicString(hrefString());
 }
 
 void SVGImageElement::didMoveToNewDocument(Document& oldDocument) {
   imageLoader().elementDidMoveToNewDocument();
   SVGGraphicsElement::didMoveToNewDocument(oldDocument);
 }
 
+PassRefPtr<Image> SVGImageElement::getSourceImageForCanvas(
+    SourceImageStatus* status,
+    AccelerationHint hint,
+    SnapshotReason reason,
+    const FloatSize& defaultObjectSize) const {
+  if (!cachedImage()) {

[Justin Novosad, 2017/03/01 20:56:29]

What happened to that extra check I saw you put in your local build?

[fserb, 2017/03/06 21:05:13]

it's there. :)

+    *status = IncompleteSourceImageStatus;
+    return nullptr;
+  }
+
+  if (cachedImage()->errorOccurred()) {
+    *status = UndecodableSourceImageStatus;
+    return nullptr;
+  }
+
+  *status = NormalSourceImageStatus;
+  return cachedImage()->getImage()->imageForDefaultFrame();
+}
+
+bool SVGImageElement::wouldTaintOrigin(
+    SecurityOrigin* destinationSecurityOrigin) const {
+  if (!cachedImage())
+    return false;
+  return !cachedImage()->isAccessAllowed(destinationSecurityOrigin);

[Justin Novosad, 2017/03/01 20:56:30]

This needs to be tested in a layout test.  Example:
LayoutTests/http/security/canvas-remote-read-remote-image.html

You basically need a version of that test that uses an SVGImageElement.  Also,
new tests should use testharness.js.

[fserb, 2017/03/06 21:05:13]

done.

+}
+
+FloatSize SVGImageElement::elementSize(
+    const FloatSize& defaultObjectSize) const {
+  if (!cachedImage())
+    return FloatSize();
+  return FloatSize(cachedImage()->getImage()->size());
+}
+
+bool SVGImageElement::isAccelerated() const {
+  return false;
+}
+
+int SVGImageElement::sourceWidth() {
+  if (!cachedImage())
+    return width();
+  return cachedImage()->getImage()->width();
+}
+
+int SVGImageElement::sourceHeight() {
+  if (!cachedImage())
+    return height();
+  return cachedImage()->getImage()->height();
+}
+
 }  // namespace blink