Closed Bug 1267835 Opened 8 years ago Closed 8 years ago

Land version 1.3.1 of the Loop system add-on in mozilla-central

Categories

(Hello (Loop) :: Client, defect, P1)

defect

Tracking

(firefox47 verified, firefox48 verified, firefox49 verified)

VERIFIED FIXED
mozilla49
Tracking Status
firefox47 --- verified
firefox48 --- verified
firefox49 --- verified

People

(Reporter: standard8, Assigned: standard8)

References

()

Details

(Whiteboard: [btpp-fix-now])

Attachments

(1 file)

Attachment #8745616 - Flags: review+
Summary of changes:

- Stop the panel being able to be display in private browsing mode (bug 1261849)
- Stop listeners being duplicated when opening the panel (bug 1250849)
- Fixes for intermittent failures in tests
https://hg.mozilla.org/mozilla-central/rev/b567617d9f02
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Comment on attachment 8745616 [details] [diff] [review]
Land version 1.3.1 of the Loop system add-on in mozilla-central, rs=Standard8 for already reviewed code.

Approval Request Comment
[Feature/regressing bug #]: Firefox Hello
[User impact if declined]: Users may be able to get into Hello in private browsing mode causing confusion (bug 1261849). This also fixes an issue with listeners being added multiple times (if Hello panel was used it a lot in one session it could cause visible slow-downs, but its probably not generally user visible).

Also fixes issues with intermittent/perma oranges.

[Describe test coverage new/current, TreeHerder]: Landed in m-c, panel code has various automatic tests.
[Risks and why]: Low, fixes issues in tests, very small localised changes for the panel.
[String/UUID change made/needed]: None
Attachment #8745616 - Flags: approval-mozilla-aurora?
Comment on attachment 8745616 [details] [diff] [review]
Land version 1.3.1 of the Loop system add-on in mozilla-central, rs=Standard8 for already reviewed code.

Hello 1.3.1 uplift to Fx48, baked in Nightly for a week, Aurora48+
Attachment #8745616 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Blocks: 1271260
Blocks: 1274226
Comment on attachment 8745616 [details] [diff] [review]
Land version 1.3.1 of the Loop system add-on in mozilla-central, rs=Standard8 for already reviewed code.

Approval Request Comment
[Feature/regressing bug #]: Firefox Hello 1.3.2
[User impact if declined]: Firefox 47 users won't have the same version as Firefox 46.
[Describe test coverage new/current, TreeHerder]: Landed in central & aurora. Shipped via go faster to beta 47 and now release 46
[Risks and why]: Landing shipped code.
[String/UUID change made/needed]: None
Attachment #8745616 - Flags: approval-mozilla-beta?
Comment on attachment 8745616 [details] [diff] [review]
Land version 1.3.1 of the Loop system add-on in mozilla-central, rs=Standard8 for already reviewed code.

Makes sense, Beta47+
Attachment #8745616 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Based on our testing done using Firefox 47 beta 4 and Hello version 1.3.2 I think we can close this bug as verified.

For more details about our testing please see this gdoc: https://goo.gl/mKDa2R

I will also mark 48 and 49 as verified as well since there is a new version coming already.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.