Return same promise consistently when using navigator.getBattery()

Source/modules/battery/BatteryManager.h

 // 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.
 
 #ifndef BatteryManager_h
 #define BatteryManager_h
 
 #include "bindings/core/v8/ScriptPromise.h"
-#include "bindings/core/v8/ScriptPromiseResolver.h"
+#include "bindings/core/v8/ScriptPromiseProperty.h"
+#include "core/dom/ActiveDOMObject.h"
 #include "core/dom/ContextLifecycleObserver.h"
 #include "core/frame/PlatformEventController.h"
 #include "modules/EventTargetModules.h"
 #include "platform/heap/Handle.h"
 
 namespace blink {
 
 class BatteryStatus;
 
 class BatteryManager final : public RefCountedGarbageCollectedEventTargetWithInlineData<BatteryManager>, public ActiveDOMObject, public PlatformEventController {
@@ skipping 29 matching lines @@
 
     // ActiveDOMObject implementation.
     virtual void suspend() override;
     virtual void resume() override;
     virtual void stop() override;
     virtual bool hasPendingActivity() const override;
 
     DECLARE_VIRTUAL_TRACE();
 
 private:
-    enum State {
-        NotStarted,
-        Pending,
-        Resolved,
-    };
-
     explicit BatteryManager(ExecutionContext*);
 
-    RefPtrWillBeMember<ScriptPromiseResolver> m_resolver;
+    typedef ScriptPromiseProperty<Member<BatteryManager>, Member<BatteryManager>, RefPtrWillBeMember<DOMException>> BatteryProperty;

[mlamouri (slow - plz ping), 2015/03/07 14:54:42]

nit: you can use C++11 form which I think is more readable (using
BatteryProperty = ScriptPromiseProperty<...>;)

[timvolodine, 2015/03/12 13:53:11]

Done.

+    Member<BatteryProperty> m_batteryProperty;
     Member<BatteryStatus> m_batteryStatus;
-    State m_state;
+    bool m_isResolved;
 };
 
 } // namespace blink
 
 #endif // BatteryManager_h