Convert MediaLog from being ref counted to owned by WebMediaPlayer.

media/blink/webmediaplayer_impl_unittest.cc

 // Copyright 2016 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 "media/blink/webmediaplayer_impl.h"
 
 #include <stdint.h>
 
 #include <memory>
 
@@ skipping 186 matching lines @@
   WebMediaPlayerImplTest()
       : media_thread_("MediaThreadForTest"),
         web_view_(
             blink::WebView::Create(nullptr,
                                    blink::kWebPageVisibilityStateVisible)),
         web_local_frame_(
             blink::WebLocalFrame::Create(blink::WebTreeScopeType::kDocument,
                                          &web_frame_client_,
                                          nullptr,
                                          nullptr)),
-        media_log_(new MediaLog()),
         audio_parameters_(TestAudioParameters::Normal()) {
     web_view_->SetMainFrame(web_local_frame_);
     media_thread_.StartAndWaitForTesting();
   }
 
   void InitializeWebMediaPlayerImpl(bool allow_suspend) {
+    std::unique_ptr<MediaLog> media_log(new MediaLog());
+    std::unique_ptr<RendererFactory> renderer_factory(
+        new DefaultRendererFactory(
+            media_log.get(), nullptr,
+            DefaultRendererFactory::GetGpuFactoriesCB()));
+
     wmpi_ = base::MakeUnique<WebMediaPlayerImpl>(
         web_local_frame_, &client_, nullptr, &delegate_,
-        base::MakeUnique<DefaultRendererFactory>(
-            media_log_, nullptr, DefaultRendererFactory::GetGpuFactoriesCB()),
-        url_index_,
-        WebMediaPlayerParams(
-            WebMediaPlayerParams::DeferLoadCB(),
-            scoped_refptr<SwitchableAudioRendererSink>(), media_log_,
+        std::move(renderer_factory), url_index_,
+        base::MakeUnique<WebMediaPlayerParams>(
+            std::move(media_log), WebMediaPlayerParams::DeferLoadCB(),
+            scoped_refptr<SwitchableAudioRendererSink>(),
             media_thread_.task_runner(), message_loop_.task_runner(),
             message_loop_.task_runner(), WebMediaPlayerParams::Context3DCB(),
             base::Bind(&OnAdjustAllocatedMemory), nullptr, nullptr, nullptr,
             kMaxKeyframeDistanceToDisableBackgroundVideo,
             kMaxKeyframeDistanceToDisableBackgroundVideoMSE, false,
             allow_suspend, false));
   }
 
   ~WebMediaPlayerImplTest() override {
     // Destruct WebMediaPlayerImpl and pump the message loop to ensure that
@@ skipping 117 matching lines @@
 
   // "Media" thread. This is necessary because WMPI destruction waits on a
   // WaitableEvent.
   base::Thread media_thread_;
 
   // Blink state.
   blink::WebFrameClient web_frame_client_;
   blink::WebView* web_view_;
   blink::WebLocalFrame* web_local_frame_;
 
-  scoped_refptr<MediaLog> media_log_;
   linked_ptr<media::UrlIndex> url_index_;
 
   // Audio hardware configuration.
   AudioParameters audio_parameters_;
 
   // The client interface used by |wmpi_|. Just a dummy for now, but later we
   // may want a mock or intelligent fake.
   DummyWebMediaPlayerClient client_;
 
   testing::NiceMock<MockWebMediaPlayerDelegate> delegate_;
@@ skipping 610 matching lines @@
                         WebMediaPlayerImplBackgroundBehaviorTest,
                         ::testing::Combine(::testing::Bool(),
                                            ::testing::Bool(),
                                            ::testing::Values(5, 300),
                                            ::testing::Values(5, 100),
                                            ::testing::Bool(),
                                            ::testing::Bool(),
                                            ::testing::Bool()));
 
 }  // namespace media