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

Side by Side Diff: chrome/browser/media/router/mojo/media_route_controller.h

Issue 2728543009: [Media Router] Custom Controls 2 - add MediaRouter::GetRouteController() (Closed)
Patch Set: Combine CreateMRController and SetMRStatusObserver Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_
6 #define CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_ 6 #define CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
11 #include "base/observer_list.h" 11 #include "base/observer_list.h"
12 #include "chrome/browser/media/router/media_route.h" 12 #include "chrome/browser/media/router/media_route.h"
13 #include "chrome/browser/media/router/mojo/media_controller.mojom.h" 13 #include "chrome/browser/media/router/mojo/media_controller.mojom.h"
14 #include "mojo/public/cpp/bindings/binding.h" 14 #include "mojo/public/cpp/bindings/binding.h"
15 15
16 namespace media_router { 16 namespace media_router {
17 17
18 class MediaRouter;
19
18 // A controller for a MediaRoute. Forwards commands for controlling the route to 20 // A controller for a MediaRoute. Forwards commands for controlling the route to
19 // an out-of-process controller. Notifies its observers whenever there is a 21 // an out-of-process controller. Notifies its observers whenever there is a
20 // change in the route's MediaStatus. 22 // change in the route's MediaStatus.
21 // 23 //
22 // It is owned by its observers, each of which holds a scoped_refptr to it. All 24 // It is owned by its observers, each of which holds a scoped_refptr to it. All
23 // the classes that hold a scoped_refptr must inherit from the Observer class. 25 // the classes that hold a scoped_refptr must inherit from the Observer class.
24 // An observer should be instantiated with a scoped_refptr obtained through 26 // An observer should be instantiated with a scoped_refptr obtained through
25 // MediaRouter::GetRouteController(). 27 // MediaRouter::GetRouteController().
26 // 28 //
27 // A MediaRouteController instance is destroyed when all its observers dispose 29 // A MediaRouteController instance is destroyed when all its observers dispose
28 // their references to it. When the Mojo connection with the out-of-process 30 // their references to it. When the Mojo connection with the out-of-process
29 // controller is terminated or has an error, OnControllerInvalidated() will be 31 // controller is terminated or has an error, Invalidate() will be called by the
30 // called by the MediaRouter or a Mojo error handler to make observers dispose 32 // MediaRouter or OnMojoConnectionError() to make observers dispose their
31 // their refptrs. 33 // refptrs.
32 class MediaRouteController : public mojom::MediaStatusObserver, 34 class MediaRouteController : public mojom::MediaStatusObserver,
33 public base::RefCounted<MediaRouteController> { 35 public base::RefCounted<MediaRouteController> {
34 public: 36 public:
35 // Observes MediaRouteController for MediaStatus updates. The ownership of a 37 // Observes MediaRouteController for MediaStatus updates. The ownership of a
36 // MediaRouteController is shared by its observers. 38 // MediaRouteController is shared by its observers.
37 class Observer { 39 class Observer {
38 public: 40 public:
39 // Adds itself as an observer to |controller|. 41 // Adds itself as an observer to |controller|.
40 explicit Observer(scoped_refptr<MediaRouteController> controller); 42 explicit Observer(scoped_refptr<MediaRouteController> controller);
41 43
(...skipping 17 matching lines...) Expand all
59 // Called by InvalidateController() after the reference to the controller is 61 // Called by InvalidateController() after the reference to the controller is
60 // disposed. Overridden by subclasses to do custom cleanup. 62 // disposed. Overridden by subclasses to do custom cleanup.
61 virtual void OnControllerInvalidated(); 63 virtual void OnControllerInvalidated();
62 64
63 scoped_refptr<MediaRouteController> controller_; 65 scoped_refptr<MediaRouteController> controller_;
64 66
65 DISALLOW_COPY_AND_ASSIGN(Observer); 67 DISALLOW_COPY_AND_ASSIGN(Observer);
66 }; 68 };
67 69
68 // Constructs a MediaRouteController that forwards media commands to 70 // Constructs a MediaRouteController that forwards media commands to
69 // |media_controller|. |media_controller| must be bound to a message pipe. 71 // |mojo_media_controller|. |media_router| will be notified when the
72 // MediaRouteController is destroyed via DetachRouteController().
70 MediaRouteController(const MediaRoute::Id& route_id, 73 MediaRouteController(const MediaRoute::Id& route_id,
71 mojom::MediaControllerPtr media_controller); 74 mojom::MediaControllerPtr mojo_media_controller,
75 MediaRouter* media_router);
72 76
73 // Media controller methods for forwarding commands to a 77 // Media controller methods for forwarding commands to a
74 // mojom::MediaControllerPtr held in |media_controller_|. 78 // mojom::MediaControllerPtr held in |mojo_media_controller_|.
75 void Play(); 79 void Play();
76 void Pause(); 80 void Pause();
77 void Seek(base::TimeDelta time); 81 void Seek(base::TimeDelta time);
78 void SetMute(bool mute); 82 void SetMute(bool mute);
79 void SetVolume(float volume); 83 void SetVolume(float volume);
80 84
81 // mojom::MediaStatusObserver: 85 // mojom::MediaStatusObserver:
82 // Notifies |observers_| of a status update. 86 // Notifies |observers_| of a status update.
83 void OnMediaStatusUpdated(const MediaStatus& status) override; 87 void OnMediaStatusUpdated(const MediaStatus& status) override;
84 88
85 // Called when the connection between |this| and |media_controller_| is no 89 // Notifies |observers_| to dispose their references to the controller. The
86 // longer valid. Notifies |observers_| to dispose their references to |this|. 90 // controller gets destroyed when all the references are disposed.
87 // |this| gets destroyed when all the references are disposed.
88 void Invalidate(); 91 void Invalidate();
89 92
93 // Returns a mojo pointer bound to |this| by |binding_|. This must only be
94 // called at most once in the lifetime of the controller.
95 mojom::MediaStatusObserverPtr BindObserverPtr();
96
90 MediaRoute::Id route_id() const { return route_id_; } 97 MediaRoute::Id route_id() const { return route_id_; }
91 98
92 private: 99 private:
93 friend class base::RefCounted<MediaRouteController>; 100 friend class base::RefCounted<MediaRouteController>;
94 101
95 ~MediaRouteController() override; 102 ~MediaRouteController() override;
96 103
97 void AddObserver(Observer* observer); 104 void AddObserver(Observer* observer);
98 void RemoveObserver(Observer* observer); 105 void RemoveObserver(Observer* observer);
99 106
107 // Called when the connection between |this| and the MediaControllerPtr or
108 // the MediaStatusObserver binding is no longer valid. Notifies
109 // |media_router_| and |observers_| to dispose their references to |this|.
110 void OnMojoConnectionError();
111
100 // The ID of the Media Route that |this| controls. 112 // The ID of the Media Route that |this| controls.
101 const MediaRoute::Id route_id_; 113 const MediaRoute::Id route_id_;
102 114
103 // Handle to the mojom::MediaController that receives media commands. 115 // Handle to the mojom::MediaController that receives media commands.
104 mojom::MediaControllerPtr media_controller_; 116 mojom::MediaControllerPtr mojo_media_controller_;
105 117
106 // Observers that |this| notifies of status updates. The observers share the 118 // |media_router_| will be notified when the controller is destroyed.
107 // ownership of |this| through scoped_refptr. 119 MediaRouter* const media_router_;
120
121 // The binding to observe the out-of-process provider of status updates.
122 mojo::Binding<mojom::MediaStatusObserver> binding_;
123
124 // Observers that are notified of status updates. The observers share the
125 // ownership of the controller through scoped_refptr.
108 base::ObserverList<Observer> observers_; 126 base::ObserverList<Observer> observers_;
109 127
128 // This becomes false when the controller is invalidated.
129 bool is_valid_ = true;
130
110 DISALLOW_COPY_AND_ASSIGN(MediaRouteController); 131 DISALLOW_COPY_AND_ASSIGN(MediaRouteController);
111 }; 132 };
112 133
113 } // namespace media_router 134 } // namespace media_router
114 135
115 #endif // CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_ 136 #endif // CHROME_BROWSER_MEDIA_ROUTER_MOJO_MEDIA_ROUTE_CONTROLLER_H_
OLDNEW
« no previous file with comments | « chrome/browser/media/router/mock_media_router.h ('k') | chrome/browser/media/router/mojo/media_route_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698