Bug 164213

Summary: Remove Battery Status API from the tree
Product: WebKit Reporter: Brady Eidson <beidson>
Component: WebKit Misc.Assignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, commit-queue, dbates, mcatanzaro, mehmetgelisin, mike, mitz, sam, simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=129040
Attachments:
Description Flags
Patch
none
patch
none
patch none

Brady Eidson
Reported 2016-10-30 20:26:20 PDT
Remove Battery Status API from the tree Because it's a privacy (tracking and fingerprinting) risk (https://blog.lukaszolejnik.com/battery-status-readout-as-a-privacy-risk/) Apple never enabled it on their ports. Mozilla tried to fix it (https://bugzilla.mozilla.org/show_bug.cgi?id=1124127), and then Mozilla decided it wasn't worth keeping around for web content (https://bugzilla.mozilla.org/show_bug.cgi?id=1313580) The Linux port maintainers don't see a reason to keep it around, as discussed in this thread - https://lists.webkit.org/pipermail/webkit-dev/2016-October/028468.html I'll use this bug to remove it in a few days as long as the thread continues to have no objections.
Attachments
Patch (220.26 KB, patch)
2016-10-31 10:56 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
patch (284.72 KB, patch)
2016-10-31 11:15 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
patch (157.29 KB, patch)
2016-10-31 11:22 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2016-10-31 10:56:31 PDT
Created attachment 293436 [details] Patch
Alex Christensen
Comment 2 2016-10-31 11:15:00 PDT
Created attachment 293440 [details] patch
Alex Christensen
Comment 3 2016-10-31 11:22:34 PDT
Created attachment 293442 [details] patch
WebKit Commit Bot
Comment 4 2016-10-31 11:25:00 PDT
Attachment 293442 [details] did not pass style-queue: ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 6 files If any of these errors are false positives, please file a bug against check-webkit-style.
Brady Eidson
Comment 5 2016-10-31 17:02:10 PDT
We should wait a few days to do this to give people a fair chance to voice objections. There's no rush.
Sam Weinig
Comment 6 2016-11-02 14:30:27 PDT
rs=me
Alex Christensen
Comment 7 2016-11-02 14:32:53 PDT
http://trac.webkit.org/changeset/208300
Note You need to log in before you can comment on or make changes to this bug.