514196 - implement nsIPromptService2

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Fabrice Desré]

nsIPromptService2 is used by the password manager for instance, and must be implemented.

Relevant IDLs : 
http://mxr.mozilla.org/mozilla-central/source/embedding/components/windowwatcher/public/nsIPromptService2.idl

http://mxr.mozilla.org/mozilla-central/source/netwerk/base/public/nsIAuthPrompt2.idl

Comment 1

[Fabrice Desré]

Attachment: WIP

This patch implements promptAuth().

I had to do some modifications in promptPassword.xul to get messages to display correctly. If this is ok I'll make the same modifications in the other prompts dialogs.

Comment 2

[Fabrice Desré]

Attachment: website authentication dialog

Comment 3

[Fabrice Desré]

Attachment: proxy authentication dialog with checkbox

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 398405 [details] [diff] [review]
WIP

But I'd like to investigate the scrollbox issue a bit more.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed with tweaks for handling the scrollbox:
https://hg.mozilla.org/mobile-browser/rev/9f871d520f2a

(added PromptService.sizeScrollableMsg)

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

how can I verify this bug?

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

verified with slow script dialog on digg.com with 1.9.2 nightly build from 20090921 on my n810