Reland "Shouldn't access LayerTreeTest::layer_tree_host on impl thread"

cc/trees/layer_tree_host_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/layer_tree_host.h"
 
 #include <algorithm>
 
 #include "base/synchronization/lock.h"
 #include "cc/animation/timing_function.h"
@@ skipping 2198 matching lines @@
   virtual void AfterTest() OVERRIDE {}
 
  private:
   FakeContentLayerClient client_;
   scoped_refptr<FakeContentLayer> root_layer_;
   scoped_refptr<FakeContentLayer> child_layer1_;
   scoped_refptr<FakeContentLayer> child_layer2_;
   int num_commits_;
 };
 
-SINGLE_AND_MULTI_THREAD_TEST_F(
+SINGLE_AND_MULTI_THREAD_MAIN_THREAD_PAINT_TEST_F(

[danakj, 2013/07/02 02:07:55]

I think I'd prefer if this test just set impl_side_painting = false in
InitializeSettings(). Any tests like this we can just delete one day. Rather
than adding macro infrastructure for tests-we'll-delete. At least, that's how
other tests like this deal with things now.

     LayerTreeHostTestShutdownWithOnlySomeResourcesEvicted);
 
 class LayerTreeHostTestLCDNotification : public LayerTreeHostTest {
  public:
   class NotificationClient : public ContentLayerClient {
    public:
     NotificationClient()
         : layer_(0), paint_count_(0), lcd_notification_count_(0) {}
 
     void set_layer(Layer* layer) { layer_ = layer; }
@@ skipping 720 matching lines @@
  private:
   FakeContentLayerClient client_;
   scoped_refptr<FakePictureLayer> layer_;
   bool initialized_gl_;
 };
 
 MULTI_THREAD_TEST_F(LayerTreeHostTestDeferredInitialize);
 
 }  // namespace
 }  // namespace cc