Use new APIs in base/values.h: Value::GetAsNumber and DictionaryValue::GetNumber.

chrome/browser/extensions/extension_history_api.cc

 // Copyright (c) 2011 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/extensions/extension_history_api.h"
 
 #include "base/callback.h"
 #include "base/json/json_writer.h"
 #include "base/message_loop.h"
 #include "base/string_number_conversions.h"
@@ skipping 158 matching lines @@
   if (!temp_url.is_valid()) {
     error_ = keys::kInvalidUrlError;
     return false;
   }
   url->Swap(&temp_url);
   return true;
 }
 
 bool HistoryFunction::GetTimeFromValue(Value* value, base::Time* time) {
   double ms_from_epoch = 0.0;
-  if (!value->GetAsDouble(&ms_from_epoch)) {
-    int ms_from_epoch_as_int = 0;
-    if (!value->GetAsInteger(&ms_from_epoch_as_int))
-      return false;
-    ms_from_epoch = static_cast<double>(ms_from_epoch_as_int);
-  }
+  if (!value->GetAsDouble(&ms_from_epoch))
+    return false;
   // The history service has seconds resolution, while javascript Date() has
   // milliseconds resolution.
   double seconds_from_epoch = ms_from_epoch / 1000.0;
   // Time::FromDoubleT converts double time 0 to empty Time object. So we need
   // to do special handling here.
   *time = (seconds_from_epoch == 0) ?
       base::Time::UnixEpoch() : base::Time::FromDoubleT(seconds_from_epoch);
   return true;
 }
 
@@ skipping 185 matching lines @@
       base::Time::Now(),           // To the current time.
       &cancelable_consumer_,
       NewCallback(this, &DeleteAllHistoryFunction::DeleteComplete));
 
   return true;
 }
 
 void DeleteAllHistoryFunction::DeleteComplete() {
   SendAsyncResponse();
 }