BrowserMainLoop owns FrameSinkManagerHost

content/browser/renderer_host/offscreen_canvas_provider_impl_unittest.cc

Index: content/browser/renderer_host/offscreen_canvas_provider_impl_unittest.cc
diff --git a/content/browser/renderer_host/offscreen_canvas_provider_impl_unittest.cc b/content/browser/renderer_host/offscreen_canvas_provider_impl_unittest.cc
index 6d2b10787ca64cf8dd98ba1c853ff6ad5c67822f..c6c5ef49f62a6097e3332f4cedef1c34972306a2 100644
--- a/content/browser/renderer_host/offscreen_canvas_provider_impl_unittest.cc
+++ b/content/browser/renderer_host/offscreen_canvas_provider_impl_unittest.cc
@@ -147,13 +147,11 @@ class OffscreenCanvasProviderImplTest : public testing::Test {
     ImageTransportFactory::InitializeForUnitTests(
         std::unique_ptr<ImageTransportFactory>(
             new NoTransportImageTransportFactory));
-    ImageTransportFactory::GetInstance()
-        ->GetContextFactoryPrivate()
-        ->GetFrameSinkManagerHost()
-        ->ConnectToFrameSinkManager();
 #endif
-    provider_ =
-        base::MakeUnique<OffscreenCanvasProviderImpl>(kRendererClientId);
+    frame_sink_manager_host_ = base::MakeUnique<viz::FrameSinkManagerHost>();

[danakj, 2017/06/09 22:43:37]

Please destroy things in TearDown that are created in SetUp.

[Fady Samuel, 2017/06/09 22:52:39]

Done.

+    frame_sink_manager_host_->ConnectToFrameSinkManager();
+    provider_ = base::MakeUnique<OffscreenCanvasProviderImpl>(
+        frame_sink_manager_host_.get(), kRendererClientId);
   }
   void TearDown() override {
     provider_.reset();
@@ -164,6 +162,7 @@ class OffscreenCanvasProviderImplTest : public testing::Test {
 
  private:
   base::MessageLoop message_loop_;
+  std::unique_ptr<viz::FrameSinkManagerHost> frame_sink_manager_host_;
   std::unique_ptr<OffscreenCanvasProviderImpl> provider_;
 };