[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 #99940|0                           |1
        is obsolete|                            |
  Attachment #99940|review?, commit-queue?      |
               Flag|                            |
 Attachment #129681|                            |review?, commit-queue?
               Flag|                            |




--- Comment #29 from Kihong Kwon <kihong.kwon at samsung.com>  2012-03-01 05:13:15 PST ---
Created an attachment (id=129681)
 --> (https://bugs.webkit.org/attachment.cgi?id=129681&action=review)
Patch.

I have an issue about this patch.
Battery Status API has to be added under the Navigator class, but there is no event handling logic in the navigator.
So I implemented event listener handling logic to the BatteryManager, because I didn't have other choice,
but I think this is not good to add event listener to the Battery Status API.
I think navigator needs event listener handling logic like a DOMWindow.
Leave some comments about this please.
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.