Closed Bug 1440524 Opened 2 years ago Closed 2 years ago

Update Screenshots to version 30.0.0


(Firefox :: Screenshots, enhancement)

Not set



Firefox 60
Tracking Status
firefox60 --- fixed


(Reporter: _6a68, Assigned: _6a68)



(2 files)

Update Screenshots to 30.0.0, latest version in github.

Complete changelog:

Add-on changes:

* Remove one of the addon icon svgs. ([#4132]( [b456233](
* Include onboarding slides in user timing measurements. Fixes [#4097]( [69b58f7](
* Use for interaction timings. Fixes [#4054]( [3119bb4](
* Show loader animation on truncated full page download/copy. ([#3861]( [6fa7f3a](
* Fix lint issues. ([#4076]( [51dff4c](
* Use double quotes. ([#3977]( [2764e9d](
* Use eqeqeq. ([#3977]( [35f01a4](
* Use prefer-const. ([#3977]( [eefee3d](
* Replace var with let or const. ([#3977]( [6824d8b](
* Enable consistent-return. ([#3977]( [ed632d6](
* apply svgo to all svgs. Fixes [#3853]( [28ef28c](
* Suppress errors encountered when sending timing or events pings
  Analytics network errors should never be user-visible. [73c1340](
* Cancel some user timing measurements when page visibility changes to 'hidden'
  JS background budgeting in Firefox 58 might cause Screenshots execution
  to slow down if the user switches tabs while waiting for the UI to
  Exclude cases where the user timing event ends with an upload, since the
  result of a successful upload is to switch tabs (to the shot page of the
  uploaded shot).
. Fixes [#4055]( [588d2e7](
* Fire an internal-only event if Screenshots is in a document that is backgrounded
  Use the Page Visibility API to detect when a given window loses focus. [2aed29b](
* Upstream changes to loader calls in bootstrap.js context
  Cu.import has been replaced with ChromeUtils.import, and
  XPCOM.defineLazyModuleGetter has been replaced with
  ChromeUtils.defineModuleGetter. Adjust our code accordingly.
  See bugzilla bug 1431533 for details. [07b3b39](
Comment on attachment 8953307 [details]
Bug 1440524 - Export Screenshots 30.0.0 to Firefox (translations only);
Attachment #8953307 - Flags: review?(francesco.lodolo) → review+
My previous Try push landed at the exact time the Talos windows machines got upgraded, so there is an apparent dramatic improvement in test perf between my push and the previous m-c base commit:

There should be a new base commit on new hardware by now. Re-pushing a new set of Talos runs, which should be representative of any actual perf impact caused by the version 30 export:
Comment on attachment 8953307 [details]
Bug 1440524 - Export Screenshots 30.0.0 to Firefox (translations only);
Attachment #8953307 - Flags: review?(ianb) → review+
Comment on attachment 8953308 [details]
Bug 1440524 - Export Screenshots 30.0.0 to Firefox (code excluding translations);
Attachment #8953308 - Flags: review?(ianb) → review+
Whoops, had to add another bug fix to the export.

Changelog addition:

* Remove two extra handlers and add more conditions around ctrl+c. ([#4113]( ([#4149]( [e82ef8e](
Assignee: nobody → jhirsch
QA didn't find any new blockers. Merging
Pushed by
Export Screenshots 30.0.0 to Firefox (translations only); r=flod,ianbicking
Export Screenshots 30.0.0 to Firefox (code excluding translations); r=ianbicking
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.