cc: Simplify the logic for deciding to update tile priorities.

cc/layers/picture_layer_impl.cc

Index: cc/layers/picture_layer_impl.cc
diff --git a/cc/layers/picture_layer_impl.cc b/cc/layers/picture_layer_impl.cc
index 3f1c322051dc2afc18e8df8d822d0abe908a1cc8..25dfecd3b484cd420beb1d32c3cecc0f25bf832a 100644
--- a/cc/layers/picture_layer_impl.cc
+++ b/cc/layers/picture_layer_impl.cc
@@ -265,12 +265,21 @@ void PictureLayerImpl::UpdateTilePriorities() {
   if (!tilings_->num_tilings())
     return;
 
-  UpdateLCDTextStatus();
-
-  int current_source_frame_number = layer_tree_impl()->source_frame_number();
   double current_frame_time = (layer_tree_impl()->CurrentFrameTimeTicks() -

[enne (OOO), 2013/04/22 19:19:40]

current_frame_time_in_seconds?

Also, can you use layerTreeImpl()->CurrentFrameTime().InSecondsF()?

[danakj, 2013/04/22 19:26:53]

ok
I don't see that function. You need a TimeDelta to get InSecondsF AFAICT.

                                base::TimeTicks()).InSecondsF();
 
+  bool tiling_needs_update = false;
+  for (size_t i = 0; i < tilings_->num_tilings(); ++i) {
+    if (tilings_->tiling_at(i)->NeedsUpdateForFrameAtTime(current_frame_time)) {
+      tiling_needs_update = true;
+      break;
+    }
+  }
+  if (!tiling_needs_update)
+    return;
+
+  UpdateLCDTextStatus();
+
   gfx::Transform current_screen_space_transform = screen_space_transform();
 
   gfx::Rect viewport_in_content_space;
@@ -298,7 +307,6 @@ void PictureLayerImpl::UpdateTilePriorities() {
       contents_scale_x(),
       last_screen_space_transform_,
       current_screen_space_transform,
-      current_source_frame_number,
       current_frame_time,
       store_screen_space_quads_on_tiles,
       max_tiles_for_interest_area);