Add debugging output for visual rects in PaintController.

third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/graphics/paint/PaintController.h"
 
 #include "platform/TraceEvent.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/graphics/paint/DrawingDisplayItem.h"
 #include "third_party/skia/include/core/SkPictureAnalyzer.h"
@@ skipping 488 matching lines @@
     for (auto it = list.begin(); it != list.end(); ++it, ++i) {
         const DisplayItem& displayItem = *it;
         if (i)
             stringBuilder.append(",\n");
         stringBuilder.append(String::format("{index: %d, ", (int)i));
         displayItem.dumpPropertiesAsDebugString(stringBuilder);
         if (displayItem.hasValidClient()) {
             stringBuilder.append(", cacheIsValid: ");
             stringBuilder.append(clientCacheIsValid(displayItem.client()) ? "true" : "false");
         }
+        IntRect visualRect = list.visualRect(i);
+        stringBuilder.append(WTF::String::format(", visualRect: [%d,%d,%d,%d]",

[wkorman, 2016/06/30 22:25:55]

Can we tweak to match the new toString() format so: [%d,%d %dx%d]

Also I've received guidance from others in code reviews to just use 'String' and
drop 'WTF::', and if needed add wtf/Forward.h to allow. Both here and for return
value type.

[chrishtr, 2016/06/30 22:29:10]

Done.

+            visualRect.x(), visualRect.y(),
+            visualRect.width(), visualRect.height()));
         stringBuilder.append('}');
     }
     return stringBuilder.toString();
 }
 
 void PaintController::showDebugData() const
 {
     WTFLogAlways("current display item list: [%s]\n", displayItemListAsDebugString(m_currentPaintArtifact.getDisplayItemList()).utf8().data());
     WTFLogAlways("new display item list: [%s]\n", displayItemListAsDebugString(m_newDisplayItemList).utf8().data());
 }
 
 #endif // ifndef NDEBUG
 
 } // namespace blink