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

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: Add const to function for MediaRouterAndroid. 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 0a335654b0f7566eb83ce3132d173a7ea675128f..700cc1652ae2f95904733e76b6e908c43e5491de 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;
@@ -172,6 +177,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

Powered by Google App Engine
This is Rietveld 408576698