Bug 95582

Summary: [Gtk] Add Battery Status API support
Product: WebKit Reporter: José Dapena Paz <jdapena>
Component: WebKitGTKAssignee: José Dapena Paz <jdapena>
Status: RESOLVED FIXED    
Severity: Normal CC: cgarcia, changseok, gustavo, jdapena, mrobinson, philn, webkit.review.bot, xan.lopez, zan
Priority: P2 Keywords: Gtk, LayoutTestFailure
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 95937, 115628, 115718, 115719, 115720, 125453    
Bug Blocks:    
Attachments:
Description Flags
Patch pnormand: review-, webkit-ews: commit-queue-

Description José Dapena Paz 2012-08-31 09:09:22 PDT
Battery Status API allows web applications to track power status of the device. It allows to behave different depending on that (i.e. avoid expensive calculations or polling if battery is low and discharging).

Specs: http://dvcs.w3.org/hg/dap/raw-file/tip/battery/Overview.html
Comment 1 José Dapena Paz 2012-09-04 10:50:52 PDT
Created attachment 162059 [details]
Patch
Comment 2 WebKit Review Bot 2012-09-04 10:54:49 PDT
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
Comment 3 Early Warning System Bot 2012-09-04 11:13:55 PDT
Comment on attachment 162059 [details]
Patch

Attachment 162059 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/13743569
Comment 4 Early Warning System Bot 2012-09-04 11:17:24 PDT
Comment on attachment 162059 [details]
Patch

Attachment 162059 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/13757285
Comment 5 Build Bot 2012-09-04 11:17:33 PDT
Comment on attachment 162059 [details]
Patch

Attachment 162059 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/13739629
Comment 6 Build Bot 2012-09-04 12:19:08 PDT
Comment on attachment 162059 [details]
Patch

Attachment 162059 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/13744616
Comment 7 Zan Dobersek 2013-01-24 08:55:46 PST
Can we get this work moving again? I can help if help is needed.
Comment 8 Philippe Normand 2013-04-12 12:15:22 PDT
Comment on attachment 162059 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=162059&action=review

> Source/WebCore/ChangeLog:13
> +        Test are disabled, as there is a linking issue related to
> +        WebCoreInternals.

It'd be nice to figure this issue out. Also, not sure we want to add new features in WK1 at this point :/
Comment 9 Zan Dobersek 2013-04-13 10:05:00 PDT
Comment on attachment 162059 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=162059&action=review

The specification is a candidate recommendation so I think it would be great if we can get the implementation up and working.

>> Source/WebCore/ChangeLog:13
>> +        WebCoreInternals.
> 
> It'd be nice to figure this issue out. Also, not sure we want to add new features in WK1 at this point :/

I think this is simply a matter of exporting the undefined symbols, nothing uncommon really.
Comment 10 Zan Dobersek 2013-04-13 10:06:28 PDT
(In reply to comment #9)
> The specification is a candidate recommendation so I think it would be great if we can get the implementation up and working.
> 

Link to the specification:
http://www.w3.org/TR/battery-status/
Comment 11 ChangSeok Oh 2013-08-23 10:50:32 PDT
Oops. I didn't notice this movement supporting BatteryStatus before.. :p
What's going on this? 
I've also worked a patch for this feature. If you guys don't mind, I'd like to post my patch somewhere. :)
Comment 12 Zan Dobersek 2013-08-23 11:09:09 PDT
Shame about the duplicated work.

My set of patches is complete, but still requires reviewing. The patches are spread in the bugs that are blocking this one.
Comment 13 Zan Dobersek 2013-12-11 10:22:22 PST
Aaand it's done.
Comment 14 ChangSeok Oh 2013-12-12 00:06:14 PST
(In reply to comment #13)
> Aaand it's done.

Zan, great works! :)