Closed
Bug 1167394
Opened 9 years ago
Closed 9 years ago
Tab Queue should ignore Sync notifications
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox42 fixed)
RESOLVED
FIXED
Firefox 42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: tech4pwd, Assigned: mhaigh)
Details
Attachments
(1 file)
3.96 KB,
patch
|
mcomella
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:41.0) Gecko/20100101 Firefox/41.0 Build ID: 20150513191204 Steps to reproduce: If you send a tab to Fennec from another device via Sync, it should be ignored by the Tab Queue upon being tapped. It's redundant to tap a notification only to then need to tap another notification (even tapping the toast is a nuisance). Sent tabs should always open directly in the browser.
Reporter | ||
Updated•9 years ago
|
OS: Unspecified → Android
Hardware: Unspecified → All
Updated•9 years ago
|
Flags: needinfo?(mhaigh)
Assignee | ||
Comment 1•9 years ago
|
||
I've added a flag to all sync notifications which will ensure the Tab Queue directly opens these urls.
Flags: needinfo?(mhaigh)
Attachment #8626936 -
Flags: review?(michael.l.comella)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → mhaigh
Comment on attachment 8626936 [details] [diff] [review] Tab Queue should ignore Sync notifications Review of attachment 8626936 [details] [diff] [review]: ----------------------------------------------------------------- I'm unfamiliar with CommandProcessor but this looks about right. :)
Attachment #8626936 -
Flags: review?(michael.l.comella) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fee9f5658cb9
Assignee | ||
Comment 4•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/66a2faddf3c3
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/66a2faddf3c3
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Updated•3 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
•