CmaMediaPipelineClient to watch media pipeline status

chromecast/browser/media/cma_media_pipeline_client.h

+// Copyright 2015 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.
+
+#ifndef CHROMECAST_BROWSER_MEDIA_CMA_MEDIA_PIPELINE_CLIENT_H_
+#define CHROMECAST_BROWSER_MEDIA_CMA_MEDIA_PIPELINE_CLIENT_H_
+
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "chromecast/public/media/media_pipeline_backend.h"
+
+namespace chromecast {
+namespace media {
+
+// Class to provide media backend and watch media pipeline status
+class CmaMediaPipelineClient : public base::RefCounted<CmaMediaPipelineClient> {
+ public:
+  CmaMediaPipelineClient();
+
+  virtual scoped_ptr<MediaPipelineBackend> CreateMediaPipelineBackend(
+      const media::MediaPipelineDeviceParams& params);
+
+  virtual void OnMediaPipelineBackendCreated();
+  virtual void OnMediaPipelineBackendDestroyed();
+
+ protected:
+  virtual ~CmaMediaPipelineClient();
+
+  // TODO(yucliu): Compiler will complain either:
+  //   1. RefCounted::Release access private/protected destructor
+  //   2. Destructor should be private/protected in subclass of RefCounted
+  // Need to figure out why and remove this line.
+  friend class base::RefCounted<CmaMediaPipelineClient>;

[gunsch, 2015/08/31 15:56:52]

did you try adding DISALLOW_COPY_AND_ASSIGN in this header?

[yucliu1, 2015/08/31 17:47:40]

Just reviewed ref_counted.h:
https://code.google.com/p/chromium/codesearch#chromium/src/base/memory/ref_co...,
the friend is required here.

I'll add the DISALLOW here anyway.

+};
+}
+}
+
+#endif