cc: Complete pending tile uploads if they are needed for tree activation

cc/picture_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/picture_layer_impl.h"
 
 #include "base/time.h"
 #include "cc/append_quads_data.h"
 #include "cc/checkerboard_draw_quad.h"
 #include "cc/debug_border_draw_quad.h"
@@ skipping 490 matching lines @@
 
     if (tiling->contents_scale() < min_acceptable_scale)
       continue;
 
     for (PictureLayerTiling::Iterator iter(tiling,
                                            contentsScaleX(),
                                            rect,
                                            PictureLayerTiling::LayerDeviceAlignmentUnknown);
          iter;
          ++iter) {
+      if (iter)
+        layerTreeImpl()->tile_manager()->FlushPendingTileUploadIfNeeded(*iter);

[reveman, 2013/03/07 20:31:58]

We should avoid this when in "smoothness takes priority" mode. Probably also
when animations are running. I think should be as conservative as possible in
this first patch to prevent any kind of smoothness regression.

[Sami, 2013/03/08 18:01:10]

Agreed, let's be careful about this. I've added a check for smoothness and
animations.

       // A null tile (i.e. no recording) is considered "ready".
       if (!*iter || iter->GetResourceId())
         missing_region.Subtract(iter.geometry_rect());
     }
   }
 
   return missing_region.IsEmpty();
 }
 
 PictureLayerTiling* PictureLayerImpl::AddTiling(float contents_scale) {
@@ skipping 259 matching lines @@
 scoped_ptr<base::Value> PictureLayerImpl::AsValue() const {
   scoped_ptr<base::DictionaryValue> state(new base::DictionaryValue());
   LayerImpl::AsValueInto(state.get());
 
   state->SetDouble("ideal_contents_scale", ideal_contents_scale_);
   state->Set("tilings", tilings_->AsValue().release());
   return state.PassAs<base::Value>();
 }
 
 }  // namespace cc