Mark Layer Damaged When The Client Is Gone

cc/layers/video_frame_provider_client_impl_unittest.cc

Index: cc/layers/video_frame_provider_client_impl_unittest.cc
diff --git a/cc/layers/video_frame_provider_client_impl_unittest.cc b/cc/layers/video_frame_provider_client_impl_unittest.cc
index eaa0e6fe40fc59b8ed8261f70b66977cc8533335..c93ff673d335b142e564e8aeec88d39d3810ccbc 100644
--- a/cc/layers/video_frame_provider_client_impl_unittest.cc
+++ b/cc/layers/video_frame_provider_client_impl_unittest.cc
@@ -102,6 +102,14 @@ TEST_F(VideoFrameProviderClientImplTest, StartStopRendering) {
   StopRendering();
 }
 
+TEST_F(VideoFrameProviderClientImplTest, StopRenderingUpdateDamage) {
+  CreateActiveVideoLayer();
+  StartRendering();
+  EXPECT_EQ(gfx::Rect(), video_layer_impl_->update_rect());
+  StopRendering();

[danakj, 2017/02/01 17:52:09]

Can you also stop before start and see no damage happens?

Can you also test StopUsingProvider()?

[qiangchen, 2017/02/01 18:51:59]

That does not work, because in StopRendering there is a DCHECK(rendering_),
which will fail the test.

Added test on StopUsingProvider

+  EXPECT_NE(gfx::Rect(), video_layer_impl_->update_rect());
+}
+
 TEST_F(VideoFrameProviderClientImplTest, StopUsingProvider) {
   ASSERT_TRUE(client_impl_->get_provider_for_testing());
   StartRendering();