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

Unified Diff: chrome/browser/media/android/router/media_router_android.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: Changes per imcheng@'s comments. Created 5 years, 3 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/android/router/media_router_android.h
diff --git a/chrome/browser/media/android/router/media_router_android.h b/chrome/browser/media/android/router/media_router_android.h
index 00f167408ddc132c1af5ea7a7cd0e0c269431278..eb5642ddfa62473196283760172a4513a1524bb1 100644
--- a/chrome/browser/media/android/router/media_router_android.h
+++ b/chrome/browser/media/android/router/media_router_android.h
@@ -51,6 +51,7 @@ class MediaRouterAndroid : public MediaRouter {
void AddIssue(const Issue& issue) override;
void ClearIssue(const Issue::Id& issue_id) override;
void OnPresentationSessionDetached(const MediaRoute::Id& route_id) override;
+ bool GetHasLocalRoute() override;
imcheng 2015/10/02 17:21:11 naming: Why not just HasLocalRoute() ? Also, let's
apacible 2015/10/02 18:48:20 Done.
// The methods called by the Java counterpart.
@@ -101,6 +102,10 @@ class MediaRouterAndroid : public MediaRouter {
PresentationSessionMessagesObserver* observer) override;
void UnregisterPresentationSessionMessagesObserver(
PresentationSessionMessagesObserver* observer) override;
+ void RegisterLocalMediaRoutesObserver(
+ LocalMediaRoutesObserver* observer) override;
+ void UnregisterLocalMediaRoutesObserver(
+ LocalMediaRoutesObserver* observer) override;
base::android::ScopedJavaGlobalRef<jobject> java_media_router_;

Powered by Google App Engine
This is Rietveld 408576698