Clean up cc/paint interfaces

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

Index: third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp b/third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp
index 0df90d0a9753d32a449ab89e0a001078987f05ec..a3eab6f124eacef5475febdd25b0f29878ae959c 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp
+++ b/third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp
@@ -767,12 +767,15 @@ void PaintController::showUnderInvalidationError(
 
 #ifndef NDEBUG
   const PaintRecord* newRecord =
-      newItem.isDrawing()
-          ? static_cast<const DrawingDisplayItem&>(newItem).GetPaintRecord()
-          : nullptr;
+      newItem.isDrawing() ? static_cast<const DrawingDisplayItem&>(newItem)

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

maybe a ternary isnt the ring thing here and just

newRecord = nullptr;
if (isDrawing)
  newRecord = thing;

cuz this is a long ternary now

+                                .GetPaintRecord()
+                                .get()
+                          : nullptr;
   const PaintRecord* oldRecord =

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

same

       oldItem && oldItem->isDrawing()
-          ? static_cast<const DrawingDisplayItem*>(oldItem)->GetPaintRecord()
+          ? static_cast<const DrawingDisplayItem*>(oldItem)
+                ->GetPaintRecord()
+                .get()
           : nullptr;
   LOG(INFO) << "new record:\n"
             << (newRecord ? recordAsDebugString(newRecord) : "None");