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

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

Issue 1224743002: DevTools: allow to pass list of remote debugging targets as command line argument (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Renamed SelfAsDeviceProvider to TCPDeviceProvider Created 5 years, 5 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/devtools/device/port_forwarding_browsertest.cc
diff --git a/chrome/browser/devtools/device/port_forwarding_browsertest.cc b/chrome/browser/devtools/device/port_forwarding_browsertest.cc
index 4fb3c62d930a5f232d9671ec0d9f994b5255c79f..e52ae72a89c30acfbaffb6608b0f24d77baf4026 100644
--- a/chrome/browser/devtools/device/port_forwarding_browsertest.cc
+++ b/chrome/browser/devtools/device/port_forwarding_browsertest.cc
@@ -10,7 +10,7 @@
#include "base/strings/string_number_conversions.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/devtools/device/devtools_android_bridge.h"
-#include "chrome/browser/devtools/device/self_device_provider.h"
+#include "chrome/browser/devtools/device/tcp_device_provider.h"
#include "chrome/browser/devtools/remote_debugging_server.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
@@ -81,7 +81,9 @@ IN_PROC_BROWSER_TEST_F(PortForwardingTest,
Profile* profile = browser()->profile();
AndroidDeviceManager::DeviceProviders device_providers;
- device_providers.push_back(new SelfAsDeviceProvider(kDefaultDebuggingPort));
+
+ device_providers.push_back(
+ TCPDeviceProvider::CreateForLocalhost(kDefaultDebuggingPort));
DevToolsAndroidBridge::Factory::GetForProfile(profile)->
set_device_providers_for_test(device_providers);
@@ -149,8 +151,8 @@ IN_PROC_BROWSER_TEST_F(PortForwardingDisconnectTest, DisconnectOnRelease) {
AndroidDeviceManager::DeviceProviders device_providers;
- scoped_refptr<SelfAsDeviceProvider> self_provider(
- new SelfAsDeviceProvider(kAlternativeDebuggingPort));
+ scoped_refptr<TCPDeviceProvider> self_provider(
+ TCPDeviceProvider::CreateForLocalhost(kAlternativeDebuggingPort));
device_providers.push_back(self_provider);
DevToolsAndroidBridge::Factory::GetForProfile(profile)->

Powered by Google App Engine
This is Rietveld 408576698