887669 - [Test][System] Unit test for Permission Manager

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[(inactive after 6/18) Alive Kuo [:alive]]

Note: there's two open bugs to change permission into mozbrowser API:
Bug 853711 - [Permission] Gaia fix of bug 852013: Move permission dialog into appWindow and bind to BrowserFrame
Bug 852013 - Send PermissionRequestEvent from mozbrowser iframe, instead of mozChromeEvent (edit)

But they are not under any plan now.

Comment 1

[[e-mail]]

would like to change the format from 

var PermissionManager = (function() { }()); 

to test friendly literal object format

var PermissionManager = {}; PermissionManager.init()

Comment 2

[(inactive after 6/18) Alive Kuo [:alive]]

Finally I prefer to have permission dialog to be app-specific(Multi-instances). But in this stage, turn module pattern into literal object is acceptable. Thanks!

Comment 3

[[e-mail]]

WIP: have migrated to literal object format and works well. Starting write test cases

Comment 4

[[e-mail]]

Attachment: pull request redirect to github

test ChromeEvents and first-level handlers

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 800028 [details]
pull request redirect to github

r=me if you move all PermissionManager reference to this or self.

Comment 6

[[e-mail]]

merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/cf6272fc46bfe4aa636725869c2a5685a4c33ea3

thanks!