 Chromium Code Reviews
 Chromium Code Reviews Issue 251933005:
  [ChromeDriver] Support mobile emulation on desktop Chrome.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 251933005:
  [ChromeDriver] Support mobile emulation on desktop Chrome.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/test/chromedriver/chrome_launcher.cc | 
| diff --git a/chrome/test/chromedriver/chrome_launcher.cc b/chrome/test/chromedriver/chrome_launcher.cc | 
| index ffc24283a8ccb1c37071241c1e607d0b4b081a5f..a4b57951dbb39b4d297218c2b6076e26d92ff959 100644 | 
| --- a/chrome/test/chromedriver/chrome_launcher.cc | 
| +++ b/chrome/test/chromedriver/chrome_launcher.cc | 
| @@ -156,9 +156,16 @@ Status WaitForDevToolsAndCheckVersion( | 
| const NetAddress& address, | 
| URLRequestContextGetter* context_getter, | 
| const SyncWebSocketFactory& socket_factory, | 
| + const Capabilities* capabilities, | 
| scoped_ptr<DevToolsHttpClient>* user_client) { | 
| + scoped_ptr<DeviceMetrics> device_metrics; | 
| + if (capabilities && capabilities->device_metrics) { | 
| + device_metrics.reset(new DeviceMetrics()); | 
| + *device_metrics = *capabilities->device_metrics; | 
| 
samuong
2014/05/16 23:00:35
Change these two lines to:
device_metrics.reset(n
 
sam.rawlins
2014/05/16 23:35:51
Done.
 | 
| + } | 
| + | 
| scoped_ptr<DevToolsHttpClient> client(new DevToolsHttpClient( | 
| - address, context_getter, socket_factory)); | 
| + address, context_getter, socket_factory, device_metrics.Pass())); | 
| base::TimeTicks deadline = | 
| base::TimeTicks::Now() + base::TimeDelta::FromSeconds(60); | 
| Status status = client->Init(deadline - base::TimeTicks::Now()); | 
| @@ -193,7 +200,7 @@ Status LaunchExistingChromeSession( | 
| scoped_ptr<DevToolsHttpClient> devtools_client; | 
| status = WaitForDevToolsAndCheckVersion( | 
| capabilities.debugger_address, context_getter, socket_factory, | 
| - &devtools_client); | 
| + NULL, &devtools_client); | 
| if (status.IsError()) { | 
| return Status(kUnknownError, "cannot connect to chrome at " + | 
| capabilities.debugger_address.ToString(), | 
| @@ -277,7 +284,8 @@ Status LaunchDesktopChrome( | 
| scoped_ptr<DevToolsHttpClient> devtools_client; | 
| status = WaitForDevToolsAndCheckVersion( | 
| - NetAddress(port), context_getter, socket_factory, &devtools_client); | 
| + NetAddress(port), context_getter, socket_factory, &capabilities, | 
| + &devtools_client); | 
| if (status.IsError()) { | 
| int exit_code; | 
| @@ -377,6 +385,7 @@ Status LaunchAndroidChrome( | 
| status = WaitForDevToolsAndCheckVersion(NetAddress(port), | 
| context_getter, | 
| socket_factory, | 
| + &capabilities, | 
| &devtools_client); | 
| if (status.IsError()) { | 
| device->TearDown(); |