"View Pocket List" menuitem not working

VERIFIED FIXED in Firefox 38.0.5

Status

()

defect
P1
normal
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: Dolske, Assigned: Dolske)

Tracking

Trunk
Firefox 40
Points:
---
Dependency tree / graph
Bug Flags:
firefox-backlog +

Firefox Tracking Flags

(firefox38.0.5 verified, firefox39 verified, firefox40 fixed)

Details

Attachments

(1 attachment)

No description provided.
Posted patch Patch v.1Splinter Review
(Thanks for the accidental early bug submission, Bugzilla.)

The "View Pocket List" menuitem in the bookmarks menu and bookmarks button menupanel are not working. Clicking them does nothing. Console shows:

JavaScript error: resource:///modules/Pocket.jsm, line 18: ReferenceError: Services is not defined

Fix is trivial.
Assignee: nobody → dolske
Attachment #8603767 - Flags: review?(jaws)
[This bug is a textbook case of what happens when features are rushed. Bug 1161654 broke the context menu, that was fixed in bug 1162735 but in doing so it introduced this bug. :(]
Blocks: 1162735
Priority: -- → P1
Attachment #8603767 - Flags: review?(jaws) → review+
Comment on attachment 8603767 [details] [diff] [review]
Patch v.1

[Triage Comment]

Required for Pocket / 38.0.5 release.
Attachment #8603767 - Flags: approval-mozilla-release+
Attachment #8603767 - Flags: approval-mozilla-aurora+
Iteration: --- → 40.3 - 11 May
Flags: qe-verify?
Flags: firefox-backlog+
Flags: qe-verify? → qe-verify+
QA Contact: andrei.vaida
Verified using Firefox 38.0.5 beta 1 and latest Dev Edition 2015-05-11 that "View Pocket List" menuitem from Bookmarks menu and toolbar points to Pocket List.
Environments used: Win 7 64-bit, Ubuntu 14.04 32-bit and Mac OS X 10.9.5.
Removing qe-verify flag as verification on Beta 38.0.5 and Aurora 39 should suffice.
Flags: qe-verify+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.