Last Comment Bug 725862 - Allow test_websocket.html to be run in a loop to track down oranges
: Allow test_websocket.html to be run in a loop to track down oranges
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Networking: WebSockets (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla13
Assigned To: Jason Duell [:jduell] (needinfo me)
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-09 15:56 PST by Jason Duell [:jduell] (needinfo me)
Modified: 2012-02-14 02:27 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
v1 (5.09 KB, patch)
2012-02-09 15:56 PST, Jason Duell [:jduell] (needinfo me)
mcmanus: review+
Details | Diff | Splinter Review

Description Jason Duell [:jduell] (needinfo me) 2012-02-09 15:56:48 PST
Created attachment 595894 [details] [diff] [review]
v1

Took a bit more tweaking than I expected, but simple enough.
Comment 1 Patrick McManus [:mcmanus] 2012-02-10 05:19:32 PST
can we try just running this a whole bunch of times on try to get the same info, without checking anything in?
Comment 2 Jason Duell [:jduell] (needinfo me) 2012-02-10 10:24:31 PST
Sure we can.  But it's very likely that we'll want to run lots of iterations while debugging or on try again someday, so I built in the infrastructure to make it something you can turn on just by changing a variable from "testsuite_iteration = 1" (which will always be the case on m-c) to whatever you want for a try run.  The patch touches enough stuff that I wouldn't count on being able to apply it at random times in the future, so it seems better to me to bake it in.
Comment 3 Patrick McManus [:mcmanus] 2012-02-10 10:28:13 PST
Comment on attachment 595894 [details] [diff] [review]
v1

Review of attachment 595894 [details] [diff] [review]:
-----------------------------------------------------------------

I'm not going to stand in the way of orange progress :)
Comment 4 Jason Duell [:jduell] (needinfo me) 2012-02-13 11:34:15 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/da94847d66d4
Comment 5 Marco Bonardo [::mak] 2012-02-14 02:27:15 PST
https://hg.mozilla.org/mozilla-central/rev/da94847d66d4

Note You need to log in before you can comment on or make changes to this bug.