Closed
Bug 464918
Opened 16 years ago
Closed 16 years ago
Console clearing test for private browsing should not use a timeout
Categories
(Firefox :: Private Browsing, defect)
Firefox
Private Browsing
Tracking
()
RESOLVED
FIXED
Firefox 3.1b3
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
(Keywords: fixed1.9.1)
Attachments
(2 files, 1 obsolete file)
2.15 KB,
patch
|
mconnor
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
2.30 KB,
patch
|
Details | Diff | Splinter Review |
We can do better in the console clearing test for the private browsing service, and make it not use a timeout. The reason we needed the timeout previously was that console listeners are async proxy objects, so the listener object was not notified on time for us to test this serially.
My solution here is to log a special "exit" message on the console at the end of the test, so that the listener can pick it up and do the final checks then and finish the test.
This will hopefully help the test suite to run a tiny bit faster. :-)
Attachment #348185 -
Flags: review?(mconnor)
Updated•16 years ago
|
Attachment #348185 -
Flags: review?(mconnor) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #348185 -
Flags: approval1.9.1?
Comment 1•16 years ago
|
||
Comment on attachment 348185 [details] [diff] [review]
Patch (v1)
a191=beltzner
Attachment #348185 -
Flags: approval1.9.1? → approval1.9.1+
Assignee | ||
Comment 2•16 years ago
|
||
Assignee | ||
Comment 3•16 years ago
|
||
Attachment #350513 -
Attachment is obsolete: true
Comment 4•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1b3
Assignee | ||
Comment 5•16 years ago
|
||
Mass moving of all Firefox::General private browsing bugs to Firefox::Private Browsing.
Component: General → Private Browsing
Assignee | ||
Updated•16 years ago
|
QA Contact: general → private.browsing
Updated•16 years ago
|
Keywords: fixed1.9.1
You need to log in
before you can comment on or make changes to this bug.
Description
•