Chromium Code Reviews| Index: chrome/browser/media/router/media_router.h |
| diff --git a/chrome/browser/media/router/media_router.h b/chrome/browser/media/router/media_router.h |
| index 3acee0fe8dc2a7839a9b252cc2d3699f45f73340..fe761d99aa8b66a816c5b1cc78567bf4885ca5de 100644 |
| --- a/chrome/browser/media/router/media_router.h |
| +++ b/chrome/browser/media/router/media_router.h |
| @@ -9,6 +9,7 @@ |
| #include <vector> |
| #include "base/callback.h" |
| +#include "chrome/browser/media/router/issue.h" |
| #include "chrome/browser/media/router/media_route.h" |
| #include "chrome/browser/media/router/media_route_id.h" |
| #include "chrome/browser/media/router/media_sink.h" |
| @@ -16,10 +17,11 @@ |
| namespace media_router { |
| +class IssuesObserver; |
| class MediaRoutesObserver; |
| class MediaSinksObserver; |
| -// Type of callback used in |RequestRoute()|. Callback is invoked when the |
| +// Type of callback used in |CreateRoute()|. Callback is invoked when the |
| // route request either succeeded or failed. |
| // The first argument is the route created. If the route request failed, this |
| // will be a nullptr. |
| @@ -38,9 +40,9 @@ class MediaRouter { |
| // Requests a media route from |source| to |sink_id|. |
| // |callback| is invoked with a response indicating success or failure. |
| - virtual void RequestRoute(const MediaSourceId& source, |
| - const MediaSinkId& sink_id, |
| - const MediaRouteResponseCallback& callback) = 0; |
| + virtual void CreateRoute(const MediaSourceId& source, |
| + const MediaSinkId& sink_id, |
| + const MediaRouteResponseCallback& callback) = 0; |
| // Closes the media route specified by |route_id|. |
| virtual void CloseRoute(const MediaRouteId& route_id) = 0; |
| @@ -51,6 +53,9 @@ class MediaRouter { |
| virtual void PostMessage(const MediaRouteId& route_id, |
| const std::string& message) = 0; |
| + // Clears the issue with the id |issue_id|. |
| + virtual void ClearIssue(const Issue::IssueId& issue_id) = 0; |
| + |
| // Receives updates from a MediaRouter instance. |
| class Delegate { |
| public: |
| @@ -61,12 +66,13 @@ class MediaRouter { |
| const std::string& message) = 0; |
| }; |
| - protected: |
| + private: |
| + friend class IssuesObserver; |
| friend class MediaSinksObserver; |
| friend class MediaRoutesObserver; |
| - // The following APIs are called by MediaSinksObserver/MediaRoutesObserver |
| - // and implementations of MediaRouter only. |
| + // The following functions are called by IssuesObserver, MediaSinksObserver, |
| + // and MediaRoutesObserver. |
| // Registers |observer| with this MediaRouter. |observer| specifies a media |
| // source and will receive updates with media sinks that are compatible with |
| @@ -94,6 +100,13 @@ class MediaRouter { |
| // Removes a previously added MediaRoutesObserver. |observer| will stop |
| // receiving further updates. |
| virtual void UnregisterMediaRoutesObserver(MediaRoutesObserver* observer) = 0; |
| + |
| + // Adds the IssuesObserver |observer|. |
| + virtual void AddIssuesObserver(IssuesObserver* observer) = 0; |
| + |
| + // Removes the IssuesObserver |observer|. |
| + virtual void RemoveIssuesObserver(IssuesObserver* observer) = 0; |
| + |
|
Wez
2015/05/20 23:37:47
nit: Unnecessary blank line?
imcheng (use chromium acct)
2015/05/21 19:28:50
Done.
|
| }; |
| } // namespace media_router |