Remove cullRect() from PaintOpBuffer.

third_party/WebKit/Source/core/paint/BoxReflectionUtils.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 "core/paint/BoxReflectionUtils.h"
 
 #include "core/layout/LayoutBox.h"
 #include "core/paint/NinePieceImagePainter.h"
 #include "core/paint/PaintLayer.h"
 #include "platform/LengthFunctions.h"
@@ skipping 33 matching lines @@
       offset =
           -FloatValueForLength(reflect_style->Offset(), frame_rect.Width());
       break;
     case kReflectionRight:
       direction = BoxReflection::kHorizontalReflection;
       offset = 2 * frame_rect.Width() +
                FloatValueForLength(reflect_style->Offset(), frame_rect.Width());
       break;
   }
 
-  sk_sp<PaintRecord> mask;
   const NinePieceImage& mask_nine_piece = reflect_style->Mask();
-  if (mask_nine_piece.HasImage()) {
-    LayoutRect mask_rect(LayoutPoint(), frame_layout_rect.Size());
-    LayoutRect mask_bounding_rect(mask_rect);
-    mask_bounding_rect.Expand(style.ImageOutsets(mask_nine_piece));
-    FloatRect mask_bounding_float_rect(mask_bounding_rect);
+  if (!mask_nine_piece.HasImage())
+    return BoxReflection(direction, offset, nullptr, FloatRect());
 
-    // TODO(jbroman): PaintRecordBuilder + DrawingRecorder seems excessive.
-    // If NinePieceImagePainter operated on SkCanvas, we'd only need a
-    // PictureRecorder here.
-    PaintRecordBuilder builder(mask_bounding_float_rect);
-    {
-      GraphicsContext& context = builder.Context();
-      DrawingRecorder drawing_recorder(context, layer.GetLayoutObject(),
-                                       DisplayItem::kReflectionMask,
-                                       mask_bounding_float_rect);
-      NinePieceImagePainter().Paint(builder.Context(), layer.GetLayoutObject(),
-                                    mask_rect, style, mask_nine_piece,
-                                    SkBlendMode::kSrcOver);
-    }
-    mask = builder.EndRecording();
+  LayoutRect mask_rect(LayoutPoint(), frame_layout_rect.Size());
+  LayoutRect mask_bounding_rect(mask_rect);
+  mask_bounding_rect.Expand(style.ImageOutsets(mask_nine_piece));
+  FloatRect mask_bounding_float_rect(mask_bounding_rect);
+
+  // TODO(jbroman): PaintRecordBuilder + DrawingRecorder seems excessive.
+  // If NinePieceImagePainter operated on SkCanvas, we'd only need a
+  // PictureRecorder here.
+  PaintRecordBuilder builder(mask_bounding_float_rect);
+  {
+    GraphicsContext& context = builder.Context();
+    DrawingRecorder drawing_recorder(context, layer.GetLayoutObject(),
+                                     DisplayItem::kReflectionMask,
+                                     mask_bounding_float_rect);
+    NinePieceImagePainter().Paint(builder.Context(), layer.GetLayoutObject(),
+                                  mask_rect, style, mask_nine_piece,
+                                  SkBlendMode::kSrcOver);
   }
-
-  return BoxReflection(direction, offset, std::move(mask));
+  return BoxReflection(direction, offset, builder.EndRecording(),
+                       mask_bounding_float_rect);
 }
 
 }  // namespace blink