Closed
Bug 1385440
Opened 7 years ago
Closed 7 years ago
[sync-send tab] Only one device receives the link when "Send to All Devices" is clicked
Categories
(Firefox :: Sync, defect, P1)
Tracking
()
VERIFIED
FIXED
Firefox 57
People
(Reporter: Abe_LV, Assigned: eoger)
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
markh
:
review+
lizzard
:
approval-mozilla-beta+
|
Details |
Steps to Reproduce: 1. Open Nightly on three devices and log-in to sync to all of them 2. open any http or https link and right click on the page then hover over "Send Page to Device" and click "Send to All Devices" Actual Result: Only one of the devices receives the link Expected Result: All devices should receive the link
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(eoger)
Comment 1•7 years ago
|
||
I can reproduce this - the logs show us attempting to send them correctly, but the command only ends up in one client record. I suspect this is the same race we briefly discussed last week?
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(eoger)
Priority: -- → P1
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → eoger
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8892957 [details] Bug 1385440 - Serialize client commands additions. https://reviewboard.mozilla.org/r/163966/#review169502 Awesome, thanks!
Attachment #8892957 -
Flags: review?(markh) → review+
Pushed by eoger@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5b9526464192 Serialize client commands additions. r=markh
Comment 5•7 years ago
|
||
After this lands, let's verify it works correctly then request uplift ASAP - adding ni? on Ed so we don't forget this :)
Flags: needinfo?(eoger)
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5b9526464192
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Assignee | ||
Comment 7•7 years ago
|
||
Comment on attachment 8892957 [details] Bug 1385440 - Serialize client commands additions. Approval Request Comment [Feature/Bug causing the regression]: 1210296 [User impact if declined]: The "Send Tab to all devices" button will not work. [Is this code covered by automated tests?]: Yes [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: - [List of other uplifts needed for the feature/fix]: - [Is the change risky?]: No [Why is the change risky/not risky?]: The changes in the code are trivial and we introduced a test to make sure this regression wouldn't happen again. [String changes made/needed]: -
Flags: needinfo?(eoger)
Attachment #8892957 -
Flags: approval-mozilla-beta?
Comment 8•7 years ago
|
||
Andrei or Ioana - can you help to verify the fix in 57 before we uplift? Thanks.
Flags: needinfo?(chiorean.ioana)
Flags: needinfo?(andrei.vaida)
Comment 9•7 years ago
|
||
Is this a new feature? Does it need a release note?
Assignee | ||
Comment 10•7 years ago
|
||
Not really, this is mostly a fix.
Comment 11•7 years ago
|
||
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #8) > Andrei or Ioana - can you help to verify the fix in 57 before we uplift? > Thanks. I will ask Ninu and Sorina, they handle FxA testing. I think they can verify desktop side too (or maybe Kanchan if not)
Flags: needinfo?(sorina.florean)
Flags: needinfo?(mihai.ninu)
Flags: needinfo?(chiorean.ioana)
Flags: needinfo?(andrei.vaida)
Comment 12•7 years ago
|
||
Verified as fixed on the latest Nightly build (57.0a1 - 08/08/2017) This issue was tested by sending from the desktop build to all synced devices (list below). The sent tab was received on all devices. Samsung Galaxy S6 EDGE - Android 7.0 LG G5 - Android 6.0.1 Nexus 6P - Android 8 Samsung Galaxy Tab S3 - Android 7.0 HTC ONE M9 - Android 6.0
Updated•7 years ago
|
status-firefox56:
--- → affected
Comment 13•7 years ago
|
||
Comment on attachment 8892957 [details] Bug 1385440 - Serialize client commands additions. Thanks very much for the verification! Let's uplift this fix for 56 beta 2.
Attachment #8892957 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 14•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/bb925de1a0f3
Flags: in-testsuite+
Comment 15•7 years ago
|
||
Tested again using the latest Beta build on desktop (56.0b2) and latest beta on mobile (56.0b1) and I can confirm that this issue has been fixed. I will verify it as fixed on Wednesday(August 16th) when we have a new beta on mobile. Devices used: Nexus 6P - Android 8 LG v20 - Android 7.0 ONEPLUS A3000 - Android 6.0.1 Xiaomi mi I4 - Android 5.0.2
Comment 16•7 years ago
|
||
Verified Fixed based on Comment 12 and Comment 15.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•