Closed Bug 1380817 Opened 3 years ago Closed 3 years ago

Update Screenshots to version 10.5.0

Categories

(Firefox :: Screenshots, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 56
Tracking Status
firefox55 --- verified
firefox56 --- verified

People

(Reporter: ianbicking, Assigned: ianbicking)

References

Details

Attachments

(1 file)

Assignee: nobody → ianb
Comment on attachment 8886367 [details]
Bug 1380817 - Export Screenshots 10.5.0 to Firefox;

https://reviewboard.mozilla.org/r/157112/#review162286

This seems to be missing the newly added strings from the private browsing change.
Attachment #8886367 - Flags: review?(dtownsend) → review-
Comment on attachment 8886367 [details]
Bug 1380817 - Export Screenshots 10.5.0 to Firefox;

https://reviewboard.mozilla.org/r/157112/#review162580
Attachment #8886367 - Flags: review?(dtownsend) → review+
Keywords: checkin-needed
Blocks: 1381132
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/40bef89faa11
Export Screenshots 10.5.0 to Firefox; r=mossop
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/40bef89faa11
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Comment on attachment 8886367 [details]
Bug 1380817 - Export Screenshots 10.5.0 to Firefox;

Approval Request Comment
[Feature/Bug causing the regression]:
Screenshots

[User impact if declined]:
- People using Screenshots in a Private Browsing window will have their downloads listed in the main download manager
- Users with dark themes cannot see the icon
- Windows users encounter download issues with many sites
- That "Save" leads to an upload may confuse some users

[Is this code covered by automated tests?]:
No

[Has the fix been verified in Nightly?]:
Yes

[Needs manual test from QE? If yes, steps to reproduce]: 
SoftVision has QA'd this

[List of other uplifts needed for the feature/fix]:
Bug 1377302 and Bug 1379464 (for dark theme support)

[Is the change risky?]:
No

[Why is the change risky/not risky?]:
Changes are to the WebExtension portion of Screenshots

[String changes made/needed]:
No (it does include a string addition, but we're localizing outside of the Firefox process, a future update will include translations for the new string)
Attachment #8886367 - Flags: approval-mozilla-beta?
Comment on attachment 8886367 [details]
Bug 1380817 - Export Screenshots 10.5.0 to Firefox;

screenshots update for fx55, should be in 55.0b10
Attachment #8886367 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
The fixes stated in comment 0 reached Firefox Nightly 56.0a1 (2017-07-15) and Firefox 55.0b10 build1 (20170717063821). Marking this issue as verified.
Status: RESOLVED → VERIFIED
Target Milestone: --- → mozilla56
Target Milestone: mozilla56 → Firefox 56
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.