Closed
Bug 778256
Opened 12 years ago
Closed 8 years ago
Geolocation mozSettings tests fail on Android; mozSettings is undefined
Categories
(Core :: DOM: Geolocation, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: mbrubeck, Unassigned)
References
Details
Attachments
(2 obsolete files)
The tests from bug 777594 fail on Android with the following error: 2970 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/tests/mochitest/geolocation/test_mozsettings.html | an unexpected uncaught JS exception reported through window.onerror - TypeError: mozSettings is undefined at http://mochi.test:8888/tests/dom/tests/mochitest/geolocation/test_mozsettings.html:24 2973 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/tests/mochitest/geolocation/test_mozsettingsWatch.html | an unexpected uncaught JS exception reported through window.onerror - TypeError: mozSettings is undefined at http://mochi.test:8888/tests/dom/tests/mochitest/geolocation/test_mozsettingsWatch.html:24 The tests were disabled temporarily in https://hg.mozilla.org/integration/mozilla-inbound/rev/9b056f7ae1cd
Comment 1•11 years ago
|
||
This problem is appearing on other platforms as well. This was exposed on the try server when bug 847279 caused the test_mozsettings.html test case to be run at the beginning of mochitest-4 instead of at the end of mochitest-3. It is easily reproducable on my machine.
Comment 2•11 years ago
|
||
It appears that there is a race condition between the test case using mozSettings and the creation of the navigator.mozSettings node after setting dom.mozSettings.enabled to true. This patch works around the problem by using a zero delay setTimeout() call. Its unclear to me what the correct solution is for this. Is it a bug in mozSettings creation or is this behavior expected?
Comment 3•11 years ago
|
||
Note, any fix to test_mozsettings.html should also probably be applied to test_mozsettingsWatch.html as they use the same code structure.
Comment 5•11 years ago
|
||
Comment on attachment 737965 [details] [diff] [review] Workaround for mozSettings creation race condition. Thanks Reuben! I think it was probably incorrect for me to throw this in this bug at all. The Android problem is probably unrelated to the recent timing changes. Sorry for mucking up the bug history here.
Attachment #737965 -
Attachment is obsolete: true
Updated•11 years ago
|
Attachment #737963 -
Attachment is obsolete: true
Comment 6•11 years ago
|
||
I verified that mozsettings still does not work on Android, even with the async loading changes. Therefore, I opened bug 862666 to deal with these newer issues. Sorry again for mucking up the history here.
Comment 7•11 years ago
|
||
Android still fails after bug 862353. https://tbpl.mozilla.org/?tree=Try&rev=7d3cdee236f3 I think these are unrelated. The dependency for this bug was probably for the issue that was moved out to bug 862666.
No longer depends on: 862353
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•