CodeHealth: Remove use of Value::GetAsDouble

This change removes the use of Value::GetAsDouble
and replaces with Value::is_double(), Value::GetDouble(),
Value::GetIfDouble().

extensions/renderer/activity_log_converter_strategy_unittest.cc

Bug: 1187006
Change-Id: Ib80543b7a8081e80c11f0877f3f691ec01341fb1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3009302
Auto-Submit: Maksim Sadym <sadym@chromium.org>
Commit-Queue: Finnur Thorarinsson <finnur@chromium.org>
Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Cr-Commit-Position: refs/heads/master@{#899959}

extensions/renderer/activity_log_converter_strategy_unittest.cc

@@ -56,10 +56,9 @@ class ActivityLogConverterStrategyTest : public testing::Test {
 
   testing::AssertionResult VerifyDouble(v8::Local<v8::Value> v8_value,
                                         double expected) {
-    double out;
     std::unique_ptr<base::Value> value(
         converter_->FromV8Value(v8_value, context()));
-    if (value->is_double() && value->GetAsDouble(&out) && out == expected)
+    if (value->is_double() && value->GetDouble() == expected)
       return testing::AssertionSuccess();
     return testing::AssertionFailure();
   }