DevTools: Split RemotePage and RemotePageTarget

chrome/browser/devtools/devtools_targets_ui.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/devtools/devtools_targets_ui.h"
 
 #include "base/memory/weak_ptr.h"
 #include "base/stl_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
@@ skipping 287 matching lines @@
 }
 
 AdbTargetsUIHandler::~AdbTargetsUIHandler() {
   DevToolsAndroidBridge* android_bridge = GetAndroidBridge();
   if (android_bridge)
     android_bridge->RemoveDeviceListListener(this);
 }
 
 static void CallOnTarget(
     const DevToolsTargetsUIHandler::TargetCallback& callback,
-    DevToolsAndroidBridge::RemotePage* page) {
-  scoped_ptr<DevToolsAndroidBridge::RemotePage> my_page(page);
-  callback.Run(my_page ? my_page->GetTarget() : NULL);
+    scoped_refptr<DevToolsAndroidBridge> bridge,
+    scoped_refptr<DevToolsAndroidBridge::RemotePage> page) {
+  callback.Run(page.get() ? bridge->CreatePageTarget(page) : NULL);
 }
 
 void AdbTargetsUIHandler::Open(
     const std::string& browser_id,
     const std::string& url,
     const DevToolsTargetsUIHandler::TargetCallback& callback) {
   RemoteBrowsers::iterator it = remote_browsers_.find(browser_id);
   if (it == remote_browsers_.end())
     return;
 
-  DevToolsAndroidBridge* android_bridge = GetAndroidBridge();
-  if (android_bridge)
-    android_bridge->Open(it->second, url, base::Bind(&CallOnTarget, callback));
+  scoped_refptr<DevToolsAndroidBridge> android_bridge = GetAndroidBridge();

[dgozman, 2014/09/30 09:23:48]

scoped_refptr<DevToolsAndroidBridge> android_bridge(GetAndroidBridge());

[vkuzkokov, 2014/10/01 09:05:17]

Done.

+  if (android_bridge.get()) {
+    android_bridge->Open(it->second, url,
+                         base::Bind(&CallOnTarget, callback, android_bridge));
+  }
 }
 
 scoped_refptr<content::DevToolsAgentHost>
 AdbTargetsUIHandler::GetBrowserAgentHost(
     const std::string& browser_id) {
   RemoteBrowsers::iterator it = remote_browsers_.find(browser_id);
   if (it == remote_browsers_.end())
     return NULL;
 
   DevToolsAndroidBridge* android_bridge = GetAndroidBridge();
@@ skipping 49 matching lines @@
       chrome::VersionInfo version_info;
       base::Version local_version(version_info.Version());
 
       browser_data->SetBoolean(kCompatibleVersion,
           (!remote_version.IsValid()) || (!local_version.IsValid()) ||
           remote_version.components()[0] <= local_version.components()[0]);
 
       base::ListValue* page_list = new base::ListValue();
       remote_browsers_[browser_id] = browser;
             browser_data->Set(kAdbPagesList, page_list);
-      std::vector<DevToolsAndroidBridge::RemotePage*> pages =
-          android_bridge->CreatePages(browser);
-      for (std::vector<DevToolsAndroidBridge::RemotePage*>::iterator it =
+      const DevToolsAndroidBridge::RemotePages& pages = browser->pages();
+      for (DevToolsAndroidBridge::RemotePages::const_iterator it =
           pages.begin(); it != pages.end(); ++it) {
-        DevToolsAndroidBridge::RemotePage* page =  *it;
-        DevToolsTargetImpl* target = page->GetTarget();
+        DevToolsTargetImpl* target = android_bridge->CreatePageTarget(*it);
         base::DictionaryValue* target_data = Serialize(*target);
         target_data->SetBoolean(
             kAdbAttachedForeignField,
             target->IsAttached() &&
                 !android_bridge->HasDevToolsWindow(target->GetId()));
         // Pass the screen size in the target object to make sure that
         // the caching logic does not prevent the target item from updating
         // when the screen size changes.
         gfx::Size screen_size = device->screen_size();
         target_data->SetInteger(kAdbScreenWidthField, screen_size.width());
@@ skipping 117 matching lines @@
           base::StringPrintf("%d", it->first), it->second);
     }
 
     std::string device_id = base::StringPrintf(
         kAdbDeviceIdFormat,
         sit->first.c_str());
     result.Set(device_id, device_status_dict);
   }
   callback_.Run(result);
 }