Baseline password/login manager landed using the prompt system. We need to add support for using doorhangers next.
Created attachment 571244 [details] [diff] [review]
This patch makes the basic changes to LoginManagerPrompter.js to support using the new NativeWindow.doorhanger API.
The main thing yet to add is the concept of a "timeout" or "persistence" to a doorhanger. Normally, we remove doorhangers on pageload, but with password doorhangers, a page load is expected (to the login OK or invalid page) so we shouldn't dismiss the doorhanger right a way.
Will add that support to Doorhanger next.
Created attachment 574778 [details] [diff] [review]
This patch converts the Fennec LoginManagerPrompter.js from using "notificationbox" API to use "doorhanger" API. I tried to change as little code as possible. We just work to get the NativeWindow object instead of the notifyBox.
I use the new persistence and timeout doorhanger features to mimic the same behavior in Fennec as found on desktop.
(I was having a problem while testing this patch. The doorhanger didn't want to dismiss after tapping a button. Updating my source tree to get the latest patches fixed that issue.)
Comment on attachment 574778 [details] [diff] [review]
We should probably update all the comments that talk about showing notification bars, since we don't do that anymore.
I updated some of the "notification bar" comments to "notification doorhanger", but left the function names as is
Build ID: Mozilla/5.0 (Android; Linux armv7l; rv:11.0a1) Gecko/20111130 Firefox/11.0a1 Fennec/11.0a1
Device: HTC Desire Z
OS: Android 2.3
Door hangers are used for password manager.