1975014 - Add a pref for Compression Dictionaries

Status: RESOLVED FIXED

(Core :: Networking: Cache, enhancement, P3)

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Attachment: Bug 1975014: Add a pref for Compression Dictionaries r=#necko-reviewers!

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:jesup, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/124a4b4fc14c
https://hg.mozilla.org/integration/autoland/rev/d25d80d93174
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 4

[Sandor Molnar[:smolnar]]

Backed out for causing multiple bustages/failures

Backout link: https://hg.mozilla.org/integration/autoland/rev/837a3d407e1c473641116eb9e1a150237c764645

Push with failures

Failure log -> TEST-UNEXPECTED-ERROR | /builds/worker/checkouts/gecko/netwerk/test/unit/test_dictionary_storage.js:595:9 | 'headerValue' is assigned a value but never used. (no-unused-vars)

Failure log -> ERROR - /builds/worker/checkouts/gecko/netwerk/streamconv/converters/nsHTTPCompressConv.cpp:358:23: error: format '%d' expects argument of type 'int', but argument 4 has type 'nsresult' [-Werror=format=]

Failure log -> ERROR - /builds/worker/workspace/obj-build/dist/include/mozilla/net/Dictionary.h:313:14: error: unused variable 'rv' [-Werror,-Wunused-variable]

Failure log -> TEST-UNEXPECTED-FAIL | netwerk/test/unit/test_trr.js | test_server_up - [test_server_up : 84] "undefined" == "Error: could not find id"

Failure log -> TEST-UNEXPECTED-FAIL | /service-workers/service-worker/worker-interception.https.html

Failure log -> TEST-UNEXPECTED-FAIL | devtools/client/netmonitor/test/browser_net_copy_headers.js | Timed out while polling clipboard for requested data

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/d25d80d93174

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/e7d7622f1f94
https://hg.mozilla.org/integration/autoland/rev/94b3853d228b
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 7

[amarc]

Backed out for causing multiple failures

• Backout link
• Push with failures
• Failure Log
• Failure Log 2
• Failure Log 3

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/3a51e8f6a3a4
https://hg.mozilla.org/integration/autoland/rev/32da99b46c16
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a57404e2ad51
https://hg.mozilla.org/integration/autoland/rev/5a2f9906e7a8
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/3ce6095849c0
https://hg.mozilla.org/integration/autoland/rev/de97f7b81f69
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 11

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/de97f7b81f69

Comment 12

[Atila Butkovits]

Backed out for causing failures /dictionary-compressed.tentative.https.html.

Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/339cb96b8aea9a433abdfee35eca4472ecb7074e

Push where failures started: https://treeherder.mozilla.org/jobs?repo=autoland&searchStr=windows%2C11%2Cx86%2C24h2%2Copt%2Cweb%2Cplatform%2Ctests%2Ctest-windows11-32-24h2%2Fopt-web-platform-tests%2Cwpt12&revision=f63dea815067f6fefbac80cce46e1a0589f66a31&selectedTaskRun=OXPZFu52Tyq66R0qicmggw.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=529651894&repo=autoland&task=OXPZFu52Tyq66R0qicmggw.0&lineNumber=10724

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/00429283549a
https://hg.mozilla.org/integration/autoland/rev/7ec5a1fa98f5
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/d5cf72906da0
https://hg.mozilla.org/integration/autoland/rev/60f9fa2dd70c
Add a pref for Compression Dictionaries r=necko-reviewers,kershaw

Comment 15

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/60f9fa2dd70c

Comment 16

[Pascal Chevrel:pascalc]

Randell, is that a pref for a platform feature riding the 145 train? Should it be added to our release notes if this is the case? Thanks