Fix MediaRemoting MediaObserver

content/renderer/media/media_factory.cc

 // Copyright 2017 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 "content/renderer/media/media_factory.h"
 
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/field_trial_params.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 331 matching lines @@
 #if BUILDFLAG(ENABLE_MEDIA_REMOTING)
   media::mojom::RemotingSourcePtr remoting_source;
   auto remoting_source_request = mojo::MakeRequest(&remoting_source);
   media::mojom::RemoterPtr remoter;
   GetRemoterFactory()->Create(std::move(remoting_source),
                               mojo::MakeRequest(&remoter));
   using RemotingController = media::remoting::RendererController;
   std::unique_ptr<RemotingController> remoting_controller(
       new RemotingController(new media::remoting::SharedSession(
           std::move(remoting_source_request), std::move(remoter))));
-  base::WeakPtr<media::MediaObserver> media_observer =
-      remoting_controller->GetWeakPtr();
+  *out_media_observer = remoting_controller->GetWeakPtr();

[miu, 2017/06/05 20:02:03]

Hmm...Surprised the old code didn't generate a compiler error (unused var).

 
   auto courier_factory =
       base::MakeUnique<media::remoting::CourierRendererFactory>(
           std::move(remoting_controller));
 
   // base::Unretained is safe here because |factory_selector| owns
   // |courier_factory|.
   factory_selector->SetQueryIsRemotingActiveCB(
       base::Bind(&media::remoting::CourierRendererFactory::IsRemotingActive,
                  base::Unretained(courier_factory.get())));
@@ skipping 129 matching lines @@
     DCHECK(remote_interfaces_);
     media_interface_provider_.reset(
         new MediaInterfaceProvider(remote_interfaces_));
   }
 
   return media_interface_provider_.get();
 }
 #endif  // BUILDFLAG(ENABLE_MOJO_MEDIA)
 
 }  // namespace content