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

Unified Diff: chrome/browser/media/router/media_router.h

Issue 1383653002: MediaRouterAction: Only observe Media Routes when there is a local route. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 5 years, 2 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/media_router.h
diff --git a/chrome/browser/media/router/media_router.h b/chrome/browser/media/router/media_router.h
index cbf5404404e5f9e888e622f9984055255be138fb..587032bb864a059996829044e3e22add745eec85 100644
--- a/chrome/browser/media/router/media_router.h
+++ b/chrome/browser/media/router/media_router.h
@@ -20,6 +20,7 @@
namespace media_router {
class IssuesObserver;
+class LocalMediaRoutesObserver;
class MediaRoutesObserver;
class MediaSinksObserver;
class PresentationSessionMessagesObserver;
@@ -115,8 +116,12 @@ class MediaRouter : public KeyedService {
virtual void OnPresentationSessionDetached(
const MediaRoute::Id& route_id) = 0;
+ // Returns whether or not there is currently an active local route.
+ virtual bool HasLocalRoute() const = 0;
+
private:
friend class IssuesObserver;
+ friend class LocalMediaRoutesObserver;
friend class MediaSinksObserver;
friend class MediaRoutesObserver;
friend class PresentationSessionMessagesObserver;
@@ -174,6 +179,15 @@ class MediaRouter : public KeyedService {
// |observer| will stop receiving further updates.
virtual void UnregisterPresentationSessionMessagesObserver(
PresentationSessionMessagesObserver* observer) = 0;
+
+ // Adds the LocalMediaRoutesObserver |observer| to listen for newly created
+ // MediaRoutes.
+ virtual void RegisterLocalMediaRoutesObserver(
+ LocalMediaRoutesObserver* observer) = 0;
+
+ // Removes the LocalMediaRoutesObserver |observer|.
+ virtual void UnregisterLocalMediaRoutesObserver(
+ LocalMediaRoutesObserver* observer) = 0;
};
} // namespace media_router
« no previous file with comments | « chrome/browser/media/router/local_media_routes_observer.cc ('k') | chrome/browser/media/router/media_router.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698