Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4432)

Unified Diff: chrome/browser/media/router/mojo/media_route_controller.h

Issue 2727123002: [Media Router] Custom Controls 1 - Add MediaStatus, MediaRouteController, and mojo interfaces (Closed)
Patch Set: Change comment Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media/router/mojo/media_route_controller.h
diff --git a/chrome/browser/media/router/mojo/media_route_controller.h b/chrome/browser/media/router/mojo/media_route_controller.h
new file mode 100644
index 0000000000000000000000000000000000000000..81812e9918c592b7fd6cb81bf106366b92ac9d04
--- /dev/null
+++ b/chrome/browser/media/router/mojo/media_route_controller.h
@@ -0,0 +1,115 @@
+// 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.
+
+#ifndef CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_
+#define CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_
+
+#include <memory>
+
+#include "base/memory/ref_counted.h"
+#include "base/observer_list.h"
+#include "chrome/browser/media/router/media_route.h"
+#include "chrome/browser/media/router/mojo/media_controller.mojom.h"
+#include "mojo/public/cpp/bindings/binding.h"
+
+namespace media_router {
+
+// A controller for a MediaRoute. Forwards commands for controlling the route to
+// an out-of-process controller. Notifies its observers whenever there is a
+// change in the route's MediaStatus.
+//
+// It is owned by its observers, each of which holds a scoped_refptr to it. All
+// the classes that hold a scoped_refptr must inherit from the Observer class.
+// An observer should be instantiated with a scoped_refptr obtained through
+// MediaRouter::GetRouteController().
+//
+// A MediaRouteController instance is destroyed when all its observers dispose
+// their references to it. When the Mojo connection with the out-of-process
+// controller is terminated or has an error, OnControllerInvalidated() will be
+// called by the MediaRouter or a Mojo error handler to make observers dispose
+// their refptrs.
+class MediaRouteController : public mojom::MediaStatusObserver,
+ public base::RefCounted<MediaRouteController> {
+ public:
+ // Observes MediaRouteController for MediaStatus updates. The ownership of a
+ // MediaRouteController is shared by its observers.
+ class Observer {
+ public:
+ // Adds itself as an observer to |controller|.
+ explicit Observer(scoped_refptr<MediaRouteController> controller);
+
+ // Removes itself as an observer if |controller_| is still valid.
+ virtual ~Observer();
+
+ virtual void OnMediaStatusUpdated(const MediaStatus& status) = 0;
+
+ // Returns a reference to the observed MediaRouteController. The reference
+ // should not be stored by any object that does not subclass ::Observer.
+ scoped_refptr<MediaRouteController> controller() const {
+ return controller_;
+ }
+
+ private:
+ friend class MediaRouteController;
+
+ // Disposes the reference to the controller.
+ void InvalidateController();
+
+ // Called by InvalidateController() after the reference to the controller is
+ // disposed. Overridden by subclasses to do custom cleanup.
+ virtual void OnControllerInvalidated();
+
+ scoped_refptr<MediaRouteController> controller_;
+
+ DISALLOW_COPY_AND_ASSIGN(Observer);
+ };
+
+ // Constructs a MediaRouteController that forwards media commands to
+ // |media_controller|. |media_controller| must be bound to a message pipe.
+ MediaRouteController(const MediaRoute::Id& route_id,
+ mojom::MediaControllerPtr media_controller);
+
+ // Media controller methods for forwarding commands to a
+ // mojom::MediaControllerPtr held in |media_controller_|.
+ void Play();
+ void Pause();
+ void Seek(base::TimeDelta time);
+ void SetMute(bool mute);
+ void SetVolume(float volume);
+
+ // mojom::MediaStatusObserver:
+ // Notifies |observers_| of a status update.
+ void OnMediaStatusUpdated(const MediaStatus& status) override;
+
+ // Called when the connection between |this| and |media_controller_| is no
+ // longer valid. Notifies |observers_| to dispose their references to |this|.
+ // |this| gets destroyed when all the references are disposed.
+ void Invalidate();
+
+ MediaRoute::Id route_id() const { return route_id_; }
+
+ private:
+ friend class base::RefCounted<MediaRouteController>;
+
+ ~MediaRouteController() override;
+
+ void AddObserver(Observer* observer);
+ void RemoveObserver(Observer* observer);
+
+ // The ID of the Media Route that |this| controls.
+ const MediaRoute::Id route_id_;
+
+ // Handle to the mojom::MediaController that receives media commands.
+ mojom::MediaControllerPtr media_controller_;
+
+ // Observers that |this| notifies of status updates. The observers share the
+ // ownership of |this| through scoped_refptr.
+ base::ObserverList<Observer> observers_;
+
+ DISALLOW_COPY_AND_ASSIGN(MediaRouteController);
+};
+
+} // namespace media_router
+
+#endif // CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_

Powered by Google App Engine
This is Rietveld 408576698