cc : Reland Add IsInIdtoIndexMap to property tres

cc/layers/scrollbar_layer_unittest.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 <stddef.h>
 
 #include <unordered_map>
 
 #include "base/threading/thread_task_runner_handle.h"
 #include "cc/input/scrollbar_animation_controller.h"
@@ skipping 81 matching lines @@
   int next_id_;
   int total_ui_resource_created_;
   int total_ui_resource_deleted_;
 };
 
 class ScrollbarLayerTest : public testing::Test {
  public:
   ScrollbarLayerTest() : fake_client_(FakeLayerTreeHostClient::DIRECT_3D) {
     layer_tree_settings_.single_thread_proxy_scheduler = false;
     layer_tree_settings_.use_zero_copy = true;
+    layer_tree_settings_.scrollbar_animator = LayerTreeSettings::LINEAR_FADE;
+    layer_tree_settings_.scrollbar_fade_delay_ms = 20;
+    layer_tree_settings_.scrollbar_fade_duration_ms = 20;
 
     scrollbar_layer_id_ = -1;
 
     LayerTreeHost::InitParams params;
     params.client = &fake_client_;
     params.settings = &layer_tree_settings_;
     params.task_graph_runner = &task_graph_runner_;
 
     layer_tree_host_.reset(
         new FakeResourceTrackingLayerTreeHost(&fake_client_, &params));
@@ skipping 431 matching lines @@
   EXPECT_EQ(node->data.opacity, 0.5f);
   // The active tree opacity should not change with activation for scrollbar
   // layer.
   host_impl->ActivateSyncTree();
   layer_tree_impl = host_impl->active_tree();
   node = layer_tree_impl->property_trees()->effect_tree.Node(
       scrollbar_layer->effect_tree_index());
   EXPECT_EQ(node->data.opacity, 1.f);
 }
 
+TEST_F(ScrollbarLayerTest, ScrollbarLayerPushProperties) {
+  // Pushing changed bounds of scroll layer can lead to calling
+  // OnOpacityAnimated on scrollbar layer which means OnOpacityAnimated should
+  // be independent of scrollbar layer's properties as scrollbar layer can push
+  // its properties after scroll layer.
+  const int kThumbThickness = 3;
+  const int kTrackStart = 0;
+  std::unique_ptr<Scrollbar> scrollbar(new FakeScrollbar(false, true, true));
+
+  scoped_refptr<Layer> layer_tree_root = Layer::Create();
+  scoped_refptr<Layer> scroll_layer = Layer::Create();
+  scroll_layer->SetScrollClipLayerId(layer_tree_root->id());
+  scoped_refptr<Layer> child1 = Layer::Create();
+  scoped_refptr<Layer> scrollbar_layer;
+  const bool kIsLeftSideVerticalScrollbar = false;
+  scrollbar_layer = SolidColorScrollbarLayer::Create(
+      scrollbar->Orientation(), kThumbThickness, kTrackStart,
+      kIsLeftSideVerticalScrollbar, child1->id());
+  scroll_layer->SetScrollClipLayerId(layer_tree_root->id());
+  scrollbar_layer->ToScrollbarLayer()->SetScrollLayer(scroll_layer->id());
+  scroll_layer->AddChild(child1);
+  scroll_layer->InsertChild(scrollbar_layer, 1);
+  layer_tree_root->AddChild(scroll_layer);
+  layer_tree_host_->SetRootLayer(layer_tree_root);
+
+  layer_tree_root->SetBounds(gfx::Size(2, 2));
+  scroll_layer->SetBounds(gfx::Size(10, 10));
+  layer_tree_host_->UpdateLayers();
+  LayerImpl* layer_impl_tree_root =
+      layer_tree_host_->CommitAndCreateLayerImplTree();
+  LayerTreeHostImpl* host_impl = layer_tree_host_->host_impl();
+  EXPECT_TRUE(host_impl->ScrollbarAnimationControllerForId(scroll_layer->id()));
+
+  scroll_layer->SetBounds(gfx::Size(20, 20));
+  scroll_layer->SetForceRenderSurfaceForTesting(true);
+  layer_tree_host_->UpdateLayers();
+  host_impl->CreatePendingTree();
+  layer_impl_tree_root = layer_tree_host_->CommitAndCreatePendingTree();
+  host_impl->ActivateSyncTree();
+  EffectNode* node =
+      host_impl->active_tree()->property_trees()->effect_tree.Node(
+          scrollbar_layer->effect_tree_index());
+  EXPECT_EQ(node->data.opacity, 1.f);

[jaydasika, 2016/05/26 02:17:06]

If we use effect tree index in LayerImpl::OnOpacityAnimated and keep the DCHECK
around, this tests will flakily fail the DCHECK. 

+}
+
 class ScrollbarLayerSolidColorThumbTest : public testing::Test {
  public:
   ScrollbarLayerSolidColorThumbTest() {
     LayerTreeSettings layer_tree_settings;
     host_impl_.reset(new FakeLayerTreeHostImpl(
         layer_tree_settings, &task_runner_provider_, &shared_bitmap_manager_,
         &task_graph_runner_));
 
     const int kThumbThickness = 3;
     const int kTrackStart = 0;
@@ skipping 509 matching lines @@
   TestScale(gfx::Rect(1240, 0, 15, 1333), 2.7754839f);
   TestScale(gfx::Rect(1240, 0, 15, 677), 2.46677136f);
 
   // Horizontal Scrollbars.
   TestScale(gfx::Rect(0, 1240, 1333, 15), 2.7754839f);
   TestScale(gfx::Rect(0, 1240, 677, 15), 2.46677136f);
 }
 
 }  // namespace
 }  // namespace cc