[Webkit-unassigned] [Bug 62698] Support for Battery Status API

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=62698


Kihong Kwon <kihong.kwon at samsung.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #129806|0                           |1
        is obsolete|                            |
 Attachment #129806|review?                     |
               Flag|                            |
 Attachment #129837|                            |review?, commit-queue?
               Flag|                            |




--- Comment #42 from Kihong Kwon <kihong.kwon at samsung.com>  2012-03-02 00:24:46 PST ---
Created an attachment (id=129837)
 --> (https://bugs.webkit.org/attachment.cgi?id=129837&action=review)
Patch.

I would need your comments for the issue mentioned below.

Battery Status API needs to be added under the Navigator class, but there is not any event handling logic there.
Therefore, because I have no other choice, I have implemented the necessary event handling logic in the BatteryManager.
I, however, think it may not look good to add the event listener in the Batter Status API itself.

IMHO, the Navigator class would need some sort of event listener handling logic similar to DOMWindow.
I would like to have your comments how you think this.

Thank you.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.