Changeset 215963 in webkit

Timestamp:

Apr 28, 2017 5:48:08 PM (7 years ago)

Author:

Simon Fraser

Message:

Modernize RenderTreeAsText code
​https://bugs.webkit.org/show_bug.cgi?id=171458

Reviewed by Zalan Bujtas.

Use modern loops. More references to render objects and layers.

• rendering/RenderTreeAsText.cpp:

(WebCore::write):
(WebCore::writeRenderNamedFlowThreads):
(WebCore::writeLayers):
(WebCore::writeSelection):
(WebCore::externalRepresentation):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderTreeAsText.cpp (modified) (11 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-04-28  Simon Fraser  <simon.fraser@apple.com>
+
+        Modernize RenderTreeAsText code
+        https://bugs.webkit.org/show_bug.cgi?id=171458
+
+        Reviewed by Zalan Bujtas.
+
+        Use modern loops. More references to render objects and layers.
+
+        * rendering/RenderTreeAsText.cpp:
+        (WebCore::write):
+        (WebCore::writeRenderNamedFlowThreads):
+        (WebCore::writeLayers):
+        (WebCore::writeSelection):
+        (WebCore::externalRepresentation):
+
 2017-04-28  Wenson Hsieh  <wenson_hsieh@apple.com>
 

trunk/Source/WebCore/rendering/RenderTreeAsText.cpp

@@ -82 +82 @@
 using namespace HTMLNames;
 
-static void writeLayers(TextStream&, const RenderLayer* rootLayer, RenderLayer*, const LayoutRect& paintDirtyRect, int indent = 0, RenderAsTextBehavior = RenderAsTextBehaviorNormal);
+static void writeLayers(TextStream&, const RenderLayer& rootLayer, RenderLayer&, const LayoutRect& paintDirtyRect, int indent = 0, RenderAsTextBehavior = RenderAsTextBehaviorNormal);
 
 static void printBorderStyle(TextStream& ts, const EBorderStyle borderStyle)
@@ -599 +599 @@
                     view.layout();
                 if (RenderLayer* layer = root->layer())
-                    writeLayers(ts, layer, layer, layer->rect(), indent + 1, behavior);
+                    writeLayers(ts, *layer, *layer, layer->rect(), indent + 1, behavior);
             }
         }
@@ -720 +720 @@
 }
 
