Closed
Bug 1482464
Opened 8 years ago
Closed 8 years ago
Unpin stops working in Top Sites / TypeError: e.identifier is null
Categories
(Firefox :: New Tab Page, defect, P1)
Firefox
New Tab Page
Tracking
()
People
(Reporter: nanj, Assigned: ahillier)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 external file)
|
1.05 MB,
image/gif
|
Details |
An error, "TypeError: e.identifier is null. TopSitesFeed.jsm:191:50", occurred when trying to unpin a top site in the latest nightly, this happened to other pinned sites as well with the same error message.
Updated•8 years ago
|
Priority: -- → P2
| Assignee | ||
Updated•8 years ago
|
Assignee: nobody → ahillier
Comment 1•8 years ago
|
||
Noted on irc, e.identifier is `null` for added search engines. So probably just check null before using.
Updated•8 years ago
|
Summary: Unpin stops working in Top Sites → Unpin stops working in Top Sites / TypeError: e.identifier is null
Comment 3•8 years ago
|
||
Comment 4•8 years ago
|
||
Commit pushed to master at https://github.com/mozilla/activity-stream
https://github.com/mozilla/activity-stream/commit/29d5513164bddc0b7858891ef08ecfd9d730d57f
Fix Bug 1482464: check for e.identifier before trying to match (#4319)
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Iteration: --- → 63.4 - Aug 20
Comment 5•8 years ago
|
||
Comment 7•8 years ago
|
||
status-firefox62:
--- → fixed
Updated•8 years ago
|
QA Whiteboard: [good first verify]
Updated•7 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•