Always antialias clips in ClipDisplayItem.

cc/playback/clip_display_item.cc

 // 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 "cc/playback/clip_display_item.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 44 matching lines @@
   for (const auto& rrect : rounded_clip_rects_) {
     SkRRectToProto(rrect, details->add_rounded_rects());
   }
 }
 
 void ClipDisplayItem::Raster(SkCanvas* canvas,
                              const gfx::Rect& canvas_target_playback_rect,
                              SkPicture::AbortCallback* callback) const {
   canvas->save();
   canvas->clipRect(SkRect::MakeXYWH(clip_rect_.x(), clip_rect_.y(),
-                                    clip_rect_.width(), clip_rect_.height()));
+                                    clip_rect_.width(), clip_rect_.height()),
+                   SkRegion::kIntersect_Op, true);

[ajuma, 2016/01/08 14:15:04]

Please use a variable for the bool rather than just a literal, like we were
doing before in the 'else' case below.

[chrishtr, 2016/01/12 23:43:14]

Done

   for (size_t i = 0; i < rounded_clip_rects_.size(); ++i) {
     if (rounded_clip_rects_[i].isRect()) {
-      canvas->clipRect(rounded_clip_rects_[i].rect());
+      canvas->clipRect(rounded_clip_rects_[i].rect(), SkRegion::kIntersect_Op,
+                       true);
     } else {
-      bool antialiased = true;
-      canvas->clipRRect(rounded_clip_rects_[i], SkRegion::kIntersect_Op,
-                        antialiased);
+      canvas->clipRRect(rounded_clip_rects_[i], SkRegion::kIntersect_Op, true);
     }
   }
 }
 
 void ClipDisplayItem::AsValueInto(const gfx::Rect& visual_rect,
                                   base::trace_event::TracedValue* array) const {
   std::string value = base::StringPrintf(
       "ClipDisplayItem rect: [%s] visualRect: [%s]",
       clip_rect_.ToString().c_str(), visual_rect.ToString().c_str());
   for (const SkRRect& rounded_rect : rounded_clip_rects_) {
@@ skipping 47 matching lines @@
     base::trace_event::TracedValue* array) const {
   array->AppendString(base::StringPrintf("EndClipDisplayItem visualRect: [%s]",
                                          visual_rect.ToString().c_str()));
 }
 
 size_t EndClipDisplayItem::ExternalMemoryUsage() const {
   return 0;
 }
 
 }  // namespace cc