-static void writeRenderNamedFlowThreads(TextStream& ts, RenderView& renderView, const RenderLayer* rootLayer,
-    const LayoutRect& paintRect, int indent, RenderAsTextBehavior behavior)
+static void writeRenderNamedFlowThreads(TextStream& ts, RenderView& renderView, const RenderLayer& rootLayer, const LayoutRect& paintRect, int indent, RenderAsTextBehavior behavior)
 {
     if (!renderView.hasRenderNamedFlowThreads())
         return;
-
-    const RenderNamedFlowThreadList* list = renderView.flowThreadController().renderNamedFlowThreadList();
 
     writeIndent(ts, indent);
     ts << "Named flows\n";
 
-    for (RenderNamedFlowThreadList::const_iterator iter = list->begin(); iter != list->end(); ++iter) {
-        const RenderNamedFlowThread* renderFlowThread = *iter;
-
+    for (const auto* renderFlowThread : *renderView.flowThreadController().renderNamedFlowThreadList()) {
         writeIndent(ts, indent + 1);
         ts << "Named flow '" << renderFlowThread->flowThreadName() << "'\n";
 
         RenderLayer* layer = renderFlowThread->layer();
-        writeLayers(ts, rootLayer, layer, paintRect, indent + 2, behavior);
+        writeLayers(ts, rootLayer, *layer, paintRect, indent + 2, behavior);
 
         // Display the valid and invalid render regions attached to this flow thread.
@@ -762 +757 @@
 }
 
-static void writeLayers(TextStream& ts, const RenderLayer* rootLayer, RenderLayer* l,
-                        const LayoutRect& paintRect, int indent, RenderAsTextBehavior behavior)
-{
-    // FIXME: Apply overflow to the root layer to not break every test.  Complete hack.  Sigh.
+static void writeLayers(TextStream& ts, const RenderLayer& rootLayer, RenderLayer& layer, const LayoutRect& paintRect, int indent, RenderAsTextBehavior behavior)
+{
+    // FIXME: Apply overflow to the root layer to not break every test. Complete hack. Sigh.
     LayoutRect paintDirtyRect(paintRect);
-    if (rootLayer == l) {
-        paintDirtyRect.setWidth(std::max<LayoutUnit>(paintDirtyRect.width(), rootLayer->renderBox()->layoutOverflowRect().maxX()));
-        paintDirtyRect.setHeight(std::max<LayoutUnit>(paintDirtyRect.height(), rootLayer->renderBox()->layoutOverflowRect().maxY()));
-        l->setSize(l->size().expandedTo(snappedIntSize(maxLayoutOverflow(l->renderBox()), LayoutPoint(0, 0))));
+    if (&rootLayer == &layer) {
+        paintDirtyRect.setWidth(std::max<LayoutUnit>(paintDirtyRect.width(), rootLayer.renderBox()->layoutOverflowRect().maxX()));
+        paintDirtyRect.setHeight(std::max<LayoutUnit>(paintDirtyRect.height(), rootLayer.renderBox()->layoutOverflowRect().maxY()));
+        layer.setSize(layer.size().expandedTo(snappedIntSize(maxLayoutOverflow(layer.renderBox()), LayoutPoint(0, 0))));
     }
     
@@ -777 +771 @@
     ClipRect damageRect;
     ClipRect clipRectToApply;
-    l->calculateRects(RenderLayer::ClipRectsContext(rootLayer, TemporaryClipRects), paintDirtyRect, layerBounds, damageRect, clipRectToApply, l->offsetFromAncestor(rootLayer));
+    layer.calculateRects(RenderLayer::ClipRectsContext(&rootLayer, TemporaryClipRects), paintDirtyRect, layerBounds, damageRect, clipRectToApply, layer.offsetFromAncestor(&rootLayer));
 
     // Ensure our lists are up-to-date.
-    l->updateLayerListsIfNeeded();
-
-    bool shouldPaint = (behavior & RenderAsTextShowAllLayers) ? true : l->intersectsDamageRect(layerBounds, damageRect.rect(), rootLayer, l->offsetFromAncestor(rootLayer));
-    Vector<RenderLayer*>* negList = l->negZOrderList();
-    bool paintsBackgroundSeparately = negList && negList->size() > 0;
+    layer.updateLayerListsIfNeeded();
+
+    bool shouldPaint = (behavior & RenderAsTextShowAllLayers) ? true : layer.intersectsDamageRect(layerBounds, damageRect.rect(), &rootLayer, layer.offsetFromAncestor(&rootLayer));
+    Vector<RenderLayer*>* negativeZOrderList = layer.negZOrderList();
+    bool paintsBackgroundSeparately = negativeZOrderList && negativeZOrderList->size() > 0;
     if (shouldPaint && paintsBackgroundSeparately)
-        write(ts, *l, layerBounds, damageRect.rect(), clipRectToApply.rect(), LayerPaintPhaseBackground, indent, behavior);
-
-    if (negList) {
+        write(ts, layer, layerBounds, damageRect.rect(), clipRectToApply.rect(), LayerPaintPhaseBackground, indent, behavior);
+
+    if (negativeZOrderList) {
         int currIndent = indent;
         if (behavior & RenderAsTextShowLayerNesting) {
             writeIndent(ts, indent);
-            ts << " negative z-order list(" << negList->size() << ")\n";
+            ts << " negative z-order list(" << negativeZOrderList->size() << ")\n";
             ++currIndent;
         }
-        for (unsigned i = 0; i != negList->size(); ++i)
-            writeLayers(ts, rootLayer, negList->at(i), paintDirtyRect, currIndent, behavior);
+        
+        for (auto* currLayer : *negativeZOrderList)
+            writeLayers(ts, rootLayer, *currLayer, paintDirtyRect, currIndent, behavior);
     }
 
     if (shouldPaint)
-        write(ts, *l, layerBounds, damageRect.rect(), clipRectToApply.rect(), paintsBackgroundSeparately ? LayerPaintPhaseForeground : LayerPaintPhaseAll, indent, behavior);
-
-    if (Vector<RenderLayer*>* normalFlowList = l->normalFlowList()) {
+        write(ts, layer, layerBounds, damageRect.rect(), clipRectToApply.rect(), paintsBackgroundSeparately ? LayerPaintPhaseForeground : LayerPaintPhaseAll, indent, behavior);
+
+    if (Vector<RenderLayer*>* normalFlowList = layer.normalFlowList()) {
         int currIndent = indent;
         if (behavior & RenderAsTextShowLayerNesting) {
@@ -809 +804 @@
             ++currIndent;
         }
-        for (unsigned i = 0; i != normalFlowList->size(); ++i)
-            writeLayers(ts, rootLayer, normalFlowList->at(i), paintDirtyRect, currIndent, behavior);
-    }
-
-    if (Vector<RenderLayer*>* posList = l->posZOrderList()) {
+        
+        for (auto* currLayer : *normalFlowList)
+            writeLayers(ts, rootLayer, *currLayer, paintDirtyRect, currIndent, behavior);
+    }
+
+    if (Vector<RenderLayer*>* positiveZOrderList = layer.posZOrderList()) {
         size_t layerCount = 0;
-        for (unsigned i = 0; i != posList->size(); ++i)
-            if (!posList->at(i)->isFlowThreadCollectingGraphicsLayersUnderRegions())
+        for (auto* currLayer : *positiveZOrderList) {
+            if (!currLayer->isFlowThreadCollectingGraphicsLayersUnderRegions())
                 ++layerCount;
+        }
+        
         if (layerCount) {
             int currIndent = indent;
             // We only print the header if there's at list a non-RenderNamedFlowThread part of the list.
-            if (!posList->size() || !posList->at(0)->isFlowThreadCollectingGraphicsLayersUnderRegions()) {
+            if (!positiveZOrderList->size() || !positiveZOrderList->at(0)->isFlowThreadCollectingGraphicsLayersUnderRegions()) {
                 if (behavior & RenderAsTextShowLayerNesting) {
                     writeIndent(ts, indent);
-                    ts << " positive z-order list(" << posList->size() << ")\n";
+                    ts << " positive z-order list(" << positiveZOrderList->size() << ")\n";
                     ++currIndent;
                 }
-                for (unsigned i = 0; i != posList->size(); ++i) {
+                
+                for (auto* currLayer : *positiveZOrderList) {
                     // Do not print named flows twice.
-                    if (!posList->at(i)->isFlowThreadCollectingGraphicsLayersUnderRegions())
-                        writeLayers(ts, rootLayer, posList->at(i), paintDirtyRect, currIndent, behavior);
+                    if (!currLayer->isFlowThreadCollectingGraphicsLayersUnderRegions())
+                        writeLayers(ts, rootLayer, *currLayer, paintDirtyRect, currIndent, behavior);
                 }
             }
@@ -838 +837 @@
     // Altough the RenderFlowThread requires a layer, it is not collected by its parent,
     // so we have to treat it as a special case.
-    if (is<RenderView>(l->renderer()))
-        writeRenderNamedFlowThreads(ts, downcast<RenderView>(l->renderer()), rootLayer, paintDirtyRect, indent, behavior);
+    if (is<RenderView>(layer.renderer()))
+        writeRenderNamedFlowThreads(ts, downcast<RenderView>(layer.renderer()), rootLayer, paintDirtyRect, indent, behavior);
 }
 
@@ -876 +875 @@
 }
 
-static void writeSelection(TextStream& ts, const RenderObject* renderer)
-{
-    if (!renderer->isRenderView())
-        return;
-
-    Frame* frame = renderer->document().frame();
+static void writeSelection(TextStream& ts, const RenderBox& renderer)
+{
+    if (!renderer.isRenderView())
+        return;
+
+    Frame* frame = renderer.document().frame();
     if (!frame)
         return;
@@ -896 +895 @@
 }
 
-static String externalRepresentation(RenderBox* renderer, RenderAsTextBehavior behavior)
+static String externalRepresentation(RenderBox& renderer, RenderAsTextBehavior behavior)
 {
     TextStream ts(TextStream::LineMode::MultipleLine, TextStream::Formatting::SVGStyleRect | TextStream::Formatting::LayoutUnitsAsIntegers);
-    if (!renderer->hasLayer())
+    if (!renderer.hasLayer())
         return ts.release();
 
     LOG(Layout, "externalRepresentation: dumping layer tree");
 
-    RenderLayer* layer = renderer->layer();
-    writeLayers(ts, layer, layer, layer->rect(), 0, behavior);
+    RenderLayer& layer = *renderer.layer();
+    writeLayers(ts, layer, layer, layer.rect(), 0, behavior);
     writeSelection(ts, renderer);
     return ts.release();
@@ -922 +921 @@
         frame->document()->updateLayout();
 
-    return externalRepresentation(renderer, behavior);
+    return externalRepresentation(*renderer, behavior);
 }
 
@@ -935 +934 @@
         element->document().updateLayout();
     
-    return externalRepresentation(downcast<RenderBox>(renderer), behavior | RenderAsTextShowAllLayers);
+    return externalRepresentation(downcast<RenderBox>(*renderer), behavior | RenderAsTextShowAllLayers);
 }