Cleanup RenderTreeAsText

Source/core/rendering/RenderTreeAsText.cpp

 /*
  * Copyright (C) 2004, 2006, 2007 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 140 matching lines @@
                 result.append('{');
                 appendUnsignedAsHex(c, result);
                 result.append('}');
             }
         }
     }
     result.append('"');
     return result.toString();
 }
 
+TextStream& operator<<(TextStream& ts, const Color& c)
+{
+    return ts << c.nameForRenderTreeAsText();
+}
+
 void RenderTreeAsText::writeRenderObject(TextStream& ts, const RenderObject& o, RenderAsTextBehavior behavior)
 {
     ts << o.renderName();
 
     if (behavior & RenderAsTextShowAddresses)
         ts << " " << static_cast<const void*>(&o);
 
     if (o.style() && o.style()->zIndex())
         ts << " zI: " << o.style()->zIndex();
 
@@ skipping 43 matching lines @@
 
     ts << " " << r;
 
     if (!(o.isText() && !o.isBR())) {
         if (o.isFileUploadControl())
             ts << " " << quoteAndEscapeNonPrintables(toRenderFileUploadControl(&o)->fileTextValue());
 
         if (o.parent()) {
             Color color = o.resolveColor(CSSPropertyColor);
             if (o.parent()->resolveColor(CSSPropertyColor) != color)
-                ts << " [color=" << color.nameForRenderTreeAsText() << "]";
+                ts << " [color=" << color << "]";
 
             // Do not dump invalid or transparent backgrounds, since that is the default.
             Color backgroundColor = o.resolveColor(CSSPropertyBackgroundColor);
             if (o.parent()->resolveColor(CSSPropertyBackgroundColor) != backgroundColor
                 && backgroundColor.rgb())
-                ts << " [bgcolor=" << backgroundColor.nameForRenderTreeAsText() << "]";
+                ts << " [bgcolor=" << backgroundColor << "]";
 
             Color textFillColor = o.resolveColor(CSSPropertyWebkitTextFillColor);
             if (o.parent()->resolveColor(CSSPropertyWebkitTextFillColor) != textFillColor
                 && textFillColor != color && textFillColor.rgb())
-                ts << " [textFillColor=" << textFillColor.nameForRenderTreeAsText() << "]";
+                ts << " [textFillColor=" << textFillColor << "]";
 
             Color textStrokeColor = o.resolveColor(CSSPropertyWebkitTextStrokeColor);
             if (o.parent()->resolveColor(CSSPropertyWebkitTextStrokeColor) != textStrokeColor
                 && textStrokeColor != color && textStrokeColor.rgb())
-                ts << " [textStrokeColor=" << textStrokeColor.nameForRenderTreeAsText() << "]";
+                ts << " [textStrokeColor=" << textStrokeColor << "]";
 
             if (o.parent()->style()->textStrokeWidth() != o.style()->textStrokeWidth() && o.style()->textStrokeWidth() > 0)
                 ts << " [textStrokeWidth=" << o.style()->textStrokeWidth() << "]";
         }
 
         if (!o.isBoxModelObject())
             return;
 
         const RenderBoxModelObject& box = toRenderBoxModelObject(o);
         if (box.borderTop() || box.borderRight() || box.borderBottom() || box.borderLeft()) {
             ts << " [border:";
 
             BorderValue prevBorder = o.style()->borderTop();
             if (!box.borderTop())
                 ts << " none";
             else {
                 ts << " (" << box.borderTop() << "px ";
                 printBorderStyle(ts, o.style()->borderTopStyle());
-                Color col = o.resolveColor(CSSPropertyBorderTopColor);
-                ts << col.nameForRenderTreeAsText() << ")";
+                ts << o.resolveColor(CSSPropertyBorderTopColor) << ")";
             }
 
             if (o.style()->borderRight() != prevBorder) {
                 prevBorder = o.style()->borderRight();
                 if (!box.borderRight())
                     ts << " none";
                 else {
                     ts << " (" << box.borderRight() << "px ";
                     printBorderStyle(ts, o.style()->borderRightStyle());
-                    Color col = o.resolveColor(CSSPropertyBorderRightColor);
-                    ts << col.nameForRenderTreeAsText() << ")";
+                    ts << o.resolveColor(CSSPropertyBorderRightColor) << ")";
                 }
             }
 
             if (o.style()->borderBottom() != prevBorder) {
                 prevBorder = box.style()->borderBottom();
                 if (!box.borderBottom())
                     ts << " none";
                 else {
                     ts << " (" << box.borderBottom() << "px ";
                     printBorderStyle(ts, o.style()->borderBottomStyle());
-                    Color col = o.resolveColor(CSSPropertyBorderBottomColor);
-                    ts << col.nameForRenderTreeAsText() << ")";
+                    ts << o.resolveColor(CSSPropertyBorderBottomColor) << ")";
                 }
             }
 
             if (o.style()->borderLeft() != prevBorder) {
                 prevBorder = o.style()->borderLeft();
                 if (!box.borderLeft())
                     ts << " none";
                 else {
                     ts << " (" << box.borderLeft() << "px ";
                     printBorderStyle(ts, o.style()->borderLeftStyle());
-                    Color col = o.resolveColor(CSSPropertyBorderLeftColor);
-                    ts << col.nameForRenderTreeAsText() << ")";
+                    ts << o.resolveColor(CSSPropertyBorderLeftColor) << ")";
                 }
             }
 
             ts << "]";
         }
     }
 
     if (o.isTableCell()) {
         const RenderTableCell& c = toRenderTableCell(o);
         ts << " [r=" << c.rowIndex() << " c=" << c.col() << " rs=" << c.rowSpan() << " cs=" << c.colSpan() << "]";
@@ skipping 466 matching lines @@
     element->document().updateLayout();
 
     RenderObject* renderer = element->renderer();
     if (!renderer || !renderer->isListItem())
         return String();
 
     return toRenderListItem(renderer)->markerText();
 }
 
 } // namespace blink