Closed
Bug 1258865
Opened 8 years ago
Closed 8 years ago
Land version 1.2.2 of the Loop system add-on in mozilla-central
Categories
(Hello (Loop) :: Client, defect, P1)
Hello (Loop)
Client
Tracking
(firefox46 verified, firefox47 fixed, firefox48 fixed)
VERIFIED
FIXED
mozilla48
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Whiteboard: [btpp-fix-now])
Attachments
(5 files)
164.54 KB,
patch
|
standard8
:
review+
ritu
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
49.97 KB,
patch
|
standard8
:
review+
ritu
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
354.48 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
354.34 KB,
patch
|
ritu
:
approval-mozilla-aurora+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
354.40 KB,
patch
|
Details | Diff | Splinter Review |
This is the next update for Loop. Top-level changes: - Click indications during sharing to highlight where a person is pointing/clicking - Improvements to sharing: the invitation to copy a url now appears in the panel before the separate window opens - Editing the room name from the panel
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8733605 -
Flags: review+
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8733606 -
Flags: review+
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8733607 -
Flags: review+
Assignee | ||
Comment 4•8 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/40b54454f437fe04c74ce606a573c5f065aeb655 Bug 1258865 - Land version 1.2.2 of the Loop system add-on in mozilla-central - locale updates. rs=Standard8 for already reviewed code. https://hg.mozilla.org/integration/fx-team/rev/adc8f032dfbf951040608b5ef38a53f6e16725dd Bug 1258865 - Land version 1.2.2 of the Loop system add-on in mozilla-central - vendor updates. rs=Standard8 for already reviewed code. https://hg.mozilla.org/integration/fx-team/rev/cc5442d3b8bdd6c182d8da4993e0ad3d2e6a8fed Bug 1258865 - Land version 1.2.2 of the Loop system add-on in mozilla-central - code updates. rs=Standard8 for already reviewed code.
Assignee | ||
Updated•8 years ago
|
Rank: 1
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/40b54454f437 https://hg.mozilla.org/mozilla-central/rev/adc8f032dfbf https://hg.mozilla.org/mozilla-central/rev/cc5442d3b8bd
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Assignee | ||
Comment 6•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Attachment #8734357 -
Attachment description: Aurora version of code updates → Aurora/Beta version of code updates
Assignee | ||
Comment 7•8 years ago
|
||
Comment on attachment 8733605 [details] [diff] [review] Land version 1.2.2 of the Loop system add-on in mozilla-central - locale updates. rs=Standard8 for already reviewed code. Approval Request Comment [Feature/regressing bug #]: Firefox Hello 1.2.x release [User impact if declined]: See comment 0. In addition, this release enables e10s functionality for Hello users on Firefox 47 and later. It will remain "disabled" for users of 46. [Describe test coverage new/current, TreeHerder]: N/A String import from locales only. [Risks and why]: Low, string changes [String/UUID change made/needed]: None
Attachment #8733605 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 8•8 years ago
|
||
Comment on attachment 8733605 [details] [diff] [review] Land version 1.2.2 of the Loop system add-on in mozilla-central - locale updates. rs=Standard8 for already reviewed code. Approval Request Comment - Please see previous comment.
Attachment #8733605 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 9•8 years ago
|
||
Comment on attachment 8733606 [details] [diff] [review] Land version 1.2.2 of the Loop system add-on in mozilla-central - vendor updates. rs=Standard8 for already reviewed code. Approval Request Comment [Feature/regressing bug #]: Firefox Hello 1.2.x release [User impact if declined]: See comment 0. In addition, this release enables e10s functionality for Hello users on Firefox 47 and later. It will remain "disabled" for users of 46. [Describe test coverage new/current, TreeHerder]: This is updates to our vendor libaries which are well used by many projects. Our existing test coverage covers the parts we use of these libraries. [Risks and why]: Low, vendor library updates only. [String/UUID change made/needed]: None
Attachment #8733606 -
Flags: approval-mozilla-beta?
Attachment #8733606 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 10•8 years ago
|
||
Comment on attachment 8734357 [details] [diff] [review] Aurora/Beta version of code updates Approval Request Comment [Feature/regressing bug #]: Firefox Hello 1.2.x release [User impact if declined]: See comment 0. In addition, this release enables e10s functionality for Hello users on Firefox 47 and later. It will remain "disabled" for users of 46. [Describe test coverage new/current, TreeHerder]: Landed in m-c, tested amongst the team for a couple of weeks now. We have unit tests and integration [Risks and why]: For beta: Low, self-contained to Hello most of these updates have been in our tree for a while. For aurora: Medium as we're turning on the e10s portion of Hello, but we want to do this now, as 47 is likely to roll out e10s users, and we need to get testing as we think we've fixed almost all the issues. [String/UUID change made/needed]: None
Attachment #8734357 -
Flags: approval-mozilla-beta?
Attachment #8734357 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 11•8 years ago
|
||
Note: we also need bug 1254102 on aurora to fix various issues with Firefox crashing in e10s mode when we're tab sharing.
Depends on: 1254102
Comment 12•8 years ago
|
||
This should also fix an issue with private browsing (bug 1256362) which goes all the way to release.
Comment on attachment 8733605 [details] [diff] [review] Land version 1.2.2 of the Loop system add-on in mozilla-central - locale updates. rs=Standard8 for already reviewed code. Hello system add-on update v1.2.2, Aurora47+
Attachment #8733605 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #8733606 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox46:
--- → affected
status-firefox47:
--- → affected
Attachment #8734357 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 14•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/964565ed53bd https://hg.mozilla.org/releases/mozilla-aurora/rev/a7a99b6cb41c https://hg.mozilla.org/releases/mozilla-aurora/rev/4dfb216b20fb
Comment 15•8 years ago
|
||
Comment on attachment 8734357 [details] [diff] [review] Aurora/Beta version of code updates OK for beta uplift. This could make a beta 6 build 2 if we do one. If not, beta 7 build on Thursday and release on Friday.
Attachment #8734357 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•8 years ago
|
Attachment #8733605 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•8 years ago
|
Attachment #8733606 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 16•8 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/31a022dd8e88 https://hg.mozilla.org/releases/mozilla-beta/rev/d1949acf7016 https://hg.mozilla.org/releases/mozilla-beta/rev/4f87a2517f36
This broke some tests on beta: https://treeherder.mozilla.org/logviewer.html#?job_id=954879&repo=mozilla-beta https://treeherder.mozilla.org/logviewer.html#?job_id=954988&repo=mozilla-beta Backed out in https://hg.mozilla.org/releases/mozilla-beta/rev/5d390bfc7c86
Flags: needinfo?(standard8)
Assignee | ||
Comment 18•8 years ago
|
||
This fixes the xpcshell test issue on beta - there was some syntax that it didn't like for some reason. To fix the mochitest issue, we need bug 1245486 for which I've requested approval on.
Assignee | ||
Updated•8 years ago
|
Flags: needinfo?(standard8)
Comment 19•8 years ago
|
||
Wes or tomcat, can you land this again on beta this morning? Thanks.
Flags: needinfo?(wkocher)
Flags: needinfo?(cbook)
Comment 20•8 years ago
|
||
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #19) > Wes or tomcat, can you land this again on beta this morning? Thanks. np, done as https://hg.mozilla.org/releases/mozilla-beta/rev/e5b3ef0d7e32
Updated•8 years ago
|
Flags: qe-verify+
Assignee | ||
Comment 21•8 years ago
|
||
Landed missing csets: https://hg.mozilla.org/releases/mozilla-beta/rev/bcd08f3ce48011ba6d8a630032889593e3802157 https://hg.mozilla.org/releases/mozilla-beta/rev/191f5eb4cbd72590277296cdb90d355adb347d45
Comment 22•8 years ago
|
||
We finished Smoke testing on Firefox 46 beta 7 with 1.2.2 version of Hello add-on, we encountered 4 new issues (bugzilla query https://goo.gl/lK2255), but only one of them seems major (bug 1261843). Other than that, nothing major was found.
You need to log in
before you can comment on or make changes to this bug.
Description
•