Switch to use SharedBitmapManager all the time in cc_unittests

cc/test/fake_layer_tree_host_impl.cc

Index: cc/test/fake_layer_tree_host_impl.cc
diff --git a/cc/test/fake_layer_tree_host_impl.cc b/cc/test/fake_layer_tree_host_impl.cc
index 8c39d2e9ce7e93bb09ed21a7b75f70565a16a84e..309d3c897361ab92caf65d2864e114d2c738a5fe 100644
--- a/cc/test/fake_layer_tree_host_impl.cc
+++ b/cc/test/fake_layer_tree_host_impl.cc
@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 #include "cc/test/fake_layer_tree_host_impl.h"
+#include "cc/test/test_shared_bitmap_manager.h"
 #include "cc/trees/layer_tree_impl.h"
 
 namespace cc {
@@ -17,6 +18,8 @@ FakeLayerTreeHostImpl::FakeLayerTreeHostImpl(Proxy* proxy)
   // Explicitly clear all debug settings.
   SetDebugState(LayerTreeDebugState());
   SetViewportSize(gfx::Size(100, 100));
+  bitmap_manager_.reset(new TestSharedBitmapManager);

[danakj, 2014/03/18 23:42:44]

This is kinda unfortunate. Can we own it outside this class and pass it to the
constructor? This might look a lot like the LayerTreeHostFactory in
https://codereview.chromium.org/202763002/diff/30001/cc/layers/layer_unittest.cc

+  shared_bitmap_manager_ = bitmap_manager_.get();
 }
 
 FakeLayerTreeHostImpl::FakeLayerTreeHostImpl(const LayerTreeSettings& settings,
@@ -29,6 +32,8 @@ FakeLayerTreeHostImpl::FakeLayerTreeHostImpl(const LayerTreeSettings& settings,
                         0) {
   // Explicitly clear all debug settings.
   SetDebugState(LayerTreeDebugState());
+  bitmap_manager_.reset(new TestSharedBitmapManager);
+  shared_bitmap_manager_ = bitmap_manager_.get();
 }
 
 FakeLayerTreeHostImpl::~FakeLayerTreeHostImpl() {}