cc: Fix lost context handling when using impl-side painting.

cc/layer_tree_host_unittest_context.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/layer_tree_host.h"
 
 #include "base/basictypes.h"
 #include "cc/content_layer.h"
 #include "cc/delegated_renderer_layer.h"
 #include "cc/delegated_renderer_layer_impl.h"
 #include "cc/heads_up_display_layer.h"
 #include "cc/io_surface_layer.h"
 #include "cc/layer_impl.h"
 #include "cc/layer_tree_host_impl.h"
 #include "cc/layer_tree_impl.h"
+#include "cc/picture_layer.h"
 #include "cc/scrollbar_layer.h"
 #include "cc/single_thread_proxy.h"
 #include "cc/test/fake_content_layer.h"
 #include "cc/test/fake_content_layer_client.h"
 #include "cc/test/fake_content_layer_impl.h"
 #include "cc/test/fake_output_surface.h"
 #include "cc/test/fake_scrollbar_theme_painter.h"
 #include "cc/test/fake_video_frame_provider.h"
 #include "cc/test/fake_web_graphics_context_3d.h"
 #include "cc/test/fake_web_scrollbar.h"
@@ skipping 779 matching lines @@
 
   virtual void didRecreateOutputSurface(bool succeeded) OVERRIDE {
     EXPECT_FALSE(succeeded);
     // If we make it this far without crashing, we pass!
     endTest();
   }
 };
 
 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestFailsImmediately);
 
+class ImplSidePaintingLayerTreeHostContextTest

[enne (OOO), 2013/01/15 22:35:41]

How would this test fail before your patch?

I was wondering if this could go in
LayerTreeHostContextTestDontUseLostResources.

[reveman, 2013/01/15 23:16:26]

The DCHECK(tiles_.size() == 0) in the tile manager the destructor fails.
The only reason I didn't just include a picture layer in that test is that it's
run both in single and multi threaded mode. I could make some changes that
allows me to subclass and reuse that test for impl side painting in mult thread
mode only if you prefer that over adding a new test like my current patch is
doing?

+    : public LayerTreeHostContextTest {
+ public:
+  virtual void initializeSettings(LayerTreeSettings& settings) OVERRIDE {
+    settings.implSidePainting = true;
+  }
+};
+
+class LayerTreeHostContextTestImplSidePainting :
+    public ImplSidePaintingLayerTreeHostContextTest {
+ public:
+  virtual void setupTree() OVERRIDE {
+    scoped_refptr<Layer> root = Layer::create();
+    root->setBounds(gfx::Size(10, 10));
+    root->setAnchorPoint(gfx::PointF());
+    root->setIsDrawable(true);
+
+    scoped_refptr<PictureLayer> picture = PictureLayer::create(&client_);
+    picture->setBounds(gfx::Size(10, 10));
+    picture->setAnchorPoint(gfx::PointF());
+    picture->setIsDrawable(true);
+    root->addChild(picture);
+
+    m_layerTreeHost->setRootLayer(root);
+    LayerTreeHostContextTest::setupTree();
+  }
+
+  virtual void beginTest() OVERRIDE {
+    times_to_lose_during_commit_ = 1;
+    postSetNeedsCommitToMainThread();
+  }
+
+  virtual void afterTest() OVERRIDE {}
+
+  virtual void didRecreateOutputSurface(bool succeeded) OVERRIDE {
+    EXPECT_TRUE(succeeded);
+    endTest();
+  }
+
+ private:
+  FakeContentLayerClient client_;
+};
+
+MULTI_THREAD_TEST_F(LayerTreeHostContextTestImplSidePainting)
+
 }  // namespace
 }  // namespace cc