Manual devtools fixups for scoped_refptr operator T* removal.

chrome/browser/devtools/devtools_ui_bindings.h

Index: chrome/browser/devtools/devtools_ui_bindings.h
diff --git a/chrome/browser/devtools/devtools_ui_bindings.h b/chrome/browser/devtools/devtools_ui_bindings.h
index 37e775dda0a07a8f7872d5195d8baae4e36b38cc..52eefc36355569214061f76642eb2b86ebdf14f0 100644
--- a/chrome/browser/devtools/devtools_ui_bindings.h
+++ b/chrome/browser/devtools/devtools_ui_bindings.h
@@ -73,7 +73,7 @@ class DevToolsUIBindings : public content::NotificationObserver,
                           const base::Value* arg1,
                           const base::Value* arg2,
                           const base::Value* arg3);
-  void AttachTo(content::DevToolsAgentHost* agent_host);
+  void AttachTo(const scoped_refptr<content::DevToolsAgentHost>& agent_host);

[dgozman, 2014/08/28 09:31:21]

There is a call to this method in chrome/browser/ui/webui/inspect_ui.cc. It will
now create a temporary scoped_refptr. Maybe fix right away?

[dcheng, 2014/08/28 09:45:22]

Done. I'm going to run an automated tool over the codebase after operator T* is
removed to clean up other instances of this as well, but thanks for this catch.

   void Reattach();
   void Detach();
   bool IsAttachedTo(content::DevToolsAgentHost* agent_host);