Clean up cc/paint interfaces

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

 // Copyright 2016 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/DisplayItemList.h"
 
 #include "platform/graphics/LoggingCanvas.h"
 #include "platform/graphics/paint/DrawingDisplayItem.h"
 #include "platform/graphics/paint/PaintChunk.h"
 #include "third_party/skia/include/core/SkPictureAnalyzer.h"
@@ skipping 74 matching lines @@
       if (options & ShowClientDebugName) {
 #endif
         json->setString(
             "clientDebugName",
             String::format("clientDebugName: \"%s\"",
                            displayItem.client().debugName().ascii().data()));
       }
 #ifndef NDEBUG
       if ((options & ShowPaintRecords) && displayItem.isDrawing()) {
         if (const PaintRecord* record =
                 static_cast<const DrawingDisplayItem&>(displayItem)

[danakj, 2017/03/16 20:06:16]

a temp var here could go a long way to less wild line wrapping, maybe for the
item cast?

-                    .GetPaintRecord()) {
+                    .GetPaintRecord()
+                    .get()) {
           json->setString("record", recordAsDebugString(record));
         }
       }
 #endif
     }
     if (hasVisualRect(i)) {
       IntRect localVisualRect = visualRect(i);
       json->setString(
           "visualRect",
           String::format("[%d,%d %dx%d]", localVisualRect.x(),
                          localVisualRect.y(), localVisualRect.width(),
                          localVisualRect.height()));
     }
     jsonArray->pushObject(std::move(json));
   }
   return jsonArray;
 }
 
 }  // namespace blink