Summary: | [GTK][WK2] Add the WebKitBatteryProvider class | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zan Dobersek <zan> | ||||||||||
Component: | New Bugs | Assignee: | Zan Dobersek <zan> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | cgarcia, commit-queue, gustavo, mrobinson | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | 115718, 115719 | ||||||||||||
Bug Blocks: | 95582, 125453 | ||||||||||||
Attachments: |
|
Description
Zan Dobersek
2013-05-07 03:37:19 PDT
Created attachment 200889 [details]
Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API Perhaps we should share the EFL implementation for the moment? Created attachment 201801 [details]
Patch
Created attachment 216162 [details]
Patch
Rebased patch.
Created attachment 218761 [details]
Patch
Rebased patch.
Comment on attachment 218761 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=218761&action=review > Source/WebKit2/UIProcess/API/gtk/WebKitBatteryProvider.cpp:83 > +void WebKitBatteryProvider::updateBatteryStatus(WebCore::BatteryProviderUPowerStatus status, double timeRemaining, double level) timeRemaining to something like secondsRemaining perhaps? Committed r160445: <http://trac.webkit.org/changeset/160445> |