hoover up some CreateStringValues

chrome/browser/ui/webui/uber/uber_ui.cc

 // Copyright (c) 2012 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/ui/webui/uber/uber_ui.h"
 
 #include "base/stl_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ skipping 192 matching lines @@
   switch (type) {
     // We listen for notifications that indicate an extension has been loaded
     // (i.e., has been installed and/or enabled) or unloaded (i.e., has been
     // uninstalled and/or disabled). If one of these events has occurred, then
     // we must update the behavior of the History navigation element so that
     // it opens the history extension if one is installed and enabled or
     // opens the default history page if one is uninstalled or disabled.
     case chrome::NOTIFICATION_EXTENSION_LOADED:
     case chrome::NOTIFICATION_EXTENSION_UNLOADED: {
       Profile* profile = Profile::FromWebUI(web_ui());
-      bool overridesHistory = HasExtensionType(profile,
-          chrome::kChromeUIHistoryHost);
-      scoped_ptr<Value> controlsValue(
-          Value::CreateStringValue(chrome::kChromeUIHistoryHost));
-      scoped_ptr<Value> overrideValue(
-          Value::CreateStringValue(overridesHistory ? "yes" : "no"));
+      bool overrides_history =
+          HasExtensionType(profile, chrome::kChromeUIHistoryHost);
       web_ui()->CallJavascriptFunction(
-          "uber_frame.setNavigationOverride", *controlsValue, *overrideValue);
+          "uber_frame.setNavigationOverride",
+          base::StringValue(chrome::kChromeUIHistoryHost),
+          base::StringValue(overrides_history ? "yes" : "no"));
       break;
     }
     default:
       NOTREACHED();
   }
 }