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

Unified Diff: chrome/browser/media/android/router/media_router_android.cc

Issue 1376703002: Presentation API: reject screen availability observing on Android low ram devices. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix and add tests 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/android/router/media_router_android.cc
diff --git a/chrome/browser/media/android/router/media_router_android.cc b/chrome/browser/media/android/router/media_router_android.cc
index 758d6251f16824bc1e50bdaf69eead384f5daecf..dd7a52721e3a663769d52a43cdcf974fa2c27593 100644
--- a/chrome/browser/media/android/router/media_router_android.cc
+++ b/chrome/browser/media/android/router/media_router_android.cc
@@ -188,7 +188,7 @@ void MediaRouterAndroid::OnPresentationSessionDetached(
CloseRoute(route_id);
}
-void MediaRouterAndroid::RegisterMediaSinksObserver(
+bool MediaRouterAndroid::RegisterMediaSinksObserver(
MediaSinksObserver* observer) {
const std::string& source_id = observer->source().id();
base::ObserverList<MediaSinksObserver>* observer_list =
@@ -204,7 +204,7 @@ void MediaRouterAndroid::RegisterMediaSinksObserver(
JNIEnv* env = base::android::AttachCurrentThread();
ScopedJavaLocalRef<jstring> jsource_id =
base::android::ConvertUTF8ToJavaString(env, source_id);
- Java_ChromeMediaRouter_startObservingMediaSinks(
+ return Java_ChromeMediaRouter_startObservingMediaSinks(
env, java_media_router_.obj(), jsource_id.obj());
}
« no previous file with comments | « chrome/browser/media/android/router/media_router_android.h ('k') | chrome/browser/media/router/media_router.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698