Tiling priorities in Android Webview.

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 381 matching lines @@
   // TODO(danakj): We should always get an occlusion tracker when we are using
   // occlusion, so update this check when we don't use a pending tree in the
   // browser compositor.
   DCHECK(!occlusion_tracker ||
          layer_tree_impl()->settings().use_occlusion_for_tile_prioritization);
 
   // Transforms and viewport are invalid for tile management inside a
   // resourceless software draw, so don't update them.
   if (!layer_tree_impl()->resourceless_software_draw()) {
     visible_rect_for_tile_priority_ = visible_content_rect();
-    viewport_size_for_tile_priority_ = layer_tree_impl()->DrawViewportSize();
+    viewport_rect_for_tile_priority_ = layer_tree_impl()->ExternalTilingRect();
     screen_space_transform_for_tile_priority_ = screen_space_transform();
   }
 
   if (!CanHaveTilings()) {
     ideal_page_scale_ = 0.f;
     ideal_device_scale_ = 0.f;
     ideal_contents_scale_ = 0.f;
     ideal_source_scale_ = 0.f;
     SanityCheckTilingState();
     return;
@@ skipping 37 matching lines @@
   for (size_t i = 0; i < tilings_->num_tilings(); ++i) {
     if (tilings_->tiling_at(i)->NeedsUpdateForFrameAtTime(
             current_frame_time_in_seconds)) {
       tiling_needs_update = true;
       break;
     }
   }
   if (!tiling_needs_update)
     return;
 
-  // Use visible_content_rect, unless it's empty. If it's empty, then
-  // try to inverse project the viewport into layer space and use that.
+  // If visible_rect_for_tile_priority_ is empty or
+  // LayerTreeImpl::ExternalTilingRect is set to be different from the device
+  // viewport, try to inverse project the viewport into layer space and use
+  // that. Otherwise just use visible_rect_for_tile_priority_
   gfx::Rect visible_rect_in_content_space = visible_rect_for_tile_priority_;
-  if (visible_rect_in_content_space.IsEmpty()) {
+  gfx::Rect external_tiling_rect = layer_tree_impl()->ExternalTilingRect();
+  gfx::Transform external_tiling_transform =
+      layer_tree_impl()->ExternalTilingTransform();
+  gfx::Transform identity;
+  gfx::Transform layer_to_screen = screen_space_transform_for_tile_priority_;
+  layer_to_screen.ConcatTransform(external_tiling_transform);

[aelias_OOO_until_Jul13, 2014/07/16 21:23:31]

Please bake it into screen_space_transform_for_tile_priority_ inside
UpdateTiles() instead.

[hush (inactive), 2014/07/17 00:05:53]

Done.

+  if (visible_rect_in_content_space.IsEmpty() ||
+      layer_tree_impl()->DrawViewportSize() != external_tiling_rect.size() ||

[aelias_OOO_until_Jul13, 2014/07/16 21:23:31]

Please cast the DrawViewportSize to a rect instead.  Right now, this is missing
the case where the size is the same but the rect position is nonzero.

[hush (inactive), 2014/07/17 00:05:53]

Yes, indeed it is missing that case.
I will use DeviceViewport()
On 2014/07/16 21:23:31, aelias wrote:

+      !external_tiling_transform.IsIdentity()) {
     gfx::Transform screen_to_layer(gfx::Transform::kSkipInitialization);
-    if (screen_space_transform_for_tile_priority_.GetInverse(
-            &screen_to_layer)) {
-      visible_rect_in_content_space =
-          gfx::ToEnclosingRect(MathUtil::ProjectClippedRect(
-              screen_to_layer, gfx::Rect(viewport_size_for_tile_priority_)));
+
+    if (layer_to_screen.GetInverse(&screen_to_layer)) {
+      // Transform from view space to content space.
+      visible_rect_in_content_space = gfx::ToEnclosingRect(
+          MathUtil::ProjectClippedRect(screen_to_layer, external_tiling_rect));
+
       visible_rect_in_content_space.Intersect(gfx::Rect(content_bounds()));
     }
   }
 
   gfx::Rect visible_layer_rect = gfx::ScaleToEnclosingRect(
       visible_rect_in_content_space, 1.f / contents_scale_x());
   WhichTree tree =
       layer_tree_impl()->IsActiveTree() ? ACTIVE_TREE : PENDING_TREE;
   for (size_t i = 0; i < tilings_->num_tilings(); ++i) {
     tilings_->tiling_at(i)->UpdateTilePriorities(tree,
@@ skipping 1155 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