Skip PaintPhaseDescendantOutlinesOnly if no descendent outlines in the layer

third_party/WebKit/Source/core/paint/PaintLayer.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  *
  * Portions are Copyright (C) 1998 Netscape Communications Corporation.
  *
  * Other contributors:
  *   Robert O'Callahan <roc+@cs.cmu.edu>
  *   David Baron <dbaron@fas.harvard.edu>
  *   Christian Biesinger <cbiesinger@web.de>
  *   Randall Jesup <rjesup@wgate.com>
@@ skipping 109 matching lines @@
     , m_hasFilterInfo(false)
     , m_needsAncestorDependentCompositingInputsUpdate(true)
     , m_needsDescendantDependentCompositingInputsUpdate(true)
     , m_childNeedsCompositingInputsUpdate(true)
     , m_hasCompositingDescendant(false)
     , m_hasNonCompositedChild(false)
     , m_shouldIsolateCompositedDescendants(false)
     , m_lostGroupedMapping(false)
     , m_needsRepaint(false)
     , m_previousPaintResult(PaintLayerPainter::FullyPainted)
+    , m_needsPaintPhaseChildOutlines(false)
     , m_layoutObject(layoutObject)
     , m_parent(0)
     , m_previous(0)
     , m_next(0)
     , m_first(0)
     , m_last(0)
     , m_staticInlinePosition(0)
     , m_staticBlockPosition(0)
     , m_enclosingPaginationLayer(0)
     , m_potentialCompositingReasonsFromStyle(CompositingReasonNone)
@@ skipping 798 matching lines @@
         return const_cast<PaintLayer*>(this);
 
     for (PaintLayer* curr = compositingContainer(); curr; curr = curr->compositingContainer()) {
         if (curr->isPaintInvalidationContainer())
             return curr;
     }
 
     return nullptr;
 }
 
+PaintLayer* PaintLayer::enclosingSelfPaintingLayer() const
+{
+    for (PaintLayer* layer = const_cast<PaintLayer*>(this); layer; layer = layer->parent()) {

[chrishtr, 2016/01/12 23:52:24]

should be compositingContainer() instead of parent() right?

[Xianzhu, 2016/01/13 00:11:03]

No. We just want to find the self painting layer which may be between this layer
and compositingContainer(). This won't violet paint order because if this layer
is self painting, the function returns this; otherwise the self painting layer
should definitely be the ancestor of this layer.

[chrishtr, 2016/01/13 05:01:17]

I see, ok.

+        if (layer->isSelfPaintingLayer())
+            return layer;
+    }
+    return nullptr;
+}
+
 void PaintLayer::setNeedsCompositingInputsUpdate()
 {
     m_needsAncestorDependentCompositingInputsUpdate = true;
     m_needsDescendantDependentCompositingInputsUpdate = true;
 
     for (PaintLayer* current = this; current && !current->m_childNeedsCompositingInputsUpdate; current = current->parent())
         current->m_childNeedsCompositingInputsUpdate = true;
 
     compositor()->setNeedsCompositingUpdate(CompositingUpdateAfterCompositingInputChange);
 }
@@ skipping 1872 matching lines @@
 
 void showLayerTree(const blink::LayoutObject* layoutObject)
 {
     if (!layoutObject) {
         fprintf(stderr, "Cannot showLayerTree. Root is (nil)\n");
         return;
     }
     showLayerTree(layoutObject->enclosingLayer());
 }
 #endif