[SPv2] Associate effect property nodes for SVG paint chunks

third_party/WebKit/Source/core/layout/LayoutObject.h

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  *           (C) 2000 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2012 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 630 matching lines @@
   // (see above). Unfortunately we don't have such a class yet, because it's not
   // possible for all layoutObjects to inherit from LayoutSVGObject ->
   // LayoutObject (some need LayoutBlock inheritance for instance)
   virtual void setNeedsTransformUpdate() {}
   virtual void setNeedsBoundariesUpdate();
 
   bool isBlendingAllowed() const {
     return !isSVG() || (isSVGContainer() && !isSVGHiddenContainer()) ||
            isSVGShape() || isSVGImage() || isSVGText();
   }
-  virtual bool hasNonIsolatedBlendingDescendants() const { return false; }
+  virtual bool hasNonIsolatedBlendingDescendants() const {

[trchen, 2016/12/14 01:36:55]

Why not just remove the thing and make it part of LayoutSVGModelObject interface
you might ask?

It is because LayoutSVGRoot is LayoutBoxModelObject, not LayoutSVGModelObject.
There is a FIXME on line 603.

IMO our layout object type system is not designed well to handle things crossing
flow type boundary. LayoutSVGRoot is an inline-flow to its parent (inline
replaced), but contains SVG-flow. Likewise, LayoutSVGBlock is a SVG-flow to its
parent, but contains block-flow. Ideally this really should be solved with
properly designed diamond inheritance...

+    // SVG only. For non-SVG objects, query PaintLayer instead.
+    NOTREACHED();
+    return false;
+  }
   enum DescendantIsolationState {
     DescendantIsolationRequired,
     DescendantIsolationNeedsUpdate,
   };
   virtual void descendantIsolationRequirementsChanged(
       DescendantIsolationState) {}
 
   // Per SVG 1.1 objectBoundingBox ignores clipping, masking, filter effects,
   // opacity and stroke-width.
   // This is used for all computation of objectBoundingBox relative units and by
@@ skipping 1995 matching lines @@
 CORE_EXPORT void showLineTree(const blink::LayoutObject*);
 CORE_EXPORT void showLayoutTree(const blink::LayoutObject* object1);
 // We don't make object2 an optional parameter so that showLayoutTree
 // can be called from gdb easily.
 CORE_EXPORT void showLayoutTree(const blink::LayoutObject* object1,
                                 const blink::LayoutObject* object2);
 
 #endif
 
 #endif  // LayoutObject_h