Switch from a DataPipe per DecoderBuffer to a single one.

media/mojo/services/mojo_renderer_impl.cc

 // Copyright 2014 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/mojo/services/mojo_renderer_impl.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/base/demuxer_stream_provider.h"
 #include "media/mojo/services/mojo_demuxer_stream_impl.h"
 #include "mojo/public/cpp/application/connect.h"
 #include "mojo/public/cpp/bindings/interface_impl.h"
+#include "mojo/public/cpp/system/data_pipe.h"

[xhwang, 2014/12/06 00:03:44]

What is this for?

[DaleCurtis, 2014/12/06 00:42:59]

Originally to fix a compile error... but it seems no more. Removed.

 #include "mojo/public/interfaces/application/service_provider.mojom.h"
 
 namespace media {
 
 MojoRendererImpl::MojoRendererImpl(
     const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
     mojo::ServiceProvider* media_renderer_provider)
     : task_runner_(task_runner),
       weak_factory_(this) {
   DVLOG(1) << __FUNCTION__;
@@ skipping 175 matching lines @@
 
 void MojoRendererImpl::OnInitialized() {
   DVLOG(1) << __FUNCTION__;
   DCHECK(task_runner_->BelongsToCurrentThread());
   DCHECK(!init_cb_.is_null());
 
   base::ResetAndReturn(&init_cb_).Run();
 }
 
 }  // namespace media