Reuse property trees

cc/trees/layer_tree_host_common.cc

 // Copyright 2011 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/trees/layer_tree_host_common.h"
 
 #include <algorithm>
 
 #include "base/trace_event/trace_event.h"
 #include "cc/base/math_util.h"
@@ skipping 2538 matching lines @@
       const bool draw_transforms_match =
           ApproximatelyEqual(current_layer->draw_transform(),
                              current_layer->draw_transform_from_property_trees(
                                  inputs->property_trees->transform_tree));
       CHECK(draw_transforms_match);
 
       const bool draw_opacities_match =
           current_layer->draw_opacity() ==
           current_layer->DrawOpacityFromPropertyTrees(
               inputs->property_trees->opacity_tree);
-      CHECK(draw_opacities_match);
+      CHECK(draw_opacities_match)

[enne (OOO), 2015/04/16 21:49:33]

This seems like it should get done in a separate patch?

[Ian Vollick, 2015/04/17 14:56:22]

Done.

+          << "expected: " << current_layer->draw_opacity() << " actual: "
+          << current_layer->DrawOpacityFromPropertyTrees(
+                 inputs->property_trees->opacity_tree);
     }
   }
 
   // The dummy layer list should not have been used.
   DCHECK_EQ(0u, dummy_layer_list.size());
   // A root layer render_surface should always exist after
   // CalculateDrawProperties.
   DCHECK(inputs->root_layer->render_surface());
 }
 
@@ skipping 17 matching lines @@
       inputs->current_render_surface_layer_list_id);
 
   // The dummy layer list should not have been used.
   DCHECK_EQ(0u, dummy_layer_list.size());
   // A root layer render_surface should always exist after
   // CalculateDrawProperties.
   DCHECK(inputs->root_layer->render_surface());
 }
 
 }  // namespace cc