Remove some unnecessary heap allocations and copies.

chrome/browser/devtools/devtools_ui_bindings.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_DEVTOOLS_DEVTOOLS_UI_BINDINGS_H_
 #define CHROME_BROWSER_DEVTOOLS_DEVTOOLS_UI_BINDINGS_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 55 matching lines @@
   content::WebContents* web_contents() { return web_contents_; }
   Profile* profile() { return profile_; }
   content::DevToolsClientHost* frontend_host() { return frontend_host_.get(); }
 
   // Takes ownership over the |delegate|.
   void SetDelegate(Delegate* delegate);
   void CallClientFunction(const std::string& function_name,
                           const base::Value* arg1,
                           const base::Value* arg2,
                           const base::Value* arg3);
-  void DispatchEventOnFrontend(const std::string& event_type,
-                               const base::Value* event_data);
  private:
   // content::NotificationObserver:
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // content::DevToolsFrontendHostDelegate override:
   virtual void InspectedContentsClosing() OVERRIDE;
   virtual void DispatchOnEmbedder(const std::string& message) OVERRIDE;
 
@@ skipping 33 matching lines @@
   virtual void Subscribe(const std::string& event_type) OVERRIDE;
   virtual void Unsubscribe(const std::string& event_type) OVERRIDE;
 
   void EnableRemoteDeviceCounter(bool enable);
 
   // DevToolsAndroidBridge::DeviceCountListener override:
   virtual void DeviceCountChanged(int count) OVERRIDE;
 
   // Forwards discovered devices to frontend.
   virtual void PopulateRemoteDevices(const std::string& source,
-                                     scoped_ptr<base::ListValue> targets);
+                                     const base::ListValue& targets);
 
   void DocumentOnLoadCompletedInMainFrame();
 
   // DevToolsFileHelper callbacks.
   void FileSavedAs(const std::string& url);
   void CanceledFileSaveAs(const std::string& url);
   void AppendedTo(const std::string& url);
   void FileSystemsLoaded(
       const std::vector<DevToolsFileHelper::FileSystem>& file_systems);
   void FileSystemAdded(const DevToolsFileHelper::FileSystem& file_system);
   void IndexingTotalWorkCalculated(int request_id,
                                    const std::string& file_system_path,
                                    int total_work);
   void IndexingWorked(int request_id,
                       const std::string& file_system_path,
                       int worked);
   void IndexingDone(int request_id, const std::string& file_system_path);
   void SearchCompleted(int request_id,
                        const std::string& file_system_path,
                        const std::vector<std::string>& file_paths);
   typedef base::Callback<void(bool)> InfoBarCallback;
   void ShowDevToolsConfirmInfoBar(const base::string16& message,
                                   const InfoBarCallback& callback);
 
   // Theme and extensions support.
   void UpdateTheme();
   void AddDevToolsExtensionsToClient();
 
+  void DispatchEventOnFrontend(const std::string& event_type,
+                               const base::Value& event_data);
+
   class FrontendWebContentsObserver;
   friend class FrontendWebContentsObserver;
   scoped_ptr<FrontendWebContentsObserver> frontend_contents_observer_;
 
   Profile* profile_;
   content::WebContents* web_contents_;
   scoped_ptr<Delegate> delegate_;
   bool device_listener_enabled_;
   content::NotificationRegistrar registrar_;
   scoped_ptr<content::DevToolsClientHost> frontend_host_;
   scoped_ptr<DevToolsFileHelper> file_helper_;
   scoped_refptr<DevToolsFileSystemIndexer> file_system_indexer_;
   typedef std::map<
       int,
       scoped_refptr<DevToolsFileSystemIndexer::FileSystemIndexingJob> >
       IndexingJobsMap;
   IndexingJobsMap indexing_jobs_;
 
   typedef std::set<std::string> Subscribers;
   Subscribers subscribers_;
   scoped_ptr<DevToolsTargetsUIHandler> remote_targets_handler_;
   scoped_ptr<DevToolsEmbedderMessageDispatcher> embedder_message_dispatcher_;
   GURL url_;
   base::WeakPtrFactory<DevToolsUIBindings> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DevToolsUIBindings);
 };
 
 #endif  // CHROME_BROWSER_DEVTOOLS_DEVTOOLS_UI_BINDINGS_H_