Chromecast: Add a way to notify video plane about resolution changes

chromecast/media/cma/backend/video_plane.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 "chromecast/media/cma/backend/video_plane.h"
 
+#include "base/logging.h"
+
 namespace chromecast {
 namespace media {
 
 VideoPlane::VideoPlane() {
 }
 
 VideoPlane::~VideoPlane() {
 }
 
+class VideoPlaneRegistry {
+ public:
+  static VideoPlaneRegistry* GetInstance() {
+    return Singleton<VideoPlaneRegistry>::get();
+  }
+
+  VideoPlane* GetVideoPlane(int plane_id);
+
+ private:
+  friend struct DefaultSingletonTraits<VideoPlaneRegistry>;
+  friend class Singleton<VideoPlaneRegistry>;
+
+  VideoPlaneRegistry();
+  virtual ~VideoPlaneRegistry();
+
+  scoped_ptr<VideoPlane> video_plane_;
+
+  DISALLOW_COPY_AND_ASSIGN(VideoPlaneRegistry);
+};
+
+VideoPlaneRegistry::VideoPlaneRegistry() :
+  video_plane_(VideoPlaneFactory::CreatePlatformVideoPlane()) {
+}
+
+VideoPlaneRegistry::~VideoPlaneRegistry() {
+}
+
+VideoPlane* VideoPlaneRegistry::GetVideoPlane(int plane_id) {
+  // Only one video plane is supported at the moment
+  DCHECK_EQ(plane_id, 0);
+  return video_plane_.get();
+}
+
+VideoPlane* GetVideoPlane() {
+  // Only one video plane is supported at the moment

[gunsch, 2014/12/17 22:45:51]

if you're defining this entire class inline, why not just have GetVideoPlane()
take no args?

[servolk, 2014/12/18 00:17:37]

Done.

+  return VideoPlaneRegistry::GetInstance()->GetVideoPlane(0);
+}
+
 }  // namespace media
 }  // namespace chromecast