Closed
Bug 1324728
Opened 8 years ago
Closed 8 years ago
While sharing link to another device, the sender device itself should be excluded from the selection list
Categories
(Firefox for Android Graveyard :: Android Sync, defect)
Firefox for Android Graveyard
Android Sync
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: wesley_huang, Unassigned)
References
Details
Attachments
(1 file)
161.19 KB,
image/png
|
Details |
STR:
1. Trigger a link sharing to "Send to other devices"
2. Check on the list of selection
Actual result:
The sender itself is in the list
Expected result:
The sender itself should be excluded from the list
Reporter | ||
Updated•8 years ago
|
Blocks: fennec-polish
Flags: needinfo?(max)
Comment 1•8 years ago
|
||
This should already be the case. Did you log in multiple times on this device or do you have multiple release channels installed?
Updated•8 years ago
|
Component: General → Android Sync
Product: Firefox for Android → Android Background Services
Comment 2•8 years ago
|
||
If we want to exclude sender itself from the list, we might have to provide something like "device sn - current release channel - current sync profile id" as a unique key to filter. In current design, login/out multiple times also do not reuse/replace the profile with same "device sn - current release channel" pair. That behavior also confuse user the definition of "Sync" or "Login".
Flags: needinfo?(max)
Comment 3•8 years ago
|
||
https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/overlays/service/sharemethods/SendTab.java#172
https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/db/LocalTabsAccessor.java#48
This is already the case. Local clients have a null device ID, which is excluded when querying for remote clients inside SendTab. If you've ever cleared data or uninstalled Firefox, you will have duplicate client records.
Please reopen if you can see the same client after:
* Uninstalling Firefox.
* Installing Firefox.
* Creating a brand new Firefox Account.
* Trying to send a tab.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Comment 4•8 years ago
|
||
Re Comment 2: note that FxA has no concept of a 'device' that's the home to multiple distinct Firefox profiles/logins. Indeed, users do use FxA to send tabs between different release channels on the same device, so it's not clear that it's even desirable to try to hide different Firefoxes from the same Android device.
Most users will never see entries from Firefoxes on the same device, and those that do quite likely _want_ to see them.
Comment 5•8 years ago
|
||
For cleanup after signing out, see Bug 963446.
Updated•8 years ago
|
Product: Android Background Services → Firefox for Android
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•