The snap ratio is wrong

cc/layers/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/layers/picture_layer_impl.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/time/time.h"
@@ skipping 1033 matching lines @@
   // Always match the ideal device scale factor.
   if (raster_device_scale_ != ideal_device_scale_)
     return true;
 
   return false;
 }
 
 float PictureLayerImpl::SnappedContentsScale(float scale) {
   // If a tiling exists within the max snapping ratio, snap to its scale.
   float snapped_contents_scale = scale;
-  float snapped_ratio = kSnapToExistingTilingRatio;
+  float snapped_ratio = kMaxScaleRatioDuringPinch;

[danakj, 2014/04/29 17:30:04]

This was purposely written as a separate constant. Can you give an example of
when this is doing the wrong thing?

   for (size_t i = 0; i < tilings_->num_tilings(); ++i) {
     float tiling_contents_scale = tilings_->tiling_at(i)->contents_scale();
     float ratio = PositiveRatio(tiling_contents_scale, scale);
     if (ratio < snapped_ratio) {
       snapped_contents_scale = tiling_contents_scale;
       snapped_ratio = ratio;
     }
   }
   return snapped_contents_scale;
 }
@@ skipping 452 matching lines @@
   return iterator_index_ < iterators_.size();
 }
 
 bool PictureLayerImpl::LayerEvictionTileIterator::IsCorrectType(
     PictureLayerTiling::TilingEvictionTileIterator* it) const {
   return it->get_type() == iteration_stage_ &&
          (**it)->required_for_activation() == required_for_activation_;
 }
 
 }  // namespace cc