Remove some unnecessary heap allocations and copies.

chrome/browser/devtools/devtools_ui_bindings.cc

 // 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.
 
 #include "chrome/browser/devtools/devtools_ui_bindings.h"
 
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 369 matching lines @@
     LOG(ERROR) << "Invalid message was sent to embedder: " << message;
     return;
   }
 
   int id = 0;
   dict->GetInteger(kFrontendHostId, &id);
 
   std::string error;
   embedder_message_dispatcher_->Dispatch(method, params, &error);
   if (id) {
-    scoped_ptr<base::Value> id_value(base::Value::CreateIntegerValue(id));
-    scoped_ptr<base::Value> error_value(base::Value::CreateStringValue(error));
+    base::FundamentalValue id_value(id);
+    base::StringValue error_value(error);
     CallClientFunction("InspectorFrontendAPI.embedderMessageAck",
-                       id_value.get(), error_value.get(), NULL);
+                       &id_value, &error_value, NULL);
   }
 }
 
 void DevToolsUIBindings::ActivateWindow() {
   delegate_->ActivateWindow();
 }
 
 void DevToolsUIBindings::CloseWindow() {
   delegate_->CloseWindow();
 }
@@ skipping 212 matching lines @@
 
   DCHECK(device_listener_enabled_ != enable);
   device_listener_enabled_ = enable;
   if (enable)
     adb_bridge->AddDeviceCountListener(this);
   else
     adb_bridge->RemoveDeviceCountListener(this);
 }
 
 void DevToolsUIBindings::DeviceCountChanged(int count) {
-  base::FundamentalValue value(count);
-  DispatchEventOnFrontend(kDeviceCountChanged, &value);
+  DispatchEventOnFrontend(kDeviceCountChanged, base::FundamentalValue(count));
 }
 
 void DevToolsUIBindings::PopulateRemoteDevices(
     const std::string& source,
-    scoped_ptr<base::ListValue> targets) {
-  DispatchEventOnFrontend(kDevicesChanged, targets.get());
+    const base::ListValue& targets) {
+  DispatchEventOnFrontend(kDevicesChanged, targets);
 }
 
 void DevToolsUIBindings::FileSavedAs(const std::string& url) {
   base::StringValue url_value(url);
   CallClientFunction("InspectorFrontendAPI.savedURL", &url_value, NULL, NULL);
 }
 
 void DevToolsUIBindings::CanceledFileSaveAs(const std::string& url) {
   base::StringValue url_value(url);
   CallClientFunction("InspectorFrontendAPI.canceledSaveURL",
@@ skipping 145 matching lines @@
       }
     }
   }
   base::string16 javascript =
       base::UTF8ToUTF16(function_name + "(" + params + ");");
   web_contents_->GetMainFrame()->ExecuteJavaScript(javascript);
 }
 
 void DevToolsUIBindings::DispatchEventOnFrontend(
     const std::string& event_type,
-    const base::Value* event_data) {
+    const base::Value& event_data) {
   if (subscribers_.find(event_type) == subscribers_.end())
     return;
-  base::StringValue event_type_value = base::StringValue(event_type);
+  base::StringValue event_type_value(event_type);
   CallClientFunction("InspectorFrontendAPI.dispatchEventToListeners",
                      &event_type_value,
-                     event_data,
+                     &event_data,
                      NULL);
 }
 
 void DevToolsUIBindings::DocumentOnLoadCompletedInMainFrame() {
   // Call delegate first - it seeds importants bit of information.
   delegate_->OnLoadCompleted();
 
   UpdateTheme();
   AddDevToolsExtensionsToClient();
 }