469444 - runtests.py.in : set an explicit default |leakThreshold = 0| for a11y

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Serge Gautherie (:sgautherie)]

Recently, bug 465755 happened;
now, while working on bug 468023, I noticed/wondered about this bug.

I'm guessing that a11y needs:
*__init__(), to support |--leak-threshold|.
*evaluateCommand(), to turn the box Orange (or whatever that function does).

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Explicitly set default leakThreshold to 0 in runtests.py.in [Checkin: Comment 2]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20081211 Minefield/3.2a1pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/0d2bceefc012)

Implicit to Explicit, for now;
will also allow to set a separate threshold if needed.

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 352847 [details] [diff] [review]
(Av1) Explicitly set default leakThreshold to 0 in runtests.py.in
[Checkin: Comment 2]

http://hg.mozilla.org/mozilla-central/rev/5ab9ab1f0f03
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/bcc21e3fb68d

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #0)
> I'm guessing that a11y needs:
> *__init__(), to support |--leak-threshold|.
> *evaluateCommand(), to turn the box Orange (or whatever that function does).

It doesn't, because it inherits them from |class MozillaMochichrome| :->
(Forgive me,: I'm "learning" python and this file at the same time...)

Let me morph this bug to what I actually did.