Skip to content
Permalink
Browse files

battery_status_service: disable more privacy nightmares

The W3C Battery Status API[1] has quite a laughable statement:

"The information disclosed has minimal impact on privacy or
fingerprinting, and therefore is exposed without permission grants".

Along comes a paper "The leaking battery, A privacy analysis of the
HTML5 Battery Status API."

Clean up after the W3C and disable the battery status updater which
could be used to identity users[2].

[1] http://www.w3.org/TR/battery-status/
[2] https://eprint.iacr.org/2015/616.pdf

References: https://github.com/iridium-browser/iridium-browser/issues/40
  • Loading branch information
jengelh committed Aug 4, 2015
1 parent 31b5311 commit d3d3cb0251bda92a1d1c1cd83e40c9e56427b95a
Showing with 2 additions and 0 deletions.
  1. +2 −0 services/device/battery/battery_status_service.cc
@@ -70,6 +70,7 @@ BatteryStatusService::AddCallback(const BatteryUpdateCallback& callback) {
if (!battery_fetcher_)
battery_fetcher_ = BatteryStatusManager::Create(update_callback_);

#if 0
if (callback_list_.empty()) {
bool success = battery_fetcher_->StartListeningBatteryChange();
// On failure pass the default values back.
@@ -81,6 +82,7 @@ BatteryStatusService::AddCallback(const BatteryUpdateCallback& callback) {
// Send recent status to the new callback if already available.
callback.Run(status_);
}
#endif

return callback_list_.Add(callback);
}

0 comments on commit d3d3cb0

Please sign in to comment.
You can’t perform that action at this time.