Add Dispatcher class to the Battery Status API module.

Source/modules/battery/BatteryDispatcher.cpp

Index: Source/modules/battery/BatteryDispatcher.cpp
diff --git a/Source/modules/battery/BatteryDispatcher.cpp b/Source/modules/battery/BatteryDispatcher.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..4d24c59cf5f9649e76d4706741fe26f33845c539
--- /dev/null
+++ b/Source/modules/battery/BatteryDispatcher.cpp
@@ -0,0 +1,90 @@
+// 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/BatteryDispatcher.h"
+
+#include "core/events/Event.h"
+#include "modules/battery/BatteryStatus.h"
+#include "public/platform/Platform.h"
+
+namespace WebCore {
+
+BatteryDispatcher& BatteryDispatcher::instance()
+{
+    DEFINE_STATIC_LOCAL(BatteryDispatcher, batteryDispatcher, ());
+    return batteryDispatcher;
+}
+
+BatteryDispatcher::BatteryDispatcher()
+{
+}
+
+BatteryDispatcher::~BatteryDispatcher()
+{
+}
+
+void BatteryDispatcher::addClient(BatteryManager* batteryManager)
+{
+    addController(batteryManager);
+}
+
+void BatteryDispatcher::removeClient(BatteryManager* batteryManager)
+{
+    removeController(batteryManager);
+}
+
+void BatteryDispatcher::updateBatteryStatus(const blink::WebBatteryStatus& batteryStatus)
+{
+    RefPtr<BatteryStatus> status = BatteryStatus::create(batteryStatus.charging, batteryStatus.chargingTime, batteryStatus.dischargingTime, batteryStatus.level);
+
+    if (m_batteryStatus) {
+        if (m_batteryStatus->charging() != status->charging())
+            didChangeBatteryStatus(EventTypeNames::chargingchange, status);
+        else if (status->charging() && m_batteryStatus->chargingTime() != status->chargingTime())

[timvolodine, 2014/04/02 14:24:51]

do you need an 'else if' here? looks like this should be just an 'if'.

[Srini, 2014/04/02 17:25:36]

Yes, I think I over thought this. Ill fix this.

+            didChangeBatteryStatus(EventTypeNames::chargingtimechange, status);
+        else if (!status->charging() && m_batteryStatus->dischargingTime() != status->dischargingTime())
+            didChangeBatteryStatus(EventTypeNames::dischargingtimechange, status);
+
+        if (m_batteryStatus->level() != status->level())
+            didChangeBatteryStatus(EventTypeNames::levelchange, status);
+    } else {
+        // There was no previous state.
+        didChangeBatteryStatus(EventTypeNames::chargingchange, status);
+        didChangeBatteryStatus(EventTypeNames::levelchange, status);
+    }
+
+    m_batteryStatus = status.release();
+}
+
+void BatteryDispatcher::didChangeBatteryStatus(const AtomicString& eventType, PassRefPtr<BatteryStatus> batteryStatus)
+{
+    RefPtr<Event> event = Event::create(eventType);

[sof, 2014/04/07 12:52:44]

Change RefPtr<Event> to the Oilpan transition type RefPtrWillBeRawPtr<Event>

+    RefPtr<BatteryStatus> battery = batteryStatus;
+
+    {
+        TemporaryChange<bool> changeIsDispatching(m_isDispatching, true);
+        // Don't fire listeners removed or added during event dispatch.
+        size_t size = m_controllers.size();
+        for (size_t i = 0; i < size; ++i) {
+            if (m_controllers[i])
+                static_cast<BatteryManager*>(m_controllers[i])->didChangeBatteryStatus(event, battery);
+        }
+    }
+
+    if (m_needsPurge)
+        purgeControllers();
+}
+
+void BatteryDispatcher::startListening()
+{
+    // blink::Platform::current()->setBatteryListener(this);
+}
+
+void BatteryDispatcher::stopListening()
+{
+    // blink::Platform::current()->setBatteryListener(0);
+}
+
+}