Closed
Bug 1148459
Opened 9 years ago
Closed 9 years ago
Use new keywords API in tests
Categories
(Toolkit :: Places, defect)
Toolkit
Places
Tracking
()
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: mak, Assigned: mak)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
69.80 KB,
patch
|
ttaubert
:
review+
|
Details | Diff | Splinter Review |
Use the new keywords API in toolkit/components/places/tests/bookmarks/test_bookmarks.js
Assignee | ||
Updated•9 years ago
|
Component: Bookmarks & History → Places
Product: Firefox → Toolkit
Assignee | ||
Comment 1•9 years ago
|
||
let's make this generic for all tests toolkit/components/places/tests/browser/browser_bug248970.js toolkit/components/places/tests/bookmarks/test_bookmarks.js toolkit/components/places/tests/queries/head_queries.js toolkit/components/places/tests/unit/test_398914.js toolkit/components/places/tests/unit/test_preventive_maintenance.js toolkit/components/places/tests/unit/test_placesTxn.js toolkit/components/places/tests/unit/test_telemetry.js
Summary: Use new keywords API in toolkit/components/places/tests/bookmarks/test_bookmarks.js → Use new keywords API in tests
Assignee | ||
Comment 2•9 years ago
|
||
Updated list: toolkit/components/places/tests/autocomplete/head_autocomplete.js toolkit/components/places/tests/browser/browser_bug248970.js toolkit/components/places/tests/queries/head_queries.js toolkit/components/places/tests/unit/test_preventive_maintenance.js toolkit/components/places/tests/unit/test_placesTxn.js toolkit/components/places/tests/unit/test_telemetry.js toolkit/components/places/tests/unit/test_promiseBookmarksTree.js
Assignee | ||
Comment 3•9 years ago
|
||
I'm picking this bug.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Points: 1 → 2
Flags: qe-verify-
Flags: firefox-backlog+
Updated•9 years ago
|
Iteration: --- → 40.1 - 13 Apr
Updated•9 years ago
|
Iteration: 40.1 - 13 Apr → 40.2 - 27 Apr
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8595387 -
Flags: review?(ttaubert)
Comment 5•9 years ago
|
||
Comment on attachment 8595387 [details] [diff] [review] patch v1 Review of attachment 8595387 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/components/places/tests/queries/head_queries.js @@ +169,5 @@ > > if (qdata.keyword) { > let itemId = yield PlacesUtils.promiseItemId(item.guid); > + yield PlacesUtils.keywords.insert({ url: qdata.uri, > + keyword: qdata.keyword }); Can remove the promiseItemId() call. @@ +228,5 @@ > this.isTag = obj.isTag ? obj.isTag : false; > this.tagArray = obj.tagArray ? obj.tagArray : null; > this.isLivemark = obj.isLivemark ? obj.isLivemark : false; > + this.parentGuid = obj.parentGuid ? obj.parentGuid > + : PlacesUtils.bookmarks.rootGuid; Seems equivalent to: this.parentGuid = obj.parentGuid || PlacesUtils.bookmarks.rootGuid; ::: toolkit/components/places/tests/queries/test_onlyBookmarked.js @@ +26,5 @@ > > // Add a bookmark that should not be in the results > { isBookmark: true, > uri: "http://bookmarked-elsewhere.com/", > + parentFolder: PlacesUtils.bookmarks.menuGuid, parentGuid?
Attachment #8595387 -
Flags: review?(ttaubert) → review+
Comment 7•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/624ec1ca79d5
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in
before you can comment on or make changes to this bug.
Description
•