Battery-API-return-nothing.patch 2.2 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566
  1. From: csagan5 <32685696+csagan5@users.noreply.github.com>
  2. Date: Thu, 22 Mar 2018 22:11:57 +0100
  3. Subject: Battery API: return nothing
  4. Include @thestinger's fix for correct charging/unknown values
  5. ---
  6. .../modules/battery/battery_manager.cc | 26 +++----------------
  7. 1 file changed, 4 insertions(+), 22 deletions(-)
  8. diff --git a/third_party/blink/renderer/modules/battery/battery_manager.cc b/third_party/blink/renderer/modules/battery/battery_manager.cc
  9. --- a/third_party/blink/renderer/modules/battery/battery_manager.cc
  10. +++ b/third_party/blink/renderer/modules/battery/battery_manager.cc
  11. @@ -70,46 +70,28 @@ ScriptPromise BatteryManager::StartRequest(ScriptState* script_state) {
  12. }
  13. bool BatteryManager::charging() {
  14. - return battery_status_.Charging();
  15. + return true;
  16. }
  17. double BatteryManager::chargingTime() {
  18. - return battery_status_.charging_time().InSecondsF();
  19. + return 0.0;
  20. }
  21. double BatteryManager::dischargingTime() {
  22. - return battery_status_.discharging_time().InSecondsF();
  23. + return std::numeric_limits<double>::infinity();
  24. }
  25. double BatteryManager::level() {
  26. - return battery_status_.Level();
  27. + return 1.0;
  28. }
  29. void BatteryManager::DidUpdateData() {
  30. DCHECK(battery_property_);
  31. - BatteryStatus old_status = battery_status_;
  32. - battery_status_ = *battery_dispatcher_->LatestData();
  33. -
  34. if (battery_property_->GetState() == BatteryProperty::kPending) {
  35. battery_property_->Resolve(this);
  36. return;
  37. }
  38. -
  39. - DCHECK(GetExecutionContext());
  40. - if (GetExecutionContext()->IsContextPaused() ||
  41. - GetExecutionContext()->IsContextDestroyed()) {
  42. - return;
  43. - }
  44. -
  45. - if (battery_status_.Charging() != old_status.Charging())
  46. - DispatchEvent(*Event::Create(event_type_names::kChargingchange));
  47. - if (battery_status_.charging_time() != old_status.charging_time())
  48. - DispatchEvent(*Event::Create(event_type_names::kChargingtimechange));
  49. - if (battery_status_.discharging_time() != old_status.discharging_time())
  50. - DispatchEvent(*Event::Create(event_type_names::kDischargingtimechange));
  51. - if (battery_status_.Level() != old_status.Level())
  52. - DispatchEvent(*Event::Create(event_type_names::kLevelchange));
  53. }
  54. void BatteryManager::RegisterWithDispatcher() {
  55. --
  56. 2.20.1