Get rid of some uses of CreateDoubleValue

BUG=160586

Review URL: https://codereview.chromium.org/396493004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283348 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/extensions/api/web_view/web_view_internal_api.cc

@@ -405,7 +405,7 @@ bool WebViewInternalGetZoomFunction::RunAsyncSafe(WebViewGuest* guest) {
   EXTENSION_FUNCTION_VALIDATE(params.get());
 
   double zoom_factor = guest->GetZoom();
-  SetResult(base::Value::CreateDoubleValue(zoom_factor));
+  SetResult(new base::FundamentalValue(zoom_factor));
   SendResponse(true);
   return true;
 }

chrome/browser/prefs/pref_hash_calculator_unittest.cc

@@ -85,7 +85,7 @@ TEST(PrefHashCalculatorTest, CatchHashChanges) {
   scoped_ptr<base::Value> int_value(
       base::Value::CreateIntegerValue(1234567890));
   scoped_ptr<base::Value> double_value(
-      base::Value::CreateDoubleValue(123.0987654321));
+      new base::FundamentalValue(123.0987654321));
   scoped_ptr<base::Value> string_value(base::Value::CreateStringValue(
       "testing with special chars:\n<>{}:^^@#$\\/"));
 

chrome/browser/profiles/off_the_record_profile_impl_unittest.cc

@@ -62,7 +62,7 @@ class TestingProfileWithHostZoomMap : public TestingProfile {
       host_zoom_dictionary->RemoveWithoutPathExpansion(change.host, NULL);
     } else {
       host_zoom_dictionary->SetWithoutPathExpansion(
-          change.host, base::Value::CreateDoubleValue(level));
+          change.host, new base::FundamentalValue(level));
     }
   }
 

chrome/browser/profiles/profile_impl.cc

@@ -1074,12 +1074,10 @@ void ProfileImpl::OnZoomLevelChanged(
   double level = change.zoom_level;
   DictionaryPrefUpdate update(prefs_.get(), prefs::kPerHostZoomLevels);
   base::DictionaryValue* host_zoom_dictionary = update.Get();
-  if (content::ZoomValuesEqual(level, host_zoom_map->GetDefaultZoomLevel())) {
+  if (content::ZoomValuesEqual(level, host_zoom_map->GetDefaultZoomLevel()))
     host_zoom_dictionary->RemoveWithoutPathExpansion(change.host, NULL);
-  } else {
-    host_zoom_dictionary->SetWithoutPathExpansion(
-        change.host, base::Value::CreateDoubleValue(level));
-  }
+  else
+    host_zoom_dictionary->SetDoubleWithoutPathExpansion(change.host, level);
 }
 
 #if defined(ENABLE_SESSION_SERVICE)

chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc

@@ -275,9 +275,9 @@ void DoGetEvents(base::ListValue* results,
         // The property 'eventType' is set in HandleGetEvents as part of the
         // entire result set, so we don't need to include this here in the
         // event.
-        if (data.key() == "eventType")
+        if (data.key() == "eventType") {
           continue;
-        else if (data.key() == "time") {
+        } else if (data.key() == "time") {
           // The property 'time' is also used computationally, but must be
           // converted to JS-style time.
           double time = 0.0;
@@ -285,9 +285,9 @@ void DoGetEvents(base::ListValue* results,
             LOG(ERROR) << "Failed to get 'time' field from event.";
             continue;
           }
-          value = base::Value::CreateDoubleValue(
-              base::Time::FromInternalValue(
-                  static_cast<int64>(time)).ToJsTime());
+          value = new base::FundamentalValue(
+              base::Time::FromInternalValue(static_cast<int64>(time))
+                  .ToJsTime());
         } else {
           // All other values are user-facing, so we create a new value for
           // localized display.

chrome/browser/ui/webui/quota_internals/quota_internals_handler.cc

@@ -34,9 +34,8 @@ void QuotaInternalsHandler::RegisterMessages() {
 }
 
 void QuotaInternalsHandler::ReportAvailableSpace(int64 available_space) {
-  scoped_ptr<base::Value> avail(
-      base::Value::CreateDoubleValue(static_cast<double>(available_space)));
-  SendMessage("AvailableSpaceUpdated", *avail);
+  SendMessage("AvailableSpaceUpdated",
+              base::FundamentalValue(static_cast<double>(available_space)));
 }
 
 void QuotaInternalsHandler::ReportGlobalInfo(const GlobalStorageInfo& data) {