Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1259)

Unified Diff: third_party/WebKit/Source/modules/battery/BatteryManager.cpp

Issue 1740843002: Revert of Migrates battery_status from content/renderer/ to WebKit/platform/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/battery/BatteryManager.cpp
diff --git a/third_party/WebKit/Source/modules/battery/BatteryManager.cpp b/third_party/WebKit/Source/modules/battery/BatteryManager.cpp
index 3a8a4f0c2f52e65499a29657b4b50fb8479f5812..f7b280b33a0fc2e50d9b5a8342c2bffae33911ef 100644
--- a/third_party/WebKit/Source/modules/battery/BatteryManager.cpp
+++ b/third_party/WebKit/Source/modules/battery/BatteryManager.cpp
@@ -7,6 +7,7 @@
#include "core/dom/Document.h"
#include "core/events/Event.h"
#include "modules/battery/BatteryDispatcher.h"
+#include "modules/battery/BatteryStatus.h"
namespace blink {
@@ -27,6 +28,7 @@
BatteryManager::BatteryManager(ExecutionContext* context)
: ActiveDOMObject(context)
, PlatformEventController(toDocument(context)->page())
+ , m_batteryStatus(BatteryStatus::create())
{
}
@@ -49,30 +51,30 @@
bool BatteryManager::charging()
{
- return m_batteryStatus.charging();
+ return m_batteryStatus->charging();
}
double BatteryManager::chargingTime()
{
- return m_batteryStatus.charging_time();
+ return m_batteryStatus->chargingTime();
}
double BatteryManager::dischargingTime()
{
- return m_batteryStatus.discharging_time();
+ return m_batteryStatus->dischargingTime();
}
double BatteryManager::level()
{
- return m_batteryStatus.level();
+ return m_batteryStatus->level();
}
void BatteryManager::didUpdateData()
{
ASSERT(m_batteryProperty);
- BatteryStatus oldStatus = m_batteryStatus;
- m_batteryStatus = *BatteryDispatcher::instance().latestData();
+ BatteryStatus* oldStatus = m_batteryStatus;
+ m_batteryStatus = BatteryDispatcher::instance().latestData();
if (m_batteryProperty->state() == ScriptPromisePropertyBase::Pending) {
m_batteryProperty->resolve(this);
@@ -84,13 +86,15 @@
if (document->activeDOMObjectsAreSuspended() || document->activeDOMObjectsAreStopped())
return;
- if (m_batteryStatus.charging() != oldStatus.charging())
+ ASSERT(oldStatus);
+
+ if (m_batteryStatus->charging() != oldStatus->charging())
dispatchEvent(Event::create(EventTypeNames::chargingchange));
- if (m_batteryStatus.charging_time() != oldStatus.charging_time())
+ if (m_batteryStatus->chargingTime() != oldStatus->chargingTime())
dispatchEvent(Event::create(EventTypeNames::chargingtimechange));
- if (m_batteryStatus.discharging_time() != oldStatus.discharging_time())
+ if (m_batteryStatus->dischargingTime() != oldStatus->dischargingTime())
dispatchEvent(Event::create(EventTypeNames::dischargingtimechange));
- if (m_batteryStatus.level() != oldStatus.level())
+ if (m_batteryStatus->level() != oldStatus->level())
dispatchEvent(Event::create(EventTypeNames::levelchange));
}
@@ -138,6 +142,7 @@
DEFINE_TRACE(BatteryManager)
{
visitor->trace(m_batteryProperty);
+ visitor->trace(m_batteryStatus);
PlatformEventController::trace(visitor);
RefCountedGarbageCollectedEventTargetWithInlineData<BatteryManager>::trace(visitor);
ActiveDOMObject::trace(visitor);
« no previous file with comments | « third_party/WebKit/Source/modules/battery/BatteryManager.h ('k') | third_party/WebKit/Source/modules/battery/BatteryStatus.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698