725948 - mochitest-browser-chrome: stop special-casing StopIteration exception in browser-test.js

Status: NEW

(Testing :: Mochitest, defect, P4)

Description

[Serge Gautherie (:sgautherie)]

https://tbpl.mozilla.org/?tree=Try&rev=a872216b6fef
*mochitest-browser-chrome: no exception.
*mochitest-plain-5: 2 expected exceptions.

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) browser-test.js: Stop special-casing StopIteration exception

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) parser_web_testrunner.js: Add a comment that StopIteration exception is intended

Better document bug 366936.

NB:
{
766 ERROR TEST-UNEXPECTED-FAIL | /tests/parser/htmlparser/tests/mochitest/test_html5_tree_construction.html | nextTest() catched a StopIteration exception - [object StopIteration]
1510 ERROR TEST-UNEXPECTED-FAIL | /tests/parser/htmlparser/tests/mochitest/test_html5_tree_construction_part2.html | nextTest() catched a StopIteration exception - [object StopIteration]
}
Relying on StopIteration (for these 2 tests) seems odd to me, but I won't mind more than that.

Comment 3

[Serge Gautherie (:sgautherie)]

http://mxr.mozilla.org/comm-central/source/mailnews/test/resources/asyncTestUtils.js
149 // the real driver!
150 function _async_driver() {
...
159     catch (ex) {
160       if (ex != StopIteration && ex != asyncExpectedEarlyAbort) {

Andrew, you may wish to check whether (silently) ignoring StopIteration is still needed/wanted.
May want to document/report/fix...

Comment 4

[Serge Gautherie (:sgautherie)]

Attachment: (Cv1-CC-WIP1) asyncTestUtils.js: check where we stand wrt StopIteration

Current copy+paste'd code logs
{
TEST-UNEXPECTED-FAIL | ../../../resources/asyncTestUtils.js | Generator explosion. ex: [undefined undefined] async stack [undefined undefined] - See following stack:
}
but that was good enough for this run.

***

(In reply to Serge Gautherie (:sgautherie) from comment #3)

> Andrew, you may wish to check whether (silently) ignoring StopIteration is
> still needed/wanted.

http://build.mozillamessaging.com/tinderboxpushlog/?tree=ThunderbirdTry&rev=6ab411ccb17d
*xpcshell: 60+ exceptions
*mozmill virtualenv: (Unrelated, Don't care.)

> May want to document/report/fix...

Are these xpcshell exceptions actually by design or should a bug be filed to improve/fix them?

Comment 5

[Rob Campbell [:rc] (:robcee)]

why do we need this?

Also, I'm not sure sayrer's reviewing code anymore.

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to Rob Campbell [:rc] (robcee) from comment #5)

> why do we need this?

It doesn't seem to be needed (anymore): why keep it?

> Also, I'm not sure sayrer's reviewing code anymore.

Feel free to take over that review or to suggest another reviewer.

Comment 7

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 596004 [details] [diff] [review]
(Av1) browser-test.js: Stop special-casing StopIteration exception

we don't need to change this.

Comment 8

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 596105 [details] [diff] [review]
(Cv1-CC-WIP1) asyncTestUtils.js: check where we stand wrt StopIteration

I think the idea was that StopIteration is an expected outcome.  I expect this code was just a speculative guard, however, since if I had a real reason to worry about it I would have put a comment in.

PS: I realize this is dramatically late feedback; at the time I had been hoping to find time to do a more thorough investigation, that never happened, etc.  I think we're fine either way, but I no longer have review authority on anything comm-central anymore either.

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.