Index: cc/resources/picture_pile_impl.cc |
diff --git a/cc/resources/picture_pile_impl.cc b/cc/resources/picture_pile_impl.cc |
index c56a88f802379357ba3694f3fe13349fc8a4b3db..d629888b871e855fb6a98fa4ad7e7a5fb13b586f 100644 |
--- a/cc/resources/picture_pile_impl.cc |
+++ b/cc/resources/picture_pile_impl.cc |
@@ -16,16 +16,6 @@ |
#include "third_party/skia/include/core/SkPictureRecorder.h" |
#include "ui/gfx/geometry/rect_conversions.h" |
-namespace { |
- |
-#ifdef NDEBUG |
-const bool kDefaultClearCanvasSetting = false; |
-#else |
-const bool kDefaultClearCanvasSetting = true; |
-#endif |
- |
-} // namespace |
- |
namespace cc { |
scoped_refptr<PicturePileImpl> PicturePileImpl::CreateFromPicturePile( |
@@ -40,7 +30,7 @@ PicturePileImpl::PicturePileImpl() |
is_solid_color_(false), |
solid_color_(SK_ColorTRANSPARENT), |
has_any_recordings_(false), |
- clear_canvas_with_debug_color_(kDefaultClearCanvasSetting), |
+ clear_canvas_with_debug_color_(false), |
min_contents_scale_(0.f), |
slow_down_raster_scale_factor_for_debug_(0), |
should_attempt_to_use_distance_field_text_(false) { |
@@ -49,14 +39,14 @@ PicturePileImpl::PicturePileImpl() |
PicturePileImpl::PicturePileImpl(const PicturePile* other) |
: picture_map_(other->picture_map_), |
tiling_(other->tiling_), |
- background_color_(SK_ColorTRANSPARENT), |
- requires_clear_(true), |
+ background_color_(other->background_color_), |
+ requires_clear_(other->requires_clear_), |
can_use_lcd_text_(other->can_use_lcd_text_), |
is_solid_color_(other->is_solid_color_), |
solid_color_(other->solid_color_), |
recorded_viewport_(other->recorded_viewport_), |
has_any_recordings_(other->has_any_recordings_), |
- clear_canvas_with_debug_color_(kDefaultClearCanvasSetting), |
+ clear_canvas_with_debug_color_(other->clear_canvas_with_debug_color_), |
min_contents_scale_(other->min_contents_scale_), |
slow_down_raster_scale_factor_for_debug_( |
other->slow_down_raster_scale_factor_for_debug_), |
@@ -88,7 +78,6 @@ void PicturePileImpl::PlaybackToCanvas(SkCanvas* canvas, |
RasterSourceHelper::PrepareForPlaybackToCanvas( |
canvas, canvas_rect, gfx::Rect(tiling_.tiling_size()), contents_scale, |
background_color_, clear_canvas_with_debug_color_, requires_clear_); |
- |
RasterCommon(canvas, |
NULL, |
canvas_rect, |
@@ -367,19 +356,12 @@ void PicturePileImpl::SetShouldAttemptToUseDistanceFieldText() { |
should_attempt_to_use_distance_field_text_ = true; |
} |
-void PicturePileImpl::SetBackgoundColor(SkColor background_color) { |
- background_color_ = background_color; |
-} |
- |
-void PicturePileImpl::SetRequiresClear(bool requires_clear) { |
- requires_clear_ = requires_clear; |
-} |
- |
bool PicturePileImpl::ShouldAttemptToUseDistanceFieldText() const { |
return should_attempt_to_use_distance_field_text_; |
} |
-void PicturePileImpl::AsValueInto(base::debug::TracedValue* pictures) const { |
+void PicturePileImpl::AsValueInto( |
+ base::trace_event::TracedValue* pictures) const { |
gfx::Rect tiling_rect(tiling_.tiling_size()); |
std::set<const void*> appended_pictures; |
bool include_borders = true; |
@@ -453,10 +435,8 @@ void PicturePileImpl::PixelRefIterator::AdvanceToTilePictureWithPixelRefs() { |
void PicturePileImpl::DidBeginTracing() { |
std::set<const void*> processed_pictures; |
- for (PictureMap::iterator it = picture_map_.begin(); |
- it != picture_map_.end(); |
- ++it) { |
- const Picture* picture = it->second.GetPicture(); |
+ for (const auto& map_pair : picture_map_) { |
+ const Picture* picture = map_pair.second.GetPicture(); |
if (picture && (processed_pictures.count(picture) == 0)) { |
picture->EmitTraceSnapshot(); |
processed_pictures.insert(picture); |