Closed
Bug 820763
Opened 12 years ago
Closed 12 years ago
Stop using addvisit() in toolkit tests
Categories
(Toolkit :: Places, defect)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: mak, Assigned: raymondlee)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
The following tests are still using addVisit: toolkit/components/places/tests/browser/browser_bug248970.js toolkit/components/downloads/test/unit/test_history_expiration.js toolkit/forgetaboutsite/test/unit/test_removeDataFromDomain.js toolkit/components/places/tests/cpp/places_test_harness.h
Assignee | ||
Comment 1•12 years ago
|
||
I am not very familiar code so I have just updated the js files.
Assignee: nobody → raymond
Attachment #703215 -
Flags: review?(mak77)
Assignee | ||
Comment 2•12 years ago
|
||
(In reply to Raymond Lee [:raymondlee] from comment #1) > Created attachment 703215 [details] [diff] [review] > v1 > > I am not very familiar code so I have just updated the js files. I mean cpp
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 703215 [details] [diff] [review] v1 Review of attachment 703215 [details] [diff] [review]: ----------------------------------------------------------------- please file a separate bug blocking bug 700250 to fix the cpp test. ::: toolkit/forgetaboutsite/test/unit/test_removeDataFromDomain.js @@ +611,5 @@ > os.removeObserver(observer, "cacheservice:empty-cache"); > do_test_finished(); > + > + // Shutdown the download manager. > + Services.obs.notifyObservers(null, "quit-application", null); please do this before invoking do_test_finished, just for coherence.
Attachment #703215 -
Flags: review?(mak77) → review+
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #3) > Comment on attachment 703215 [details] [diff] [review] > v1 > > Review of attachment 703215 [details] [diff] [review]: > ----------------------------------------------------------------- > > please file a separate bug blocking bug 700250 to fix the cpp test. filed bug 832133 > > ::: toolkit/forgetaboutsite/test/unit/test_removeDataFromDomain.js > @@ +611,5 @@ > > os.removeObserver(observer, "cacheservice:empty-cache"); > > do_test_finished(); > > + > > + // Shutdown the download manager. > > + Services.obs.notifyObservers(null, "quit-application", null); > > please do this before invoking do_test_finished, just for coherence. fixed
Attachment #703215 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•12 years ago
|
||
It looks good on try https://tbpl.mozilla.org/?tree=Try&rev=bae122fbc758
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 6•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/87fe8d808537
Keywords: checkin-needed
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/87fe8d808537
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in
before you can comment on or make changes to this bug.
Description
•