Battery Status API: blink promise implementation and layout tests.

Source/modules/battery/NavigatorBattery.cpp

 // 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 "config.h"
 #include "modules/battery/NavigatorBattery.h"
 
 #include "core/frame/LocalFrame.h"
 #include "modules/battery/BatteryManager.h"
 
 namespace WebCore {
 
 NavigatorBattery::NavigatorBattery()
 {
 }
 
 NavigatorBattery::~NavigatorBattery()
 {
 }
 
-BatteryManager* NavigatorBattery::battery(Navigator& navigator)
+ScriptPromise NavigatorBattery::getBattery(ScriptState* scriptState, Navigator& navigator)
 {
-    return NavigatorBattery::from(navigator).batteryManager(navigator);
+    return NavigatorBattery::from(navigator).getBattery(scriptState);
 }
 
-BatteryManager* NavigatorBattery::batteryManager(Navigator& navigator)
+ScriptPromise NavigatorBattery::getBattery(ScriptState* scriptState)
 {
-    if (!m_batteryManager && navigator.frame())
-        m_batteryManager = BatteryManager::create(navigator.frame()->document());
-    return m_batteryManager.get();
+    if (!m_batteryManager) {
+        m_batteryManager = BatteryManager::create(scriptState->executionContext());
+        // Create a wrapper to expose this object to the V8 GC so that hasPendingActivity takes effect.
+        toV8NoInline(m_batteryManager.get(), scriptState->context()->Global(), scriptState->isolate());

[abarth-chromium, 2014/06/11 19:44:34]

Why is this necessary?  We're already holding m_batteryManager...

[timvolodine, 2014/06/12 16:38:46]

Done.

+    }
+
+    return m_batteryManager->startRequest(scriptState);
 }
 
 const char* NavigatorBattery::supplementName()
 {
     return "NavigatorBattery";
 }
 
 NavigatorBattery& NavigatorBattery::from(Navigator& navigator)
 {
     NavigatorBattery* supplement = static_cast<NavigatorBattery*>(WillBeHeapSupplement<Navigator>::from(navigator, supplementName()));
     if (!supplement) {
         supplement = new NavigatorBattery();
         provideTo(navigator, supplementName(), adoptPtrWillBeNoop(supplement));
     }
     return *supplement;
 }
 
 void NavigatorBattery::trace(Visitor* visitor)
 {
     visitor->trace(m_batteryManager);
     WillBeHeapSupplement<Navigator>::trace(visitor);
 }
 
 } // namespace WebCore