[wip] unprefix filter

third_party/WebKit/Source/core/layout/svg/SVGResources.cpp

Index: third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp b/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
index fe15b73088be99170e7c2d132b9826f235e06247..9a4ec11d64c7c57e713db7bca96cd940d397600f 100644
--- a/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
@@ -25,7 +25,7 @@
 #include "core/layout/svg/LayoutSVGResourceMarker.h"
 #include "core/layout/svg/LayoutSVGResourceMasker.h"
 #include "core/layout/svg/LayoutSVGResourcePaintServer.h"
-#include "core/style/SVGComputedStyle.h"
+#include "core/style/ComputedStyle.h"
 #include "core/svg/SVGFilterElement.h"
 #include "core/svg/SVGGradientElement.h"
 #include "core/svg/SVGPatternElement.h"
@@ -203,7 +203,7 @@ static inline SVGResources& ensureResources(OwnPtr<SVGResources>& resources)
     return *resources.get();
 }
 
-PassOwnPtr<SVGResources> SVGResources::buildResources(const LayoutObject* object, const SVGComputedStyle& style)
+PassOwnPtr<SVGResources> SVGResources::buildResources(const LayoutObject* object, const ComputedStyle& computedStyle)
 {
     ASSERT(object);
 
@@ -220,6 +220,8 @@ PassOwnPtr<SVGResources> SVGResources::buildResources(const LayoutObject* object
     TreeScope& treeScope = element->treeScope();
     SVGDocumentExtensions& extensions = element->document().accessSVGExtensions();
 
+    const SVGComputedStyle& style = computedStyle.svgStyle();
+
     OwnPtr<SVGResources> resources;
     if (clipperFilterMaskerTags().contains(tagName)) {
         if (style.hasClipper()) {
@@ -228,10 +230,17 @@ PassOwnPtr<SVGResources> SVGResources::buildResources(const LayoutObject* object
                 registerPendingResource(extensions, id, element);
         }
 
-        if (style.hasFilter()) {
-            AtomicString id = style.filterResource();
-            if (!ensureResources(resources).setFilter(getLayoutSVGResourceById<LayoutSVGResourceFilter>(treeScope, id)))
-                registerPendingResource(extensions, id, element);
+        if (computedStyle.hasFilter()) {

[fs, 2016/05/23 12:29:53]

I think LayoutSVGRoot should be left out of this - i.e not store a filter in
SVGResources. If it does it'll be sitting on two chairs - and inconsistently so
because this path is limited to only "url(#foo)". 

[Noel Gordon, 2016/05/29 04:15:28]

I stored the filter in SVGResources, and prevented FilterPainter from trying to
create / apply the filter.

+            const FilterOperations& filterOperations = computedStyle.filter();
+            if (filterOperations.size() == 1) {
+                const FilterOperation& filterOperation = *filterOperations.at(0);
+                if (filterOperation.type() == FilterOperation::REFERENCE) {
+                    const auto& referenceFilterOperation = toReferenceFilterOperation(filterOperation);
+                    AtomicString id = SVGURIReference::fragmentIdentifierFromIRIString(referenceFilterOperation.url(), element->document());

[fs, 2016/05/23 12:29:53]

uNit: could use |treeScope| here (instead of element->document()) so that we're
consistently using one or the other (should matter here ATM I think.)

That being said though, ReferenceFilterOperation::url() is already resolved
against the base URL here, and the fragment is extracted (to
ReferenceFilterOperation::fragment()), so it seems the only thing needed here is
to use those (check if the URL is local and use the stored fragment if it is.)

[Noel Gordon, 2016/05/29 04:15:28]

Yes, done.
Didn't quite follow what you are asking for here.  The filter url(reference)
could be to another document, eg., url(other.svg#fragment) best I can tell.

[fs, 2016/05/30 11:58:52]

Yes, you still need to check that the reference is local ("check if the URL is
local"), so something like:

KURL url(ParsedURLString, referenceFilterOperation.url());
if (equalIgnoringFragmentIdentifier(url, document->url())
    id = AtomicString(referenceFilterOperation.fragment());

is what I had in mind. NVM though, I can fix that up later.

+                    if (!ensureResources(resources).setFilter(getLayoutSVGResourceById<LayoutSVGResourceFilter>(treeScope, id)))
+                        registerPendingResource(extensions, id, element);
+                }
+            }
         }
 
         if (style.hasMasker()) {