Add debugging output for visual rects in PaintController.

third_party/WebKit/Source/platform/graphics/paint/DrawingDisplayItem.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/DrawingDisplayItem.h"
 
 #include "platform/graphics/GraphicsContext.h"
 #include "public/platform/WebDisplayItemList.h"
 #include "third_party/skia/include/core/SkPictureAnalyzer.h"
 
@@ skipping 26 matching lines @@
 void DrawingDisplayItem::analyzeForGpuRasterization(SkPictureGpuAnalyzer& analyzer) const
 {
     analyzer.analyzePicture(m_picture.get());
 }
 
 #ifndef NDEBUG
 void DrawingDisplayItem::dumpPropertiesAsDebugString(WTF::StringBuilder& stringBuilder) const
 {
     DisplayItem::dumpPropertiesAsDebugString(stringBuilder);
     if (m_picture) {
-        stringBuilder.append(WTF::String::format(", rect: [%f,%f,%f,%f]",
+        stringBuilder.append(WTF::String::format(", rect: [%f,%f %fx%f]",

[wkorman, 2016/07/01 19:04:06]

Can remove WTF:: here in two places, also here and in PaintController, can
alternatively do:

rect: [%s]", m_picture->ToString().ascii().data())

Is that better? I'm not sure. I've grown to deeply loathe ascii().data().

             m_picture->cullRect().x(), m_picture->cullRect().y(),
             m_picture->cullRect().width(), m_picture->cullRect().height()));
     }
 }
 #endif
 
 #if ENABLE(ASSERT)
 static bool bitmapIsAllZero(const SkBitmap& bitmap)
 {
     bitmap.lockPixels();
@@ skipping 55 matching lines @@
         return bitmapIsAllZero(bitmap);
     }
     default:
         ASSERT_NOT_REACHED();
     }
     return false;
 }
 #endif // ENABLE(ASSERT)
 
 } // namespace blink