Closed
Bug 1907212
Opened 7 months ago
Closed 7 months ago
Split large CloseTabs commands into multiple sends
Categories
(Firefox :: Sync, task)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
130 Branch
| Tracking | Status | |
|---|---|---|
| firefox130 | --- | fixed |
People
(Reporter: skhamis, Assigned: skhamis)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxsync-])
Attachments
(1 file, 2 obsolete files)
There are scenarios where a user could easily hit the 16kb payload limit trying to close a large amount of tabs. Since we don't limit how many tabs they can close, we should simply split all the queued commands over multiple sends. We do something similar in the tabs engine https://searchfox.org/mozilla-central/source/services/sync/modules/engines/tabs.sys.mjs#424 where we pack as many tabs for the user as possible so they can sync all the tabs. We should do something similar here.
|
Assignee | |
Comment 1•7 months ago
|
||
|
||
Updated•7 months ago
|
Assignee: nobody → skhamis
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•7 months ago
|
||
|
|
Assignee | |
Comment 3•7 months ago
|
||
|
|
||
Updated•7 months ago
|
Attachment #9412148 -
Attachment is obsolete: true
|
|
||
Updated•7 months ago
|
Attachment #9412149 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•7 months ago
|
Whiteboard: [fxsync-]
|
||
Updated•7 months ago
|
Pushed by skhamis@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/552493040270
Split large Close Tab commands into multiple sends r=lina
|
||
Comment 5•7 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
status-firefox130:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 130 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•