Closed Bug 885254 Opened 11 years ago Closed 11 years ago

Don't use getPageTitle in test_history.js

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: asaf, Assigned: raymondlee)

References

Details

Attachments

(1 file, 2 obsolete files)

      No description provided.
Assignee: nobody → raymond
Status: NEW → ASSIGNED
Attached patch v1 (obsolete) — Splinter Review
Attachment #805208 - Flags: review?(mak77)
Comment on attachment 805208 [details] [diff] [review]
v1

Please use promisePlaceInfo from PlacesUtils. The version in the getPlacesInfo test is there for testing minor implementation details.
Attachment #805208 - Flags: review?(mak77) → review-
Attached patch v2 (obsolete) — Splinter Review
Attachment #805208 - Attachment is obsolete: true
Attachment #805790 - Flags: review?(mano)
Attachment #805790 - Flags: review?(mano) → review+
Passed try
https://tbpl.mozilla.org/?tree=Try&rev=9e309621f4a8
Attachment #805790 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/1ace948b5b82
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/1ace948b5b82
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: