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

Unified Diff: chrome/browser/devtools/device/devtools_android_bridge.cc

Issue 2284083002: [DevTools] A setting for TCP discovery enablement (Closed)
Patch Set: Comments Created 4 years, 4 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
« no previous file with comments | « no previous file | chrome/browser/devtools/device/devtools_android_bridge_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/devtools/device/devtools_android_bridge.cc
diff --git a/chrome/browser/devtools/device/devtools_android_bridge.cc b/chrome/browser/devtools/device/devtools_android_bridge.cc
index 606e540bb38c0e8dc878bda42e74b3dae7742919..7df783b6630f8b6dd371888d4551b0b33674f018 100644
--- a/chrome/browser/devtools/device/devtools_android_bridge.cc
+++ b/chrome/browser/devtools/device/devtools_android_bridge.cc
@@ -70,6 +70,9 @@ const char kPageReloadCommand[] = "Page.reload";
const char kWebViewSocketPrefix[] = "webview_devtools_remote";
+const char kChromeDiscoveryURL[] = "localhost:9222";
+const char kNodeDiscoveryURL[] = "localhost:9229";
+
bool BrowserIdFromString(const std::string& browser_id_str,
DevToolsAndroidBridge::BrowserId* browser_id) {
size_t colon_pos = browser_id_str.find(':');
@@ -728,9 +731,17 @@ DevToolsAndroidBridge::DevToolsAndroidBridge(
pref_change_registrar_.Add(prefs::kDevToolsDiscoverUsbDevicesEnabled,
base::Bind(&DevToolsAndroidBridge::CreateDeviceProviders,
base::Unretained(this)));
- pref_change_registrar_.Add(prefs::kDevToolsTargetDiscoveryConfig,
+ pref_change_registrar_.Add(prefs::kDevToolsTCPDiscoveryConfig,
+ base::Bind(&DevToolsAndroidBridge::CreateDeviceProviders,
+ base::Unretained(this)));
+ pref_change_registrar_.Add(prefs::kDevToolsDiscoverTCPTargetsEnabled,
base::Bind(&DevToolsAndroidBridge::CreateDeviceProviders,
base::Unretained(this)));
+ base::ListValue* target_discovery = new base::ListValue();
+ target_discovery->AppendString(kChromeDiscoveryURL);
+ target_discovery->AppendString(kNodeDiscoveryURL);
+ profile->GetPrefs()->SetDefaultPrefValue(
+ prefs::kDevToolsTCPDiscoveryConfig, target_discovery);
CreateDeviceProviders();
}
@@ -954,7 +965,9 @@ void DevToolsAndroidBridge::CreateDeviceProviders() {
AndroidDeviceManager::DeviceProviders device_providers;
PrefService* service = profile_->GetPrefs();
const base::ListValue* targets =
- service->GetList(prefs::kDevToolsTargetDiscoveryConfig);
+ service->GetBoolean(prefs::kDevToolsDiscoverTCPTargetsEnabled)
+ ? service->GetList(prefs::kDevToolsTCPDiscoveryConfig)
+ : nullptr;
scoped_refptr<TCPDeviceProvider> provider = CreateTCPDeviceProvider(targets);
if (tcp_provider_callback_)
tcp_provider_callback_.Run(provider);
« no previous file with comments | « no previous file | chrome/browser/devtools/device/devtools_android_bridge_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698