Closed
Bug 902168
Opened 11 years ago
Closed 11 years ago
System JS : ERROR chrome://browser/content/forms.js:721 TypeError: Argument 1 of Range.setEnd is not an object. in green B2G mochitest-3 runs
Categories
(Firefox OS Graveyard :: Gaia::Keyboard, defect)
Tracking
(firefox24 unaffected, firefox25 unaffected, firefox26 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox24 | --- | unaffected |
firefox25 | --- | unaffected |
firefox26 | --- | fixed |
People
(Reporter: RyanVM, Assigned: xyuan)
References
Details
Attachments
(1 file)
(deleted),
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
Yuan, it appears that you are the primary maintainer of forms.js. Can you please investigate this? https://tbpl.mozilla.org/php/getParsedLog.php?id=26220049&tree=Fx-Team b2g_emulator_vm fx-team opt test mochitest-3 on 2013-08-06 09:56:04 PDT for push ec45d9c75316 slave: tst-linux64-ec2-398 10:28:07 INFO - 38167 INFO TEST-START | /tests/docshell/test/navigation/test_bug430624.html 10:28:07 INFO - 38168 INFO TEST-INFO | /tests/docshell/test/navigation/test_bug430624.html | must wait for focus 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:07 INFO - 38169 INFO TEST-PASS | /tests/docshell/test/navigation/test_bug430624.html | Check we're contentEditable after reload 10:28:07 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:07 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:23 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:23 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:23 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:23 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:23 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:23 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:23 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:23 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:23 INFO - System JS : ERROR chrome://browser/content/forms.js:721 10:28:23 INFO - TypeError: Argument 1 of Range.setEnd is not an object. 10:28:23 INFO - 38170 INFO TEST-END | /tests/docshell/test/navigation/test_bug430624.html | finished in 2307ms
Assignee | ||
Comment 2•11 years ago
|
||
The b2g virtual keyboard doesn't support editing the entire document(see bug 811177). If an editable document is focused, window.getSelection returns null and forms.js throws error. The mochitest file docshell/test/navigation/test_bug430624.html makes the entire document of an iframe editable with the following code: <body contenteditable>contentEditable</body> When the editable <body> is focused, we get error from forms.js. To remove the errors, forms.js should ignore editable document until bug 811177 is fixed.
Assignee | ||
Comment 3•11 years ago
|
||
Ignore the focus event from an editable document. Check window.getSelection() and make sure it is not null before passing to getContentEditableSelectionStart() to avoid error.
Attachment #790240 -
Flags: review?(fabrice)
Updated•11 years ago
|
Attachment #790240 -
Flags: review?(fabrice) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Reporter | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/e81b4edd39c3
Keywords: checkin-needed
Reporter | ||
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e81b4edd39c3
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•