Set needs paint property update for changes of SVG isolation and blending.

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

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2007, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Google, Inc.  All rights reserved.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.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
@@ skipping 97 matching lines @@
                                         const ComputedStyle* oldStyle) {
   LayoutSVGModelObject::styleDidChange(diff, oldStyle);
 
   bool hadIsolation =
       oldStyle && !isSVGHiddenContainer() &&
       SVGLayoutSupport::willIsolateBlendingDescendantsForStyle(*oldStyle);
   bool isolationChanged =
       hadIsolation ==
       !SVGLayoutSupport::willIsolateBlendingDescendantsForObject(this);
 
+  if (isolationChanged)
+    setNeedsPaintPropertyUpdate();
+
   if (!parent() || !isolationChanged)
     return;
 
   if (hasNonIsolatedBlendingDescendants())
     parent()->descendantIsolationRequirementsChanged(
         SVGLayoutSupport::willIsolateBlendingDescendantsForObject(this)

[Xianzhu, 2016/12/28 23:14:37]

Can you by the way refactor to avoid duplicated call to
willIsolateBlendingDescendantsForObject?

[chrishtr, 2016/12/28 23:20:47]

Done.

             ? DescendantIsolationNeedsUpdate
             : DescendantIsolationRequired);
 }
 
 bool LayoutSVGContainer::hasNonIsolatedBlendingDescendants() const {
   if (m_hasNonIsolatedBlendingDescendantsDirty) {
     m_hasNonIsolatedBlendingDescendants =
         SVGLayoutSupport::computeHasNonIsolatedBlendingDescendants(this);
     m_hasNonIsolatedBlendingDescendantsDirty = false;
   }
@@ skipping 79 matching lines @@
   // under the pointer (i.e., there is no target element), the event is not
   // dispatched."
   return false;
 }
 
 SVGTransformChange LayoutSVGContainer::calculateLocalTransform() {
   return SVGTransformChange::None;
 }
 
 }  // namespace blink