Closed
Bug 556376
Opened 16 years ago
Closed 16 years ago
Provide a common head for Places xpcshell-tests
Categories
(Toolkit :: Places, defect)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla1.9.3a4
People
(Reporter: mak, Assigned: mak)
References
Details
Attachments
(1 file, 1 obsolete file)
203.50 KB,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
Unify common tools into a single head files, and stop being lazy.
Assignee | ||
Comment 1•16 years ago
|
||
As discussed on IRC, i don't think these tests changes are going to need deep review. This one is the basic infrastructure, next ones would just be cleanups and var/svc replacements.
The idea here is that we have a head_common.js script in toolkit/components/places/tests/ all the other head_xxx.js files import this common head, and then define specific methods/vars for that specific test folder.
browser tests head also import this file.
head_common.js has Services, XPCOMUTils, NetUtil, a PlacesServices object with lazy getters for our services, and some utils.
in future we'll probably add PlacesUtils and PlacesUIUtils getters, and other common code.
I tried to scope all of the possible, so that variables conflicts are hard.
Will trypush all changes before going to m-c
Attachment #436366 -
Flags: feedback?(dietrich)
Assignee | ||
Comment 2•16 years ago
|
||
tryserver said green
Assignee | ||
Comment 3•16 years ago
|
||
hm, i'm actually thinking if would make sense to kill PlacesServices, add missing getters to PlacesUtils, and use PlacesUtils everywhere instead. Would create better examples for third party implementers looking at tests.
Assignee | ||
Comment 4•16 years ago
|
||
yeah. i'll do that, and will ask review for PU changes.
Assignee | ||
Comment 5•16 years ago
|
||
Using PlacesUtils!
asking review on PlacesUtils changes and head_common.js setup
other changes are merely name substitutions for now.
Attachment #436366 -
Attachment is obsolete: true
Attachment #436554 -
Flags: review?
Attachment #436366 -
Flags: feedback?(dietrich)
Assignee | ||
Updated•16 years ago
|
Attachment #436554 -
Flags: review? → review?(dietrich)
Comment 6•16 years ago
|
||
Comment on attachment 436554 [details] [diff] [review]
patch v1.1
looks good, thanks for doing this. r=me.
Attachment #436554 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 7•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a4
You need to log in
before you can comment on or make changes to this bug.
Description
•