Rename AdaptiveRendererFactory

media/remoting/courier_renderer_factory.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/remoting/courier_renderer_factory.h"
+
+#include "base/logging.h"
+#include "base/memory/ptr_util.h"
+#include "media/remoting/courier_renderer.h"
+
+namespace media {
+namespace remoting {
+
+CourierRendererFactory::CourierRendererFactory(
+    std::unique_ptr<RendererController> controller)
+    : controller_(std::move(controller)) {}
+
+CourierRendererFactory::~CourierRendererFactory() {}
+
+std::unique_ptr<Renderer> CourierRendererFactory::CreateRenderer(
+    const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
+    const scoped_refptr<base::TaskRunner>& worker_task_runner,
+    AudioRendererSink* audio_renderer_sink,
+    VideoRendererSink* video_renderer_sink,
+    const RequestSurfaceCB& request_surface_cb) {
+  DCHECK(ShouldCreateCourierRenderer());
+  VLOG(1) << "Create Remoting renderer.";

[miu, 2017/05/02 22:40:04]

nit: We could get rid of this VLOG now. Or, if you want to keep it for debugging
purposes, it should probably be in the selector class now (with VLOGging to
indicate when other renderer types are being created).

[tguilbert, 2017/05/03 01:34:25]

Done.

+  return base::MakeUnique<CourierRenderer>(
+      media_task_runner, controller_->GetWeakPtr(), video_renderer_sink);
+}
+
+bool CourierRendererFactory::ShouldCreateCourierRenderer() {
+  return controller_ && controller_->remote_rendering_started();
+}
+
+}  // namespace remoting
+}  // namespace media