cc: Add support for partial swaps when using impl-side painting.

cc/layers/layer_impl.cc

 // Copyright 2012 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/layers/layer_impl.h"
 
 #include "base/debug/trace_event.h"
 #include "base/json/json_reader.h"
 #include "base/strings/stringprintf.h"
 #include "cc/animation/animation_registrar.h"
@@ skipping 53 matching lines @@
       hide_layer_and_subtree_(false),
       force_render_surface_(false),
       is_container_for_fixed_position_layers_(false),
       is_3d_sorted_(false),
       background_color_(0),
       opacity_(1.0),
       blend_mode_(SkXfermode::kSrcOver_Mode),
       draw_depth_(0.f),
       needs_push_properties_(false),
       num_dependents_need_push_properties_(0),
-      current_draw_mode_(DRAW_MODE_NONE) {
+      current_draw_mode_(DRAW_MODE_NONE),
+      initialized_tile_rect_(gfx::Rect()) {

[reveman, 2014/04/11 18:11:34]

no need for this. default ctor does the same.

[sohanjg, 2014/04/14 10:16:11]

Done.

   DCHECK_GT(layer_id_, 0);
   DCHECK(layer_tree_impl_);
   layer_tree_impl_->RegisterLayer(this);
   AnimationRegistrar* registrar = layer_tree_impl_->animationRegistrar();
   layer_animation_controller_ =
       registrar->GetAnimationControllerForId(layer_id_);
   layer_animation_controller_->AddValueObserver(this);
   if (IsActive())
     layer_animation_controller_->set_value_provider(this);
   SetNeedsPushProperties();
@@ skipping 925 matching lines @@
   Animation* transform_animation =
       layer_animation_controller_->GetAnimation(Animation::Transform);
   return transform_animation && transform_animation->is_impl_only();
 }
 
 void LayerImpl::SetUpdateRect(const gfx::RectF& update_rect) {
   update_rect_ = update_rect;
   SetNeedsPushProperties();
 }
 
+void LayerImpl::SetInitializedTileRect(gfx::Rect initialized_tile_rect) {

[reveman, 2014/04/11 18:11:34]

pass by const reference instead

+  if (!this)

[reveman, 2014/04/11 18:11:34]

heh, nope.

[sohanjg, 2014/04/14 10:16:11]

Done.

+    return;
+  initialized_tile_rect_ =
+      gfx::UnionRects(initialized_tile_rect_, initialized_tile_rect);

[reveman, 2014/04/11 18:11:34]

don't call this function "SetRect" and then do a union.

[sohanjg, 2014/04/14 10:16:11]

Done.

+}
+
 void LayerImpl::SetContentBounds(const gfx::Size& content_bounds) {
   if (this->content_bounds() == content_bounds)
     return;
 
   draw_properties_.content_bounds = content_bounds;
   NoteLayerPropertyChanged();
 }
 
 void LayerImpl::SetContentsScale(float contents_scale_x,
                                  float contents_scale_y) {
@@ skipping 470 matching lines @@
 scoped_ptr<base::Value> LayerImpl::AsValue() const {
   scoped_ptr<base::DictionaryValue> state(new base::DictionaryValue());
   AsValueInto(state.get());
   return state.PassAs<base::Value>();
 }
 
 void LayerImpl::RunMicroBenchmark(MicroBenchmarkImpl* benchmark) {
   benchmark->RunOnLayer(this);
 }
 }  // namespace cc