cc: Don't draw and swap if the frame will not change.

cc/trees/damage_tracker_unittest.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/damage_tracker.h"
 
 #include "cc/base/math_util.h"
 #include "cc/layers/layer_impl.h"
 #include "cc/test/fake_impl_proxy.h"
 #include "cc/test/fake_layer_tree_host_impl.h"
@@ skipping 1244 matching lines @@
   ClearDamageForAllSurfaces(root.get());
   replica_mask_layer->SetStackingOrderChanged(true);
 
   EmulateDrawingOneFrame(root.get());
 
   gfx::RectF child_damage_rect =
           child1->render_surface()->damage_tracker()->current_damage_rect();
   EXPECT_FLOAT_RECT_EQ(gfx::RectF(206.f, 200.f, 6.f, 8.f), child_damage_rect);
 }
 
-TEST_F(DamageTrackerTest, VerifyDamageWhenForcedFullDamage) {
+TEST_F(DamageTrackerTest, DamageWhenAddedExternally) {
   scoped_ptr<LayerImpl> root = CreateAndSetUpTestTreeWithOneSurface();
   LayerImpl* child = root->children()[0];
 
-  // Case 1: This test ensures that when the tracker is forced to have full
-  //         damage, that it takes priority over any other partial damage.
+  // Case 1: This test ensures that when the tracker is given damage, that
+  //         it is included with any other partial damage.
   //
   ClearDamageForAllSurfaces(root.get());
-  child->set_update_rect(gfx::RectF(10.f, 11.f, 12.f, 13.f));
-  root->render_surface()->damage_tracker()->ForceFullDamageNextUpdate();
+  child->set_update_rect(gfx::RectF(10, 11, 12, 13));
+  root->render_surface()->damage_tracker()->AddDamageNextUpdate(
+      gfx::RectF(15, 16, 32, 33));
   EmulateDrawingOneFrame(root.get());
   gfx::RectF root_damage_rect =
-          root->render_surface()->damage_tracker()->current_damage_rect();
-  EXPECT_FLOAT_RECT_EQ(gfx::RectF(0.f, 0.f, 500.f, 500.f), root_damage_rect);
+      root->render_surface()->damage_tracker()->current_damage_rect();
+  EXPECT_FLOAT_RECT_EQ(
+      gfx::UnionRects(gfx::RectF(15, 16, 32, 33),
+                      gfx::RectF(100+10, 100+11, 12, 13)),
+      root_damage_rect);
 
-  // Case 2: An additional sanity check that forcing full damage works even
-  //         when nothing on the layer tree changed.
+  // Case 2: An additional sanity check that adding damage works even when
+  //         nothing on the layer tree changed.
   //
   ClearDamageForAllSurfaces(root.get());
-  root->render_surface()->damage_tracker()->ForceFullDamageNextUpdate();
+  root->render_surface()->damage_tracker()->AddDamageNextUpdate(
+      gfx::RectF(30, 31, 14, 15));
   EmulateDrawingOneFrame(root.get());
   root_damage_rect =
-          root->render_surface()->damage_tracker()->current_damage_rect();
-  EXPECT_FLOAT_RECT_EQ(gfx::RectF(0.f, 0.f, 500.f, 500.f), root_damage_rect);
+      root->render_surface()->damage_tracker()->current_damage_rect();
+  EXPECT_FLOAT_RECT_EQ(gfx::RectF(30, 31, 14, 15), root_damage_rect);
 }
 
 TEST_F(DamageTrackerTest, VerifyDamageForEmptyLayerList) {
   // Though it should never happen, its a good idea to verify that the damage
   // tracker does not crash when it receives an empty layer_list.
 
   scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_.active_tree(), 1);
   root->CreateRenderSurface();
 
   ASSERT_TRUE(root == root->render_target());
@@ skipping 46 matching lines @@
   // Damage should remain empty even after one frame, since there's yet no new
   // damage.
   EmulateDrawingOneFrame(root.get());
   root_damage_rect =
           root->render_surface()->damage_tracker()->current_damage_rect();
   EXPECT_TRUE(root_damage_rect.IsEmpty());
 }
 
 }  // namespace
 }  // namespace cc