Closed Bug 1362234 Opened 8 years ago Closed 8 years ago

Update Firefox Screenshots to version 6.6.1

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 55
Tracking Status
firefox55 --- fixed

People

(Reporter: jhirsch, Unassigned)

References

Details

Attachments

(1 file)

This bug updates the version of Firefox Screenshots in the tree from 6.6.0 to 6.6.1. The changes address Talos regressions tracked by bug 1361792.
Comment on attachment 8864705 [details] Bug 1362234 - Fix Screenshots Talos regressions by removing WebExtension startup delay and removing tab 'activated' event handler; https://reviewboard.mozilla.org/r/136348/#review139432 ::: browser/extensions/screenshots/webextension/background/main.js:202 (Diff revision 1) > browser.browserAction.disable(tabId); > setIconActive(true, tabId); > } > > browser.tabs.onUpdated.addListener(catcher.watchFunction((id, info, tab) => { > if (info.url && tab.active) { Since we're getting rid of the onActivated listener, we should also do this for non-active tabs.
Attachment #8864705 - Flags: review?(kmaglione+bmo) → review+
Addressed review feedback, re-pushed to Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=036449f37f161db5b236a804b8d60a2b7b479755 :kmag, I don't have level 3 commit access. Would you mind pushing this patch to autoland?
Flags: needinfo?(kmaglione+bmo)
Pushed by maglione.k@gmail.com: https://hg.mozilla.org/integration/autoland/rev/a12bfc0263ac Fix Screenshots Talos regressions by removing WebExtension startup delay and removing tab 'activated' event handler; r=kmag
Flags: needinfo?(kmaglione+bmo)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: