Closed Bug 868296 Opened 12 years ago Closed 6 years ago

Intermittent test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionend should open the popup: popupOpen - got false, expected true

Categories

(Toolkit :: Autocomplete, defect, P3)

All
Windows 8
defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: RyanVM, Unassigned)

References

Details

(Keywords: intermittent-failure)

https://tbpl.mozilla.org/php/getParsedLog.php?id=22524358&tree=Mozilla-Inbound WINNT 6.2 mozilla-inbound debug test mochitest-other on 2013-05-02 15:47:08 PDT for push 10c6814d775e slave: t-w864-ix-013 16:00:00 INFO - 17838 INFO TEST-START | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html 16:00:00 INFO - ++DOMWINDOW == 99 (1D41C170) [serial = 3021] [outer = 0534C338] 16:00:00 INFO - ++DOCSHELL 1D4113C0 == 10 [id = 583] 16:00:00 INFO - ++DOMWINDOW == 100 (1D40E8E0) [serial = 3022] [outer = 00000000] 16:00:00 INFO - [Parent 2452] WARNING: NS_ENSURE_TRUE(NS_SUCCEEDED(rv) && subjPrincipal) failed: file e:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/docshell/base/nsDocShell.cpp, line 8281 16:00:00 INFO - ++DOMWINDOW == 101 (1D411698) [serial = 3023] [outer = 1D40E8E0] 16:00:00 INFO - 17839 INFO TEST-INFO | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | must wait for load 16:00:00 INFO - ++DOMWINDOW == 102 (1D40F750) [serial = 3024] [outer = 1D40E8E0] 16:00:01 INFO - 17840 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionstart shouldn't open the popup: value 16:00:01 INFO - 17841 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionstart shouldn't open the popup: searchString 16:00:01 INFO - 17842 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionstart shouldn't open the popup: popupOpen 16:00:01 INFO - 17843 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionupdate shouldn't open the popup: value 16:00:01 INFO - 17844 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionupdate shouldn't open the popup: searchString 16:00:01 INFO - 17845 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionupdate shouldn't open the popup: popupOpen 16:00:01 INFO - 17846 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionend should open the popup: value 16:00:01 INFO - 17847 INFO TEST-PASS | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionend should open the popup: searchString 16:00:01 INFO - 17848 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/toolkit/content/tests/chrome/test_autocomplete_with_composition_on_input.html | Testing on HTML input (asynchronously search), compositionend should open the popup: popupOpen - got false, expected true 16:00:01 INFO - [Parent 2452] WARNING: NS_ENSURE_TRUE(scrollFrame) failed: file e:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/dom/base/nsDOMWindowUtils.cpp, line 1496
Did this only occur while bug 566746 /bug 697377 were in the tree?
Yes, where n=1 failures :)
Assuming this is linked to bug 566746 / bug 697377, markh can you look into this one as well?
Assignee: nobody → mhammond
I believe this is linked to those bugs, and the problem is almost certainly that the panel may open now slightly later than the test expects due to the async nature of the form history. I've a try at https://tbpl.mozilla.org/?tree=Try&rev=09f844ce56a8 with some diagnostics which is trying to prove it is simply a timing issue, but it has failed to hit it as yet - but I'll keep trying :) In the meantime, those other bugs have been re-pushed to inbound and hopefully they will stick... Assuming the above it true, the simplest fix probably involves making nsDoTestsForAutoCompleteWithComposition a little more async than it is now with a waitForCondition() to check the popup state...
(In reply to Mark Hammond [:markh] from comment #4) > I believe this is linked to those bugs, and the problem is almost certainly > that the panel may open now slightly later than the test expects due to the > async nature of the form history. Note to self - whenever you say "almost certainly", you are almost certainly wrong :) FTR, a try run with some debug prints is at https://tbpl.mozilla.org/?tree=Try&rev=c7c701e38f6c. For reasons I didn't get to the bottom of, nsAutoCompleteController::ProcessResult isn't called and thus the popup is not opened - so the problem isn't simply that the popup will appear "soon" I don't intend looking any more at this, so unassigning...
Assignee: mhammond → nobody
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.