Support fragment URLs for more kinds of SVG images

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@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.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 33 matching lines @@
 
     SVGAnimatedLength* x() const { return m_x.get(); }
     SVGAnimatedLength* y() const { return m_y.get(); }
     SVGAnimatedLength* width() const { return m_width.get(); }
     SVGAnimatedLength* height() const { return m_height.get(); }
     SVGAnimatedPreserveAspectRatio* preserveAspectRatio() { return m_preserveAspectRatio.get(); }
 
     // Exposed for testing.
     ImageResource* cachedImage() const { return imageLoader().image(); }
 
+    const AtomicString imageSourceURL() const override;

[fs, 2015/11/19 14:59:44]

Nit: What needs this to be public? (Isn't it accessed through Element only?)

[davve, 2015/11/20 08:14:21]

You're very much correct. Left-over from patch set #1.

+
 private:
     explicit SVGImageElement(Document&);
 
     bool isStructurallyExternal() const override { return !hrefString().isNull(); }
 
     bool isPresentationAttribute(const QualifiedName&) const override;
     bool isPresentationAttributeWithSVGDOM(const QualifiedName&) const override;
     void collectStyleForPresentationAttribute(const QualifiedName&, const AtomicString&, MutableStylePropertySet*) override;
 
     void svgAttributeChanged(const QualifiedName&) override;
 
     void attach(const AttachContext& = AttachContext()) override;
     InsertionNotificationRequest insertedInto(ContainerNode*) override;
 
     LayoutObject* createLayoutObject(const ComputedStyle&) override;
 
-    const AtomicString imageSourceURL() const override;
-
     bool haveLoadedRequiredResources() override;
 
     bool selfHasRelativeLengths() const override;
     void didMoveToNewDocument(Document& oldDocument) override;
     SVGImageLoader& imageLoader() const { return *m_imageLoader; }
 
     RefPtrWillBeMember<SVGAnimatedLength> m_x;
     RefPtrWillBeMember<SVGAnimatedLength> m_y;
     RefPtrWillBeMember<SVGAnimatedLength> m_width;
     RefPtrWillBeMember<SVGAnimatedLength> m_height;
     RefPtrWillBeMember<SVGAnimatedPreserveAspectRatio> m_preserveAspectRatio;
 
     OwnPtrWillBeMember<SVGImageLoader> m_imageLoader;
     bool m_needsLoaderURIUpdate : 1;
 };
 
 } // namespace blink
 
 #endif // SVGImageElement_h