[DevTools] Introduce a target discovery preference

chrome/browser/devtools/device/devtools_android_bridge.cc

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 43eb41908fcebf091a852a7ef52bad598ba915ec..04452b20a62537b0350f179c958b294a22f61cd2 100644
--- a/chrome/browser/devtools/device/devtools_android_bridge.cc
+++ b/chrome/browser/devtools/device/devtools_android_bridge.cc
@@ -728,6 +728,9 @@ DevToolsAndroidBridge::DevToolsAndroidBridge(
   pref_change_registrar_.Add(prefs::kDevToolsDiscoverUsbDevicesEnabled,
       base::Bind(&DevToolsAndroidBridge::CreateDeviceProviders,
                  base::Unretained(this)));
+  pref_change_registrar_.Add(prefs::kDevToolsTargetDiscoveryConfig,
+      base::Bind(&DevToolsAndroidBridge::CreateDeviceProviders,
+                 base::Unretained(this)));
   CreateDeviceProviders();
 }
 
@@ -903,15 +906,37 @@ void DevToolsAndroidBridge::ScheduleTaskDefault(const base::Closure& task) {
       base::TimeDelta::FromMilliseconds(kAdbPollingIntervalMs));
 }
 
-static scoped_refptr<TCPDeviceProvider> CreateTCPDeviceProvider() {
+static std::set<net::HostPortPair> ParseTargetDiscoveryPreferenceValue(
+    const base::ListValue* preferenceValue) {
+  std::set<net::HostPortPair> targets;
+  if (!preferenceValue || preferenceValue->empty())
+    return targets;
+  std::string address;
+  for (size_t i = 0; i < preferenceValue->GetSize(); i++) {
+    if (!preferenceValue->GetString(i, &address))
+      continue;
+    net::HostPortPair target = net::HostPortPair::FromString(address);
+    if (target.IsEmpty()) {
+      LOG(WARNING) << "Invalid target: " << address;
+      continue;
+    }
+    targets.insert(target);
+  }
+  return targets;
+}
+
+static scoped_refptr<TCPDeviceProvider> CreateTCPDeviceProvider(
+    const base::ListValue* targetDiscoveryConfig) {
   base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
-  if (!command_line->HasSwitch(switches::kRemoteDebuggingTargets))
+  std::set<net::HostPortPair> targets =
+      ParseTargetDiscoveryPreferenceValue(targetDiscoveryConfig);
+  if (targets.empty() &&
+      !command_line->HasSwitch(switches::kRemoteDebuggingTargets))
     return nullptr;
   std::string value =
       command_line->GetSwitchValueASCII(switches::kRemoteDebuggingTargets);
   std::vector<std::string> addresses = base::SplitString(
       value, ",", base::TRIM_WHITESPACE, base::SPLIT_WANT_NONEMPTY);
-  std::set<net::HostPortPair> targets;
   for (const std::string& address : addresses) {
     net::HostPortPair target = net::HostPortPair::FromString(address);
     if (target.IsEmpty()) {
@@ -927,8 +952,14 @@ static scoped_refptr<TCPDeviceProvider> CreateTCPDeviceProvider() {
 
 void DevToolsAndroidBridge::CreateDeviceProviders() {
   AndroidDeviceManager::DeviceProviders device_providers;
+  PrefService* service = profile_->GetPrefs();
+  const base::ListValue* targets =
+      service->GetList(prefs::kDevToolsTargetDiscoveryConfig);
+  scoped_refptr<TCPDeviceProvider> provider = CreateTCPDeviceProvider(targets);
+  if (tcp_provider_callback_)
+    tcp_provider_callback_.Run(provider);
 
-  if (scoped_refptr<TCPDeviceProvider> provider = CreateTCPDeviceProvider())
+  if (provider)
     device_providers.push_back(provider);
 
 #if defined(ENABLE_SERVICE_DISCOVERY)
@@ -937,7 +968,6 @@ void DevToolsAndroidBridge::CreateDeviceProviders() {
 
   device_providers.push_back(new AdbDeviceProvider());
 
-  PrefService* service = profile_->GetPrefs();
   const PrefService::Preference* pref =
       service->FindPreference(prefs::kDevToolsDiscoverUsbDevicesEnabled);
   const base::Value* pref_value = pref->GetValue();
@@ -953,3 +983,9 @@ void DevToolsAndroidBridge::CreateDeviceProviders() {
     StartDeviceListPolling();
   }
 }
+
+void DevToolsAndroidBridge::set_tcp_provider_callback_for_test(
+    TCPProviderCallback callback) {
+  tcp_provider_callback_ = callback;
+  CreateDeviceProviders();
+}