tab objects returned by sessions.getRecentlyClosed are missing some data

RESOLVED FIXED in Firefox 52


2 years ago
7 months ago


(Reporter: bsilverberg, Assigned: bsilverberg)



Firefox Tracking Flags

(firefox52 fixed, firefox53 fixed)


(Whiteboard: [sessions]triaged)


(2 attachments)



2 years ago
When a WebExtension has the `tabs` permission and it uses sessions.getRecentlyClosed, the tabs returned by that should include:
- favIconUrl
- title
- url

but they do not. The API code should be updated to provide these, if possible.
Comment hidden (mozreview-request)

Comment 2

2 years ago
Comment on attachment 8825812 [details]
Bug 1329971 - tab objects returned by sessions.getRecentlyClosed are missing some data,

::: browser/components/extensions/ext-utils.js:713
(Diff revision 1)
>        active: false,
>        pinned: false,
>        incognito: Boolean(tab.state && tab.state.isPrivate),
>      };
> +

Nit: Remove extra newline.
Attachment #8825812 - Flags: review?(kmaglione+bmo) → review+
Comment hidden (mozreview-request)

Comment 4

2 years ago
Thanks for the review, Kris. This looks good on try, so requesting check-in.
Keywords: checkin-needed

Comment 5

2 years ago
Pushed by
tab objects returned by sessions.getRecentlyClosed are missing some data, r=kmag
Keywords: checkin-needed

Comment 6

2 years ago
Patch landed few hours ago
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53

Comment 8

2 years ago
Created attachment 8826264 [details] [diff] [review]
Uplift for 52

Approval Request Comment
[Feature/Bug causing the regression]: Bug found in sessions API, reported and fixed by bug 1329971
[User impact if declined]: Some expected tab data will not be available when calling browser.sessions.getRecentlyClosed, which could affect add-on developers using that API in their code.
[Is this code covered by automated tests?]: A new test was added along with this patch. A try run against Aurora was submitted at
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: No 
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No. 
[Why is the change risky/not risky?]: It is a simple change to a function that is only used by the sessions API, and it is covered by a test.
[String changes made/needed]: None
Attachment #8826264 - Flags: approval-mozilla-aurora?
Comment on attachment 8826264 [details] [diff] [review]
Uplift for 52

webextensions fix for aurora52
Attachment #8826264 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+


7 months ago
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.