640870 - Form assistant(?): "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIDOMHTMLInputElement.selectionEnd]" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: chrome://browser/content/forms.js :: _formHelperGetCaretRect"

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

STR
 (1) Debug build of fennec on desktop-linux
 (2) Load http://bugzilla.mozilla.org
 (3) Click "Log in" link at top of page
 (4) Click on "Password" field to focus it

I see this after clicking on the field

************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIDOMHTMLInputElement.selectionEnd]"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: chrome://browser/content/forms.js :: _formHelperGetCaretRect :: line 546"  data: no]
************************************************************

If I try to enter text after that, I see the same exception on every character I type.

Doesn't seem to happen on other input fields on the page.  Clicking on the password field a second time brings up form assistant.

Comment 1

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

I can reproduce this. It might have something to do with the fact that the fields are hidden at first? But then why does the username field work fine?

Comment 2

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

Not blocking since this seems limited in scope and clicking again does get things to work.

Comment 3

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch to prevent the error

The patch does not make the form assistant works for this case but does prevent the error.

In fact the bugzilla code needs an update and should use the HTML5 placeholder instead of doing its own shit in https://bugzilla.mozilla.org/js/global.js

Comment 4

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

http://hg.mozilla.org/mobile-browser/rev/0e6df627a48e

Comment 5

[Cristian Nicolae (:xti)]

Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110905
Firefox/9.0a1 Fennec/9.0a1
Device: Samsung Galaxy S
OS: Android 2.2