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

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

Issue 1821823002: [Media Router] Conditionally enable mDNS on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: UI creation before provider fix Created 4 years, 9 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 8b1c30f30b85836a81d50d3a0a87b8616acb1117..ff2e7dd145745c0837fa754ae2192d3852d1c28a 100644
--- a/chrome/browser/media/android/router/media_router_android.cc
+++ b/chrome/browser/media/android/router/media_router_android.cc
@@ -244,6 +244,15 @@ void MediaRouterAndroid::ClearIssue(const Issue::Id& issue_id) {
NOTIMPLEMENTED();
}
+bool MediaRouterAndroid::is_mdns_discovery_enabled() const {
+ NOTIMPLEMENTED();
mark a. foltz 2016/03/24 00:04:41 I don't think this is necessary, you have an imple
btolsch 2016/03/24 01:44:03 The original design called for this to only be in
+ return true;
+}
+
+void MediaRouterAndroid::EnableMdnsDiscovery() {
+ NOTIMPLEMENTED();
+}
+
void MediaRouterAndroid::DetachRoute(const MediaRoute::Id& route_id) {
JNIEnv* env = base::android::AttachCurrentThread();
ScopedJavaLocalRef<jstring> jroute_id =

Powered by Google App Engine
This is Rietveld 408576698