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

Side by Side Diff: chrome/browser/devtools/devtools_targets_ui.cc

Issue 2361613002: DevTools: untangle device discovery request from the devtools android bridge. (Closed)
Patch Set: make device hosts profile-independent, plump browser context for inspect. Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/devtools/devtools_targets_ui.h" 5 #include "chrome/browser/devtools/devtools_targets_ui.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 const char kAdbDeviceIdFormat[] = "device:%s"; 58 const char kAdbDeviceIdFormat[] = "device:%s";
59 59
60 const char kAdbBrowserNameField[] = "adbBrowserName"; 60 const char kAdbBrowserNameField[] = "adbBrowserName";
61 const char kAdbBrowserUserField[] = "adbBrowserUser"; 61 const char kAdbBrowserUserField[] = "adbBrowserUser";
62 const char kAdbBrowserVersionField[] = "adbBrowserVersion"; 62 const char kAdbBrowserVersionField[] = "adbBrowserVersion";
63 const char kAdbBrowserChromeVersionField[] = "adbBrowserChromeVersion"; 63 const char kAdbBrowserChromeVersionField[] = "adbBrowserChromeVersion";
64 const char kAdbPagesList[] = "pages"; 64 const char kAdbPagesList[] = "pages";
65 65
66 const char kAdbScreenWidthField[] = "adbScreenWidth"; 66 const char kAdbScreenWidthField[] = "adbScreenWidth";
67 const char kAdbScreenHeightField[] = "adbScreenHeight"; 67 const char kAdbScreenHeightField[] = "adbScreenHeight";
68 const char kAdbAttachedForeignField[] = "adbAttachedForeign";
69 68
70 const char kPortForwardingPorts[] = "ports"; 69 const char kPortForwardingPorts[] = "ports";
71 const char kPortForwardingBrowserId[] = "browserId"; 70 const char kPortForwardingBrowserId[] = "browserId";
72 71
73 // CancelableTimer ------------------------------------------------------------ 72 // CancelableTimer ------------------------------------------------------------
74 73
75 class CancelableTimer { 74 class CancelableTimer {
76 public: 75 public:
77 CancelableTimer(base::Closure callback, base::TimeDelta delay) 76 CancelableTimer(base::Closure callback, base::TimeDelta delay)
78 : callback_(callback), 77 : callback_(callback),
(...skipping 284 matching lines...) Expand 10 before | Expand all | Expand 10 after
363 kAdbBrowserChromeVersionField, 362 kAdbBrowserChromeVersionField,
364 browser->IsChrome() && !parsed.empty() ? parsed[0] : 0); 363 browser->IsChrome() && !parsed.empty() ? parsed[0] : 0);
365 std::string browser_id = browser->GetId(); 364 std::string browser_id = browser->GetId();
366 browser_data->SetString(kTargetIdField, browser_id); 365 browser_data->SetString(kTargetIdField, browser_id);
367 browser_data->SetString(kTargetSourceField, source_id()); 366 browser_data->SetString(kTargetSourceField, source_id());
368 367
369 base::ListValue* page_list = new base::ListValue(); 368 base::ListValue* page_list = new base::ListValue();
370 remote_browsers_[browser_id] = browser; 369 remote_browsers_[browser_id] = browser;
371 browser_data->Set(kAdbPagesList, page_list); 370 browser_data->Set(kAdbPagesList, page_list);
372 for (const auto& page : browser->pages()) { 371 for (const auto& page : browser->pages()) {
373 scoped_refptr<DevToolsAgentHost> host = 372 scoped_refptr<DevToolsAgentHost> host = page->CreateTarget();
374 android_bridge_->CreatePageTarget(page);
375 std::unique_ptr<base::DictionaryValue> target_data = Serialize(host); 373 std::unique_ptr<base::DictionaryValue> target_data = Serialize(host);
376 target_data->SetBoolean(
377 kAdbAttachedForeignField,
378 host->IsAttached() &&
379 !android_bridge_->HasDevToolsWindow(host->GetId()));
380 // Pass the screen size in the target object to make sure that 374 // Pass the screen size in the target object to make sure that
381 // the caching logic does not prevent the target item from updating 375 // the caching logic does not prevent the target item from updating
382 // when the screen size changes. 376 // when the screen size changes.
383 gfx::Size screen_size = device->screen_size(); 377 gfx::Size screen_size = device->screen_size();
384 target_data->SetInteger(kAdbScreenWidthField, screen_size.width()); 378 target_data->SetInteger(kAdbScreenWidthField, screen_size.width());
385 target_data->SetInteger(kAdbScreenHeightField, screen_size.height()); 379 target_data->SetInteger(kAdbScreenHeightField, screen_size.height());
386 targets_[host->GetId()] = host; 380 targets_[host->GetId()] = host;
387 page_list->Append(std::move(target_data)); 381 page_list->Append(std::move(target_data));
388 } 382 }
389 browser_list->Append(std::move(browser_data)); 383 browser_list->Append(std::move(browser_data));
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 device_status_dict->SetString(kPortForwardingBrowserId, 494 device_status_dict->SetString(kPortForwardingBrowserId,
501 sit->first->GetId()); 495 sit->first->GetId());
502 496
503 std::string device_id = base::StringPrintf( 497 std::string device_id = base::StringPrintf(
504 kAdbDeviceIdFormat, 498 kAdbDeviceIdFormat,
505 sit->first->serial().c_str()); 499 sit->first->serial().c_str());
506 result.Set(device_id, device_status_dict); 500 result.Set(device_id, device_status_dict);
507 } 501 }
508 callback_.Run(result); 502 callback_.Run(result);
509 } 503 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698