1988076 - Add shutdown and related cleanup for Compression Dictionaries

Status: RESOLVED FIXED

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

Comment 1

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

Attachment: Bug 1988076: Shutdown cleanup issues for Compression Dictionaries r=#necko-reviewers!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/0174e07469d2
https://hg.mozilla.org/integration/autoland/rev/b49eee879589
Shutdown cleanup issues for Compression Dictionaries r=necko-reviewers,valentin

Comment 3

[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 4

[Cristian Tuns]

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

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/298f2719caf6
https://hg.mozilla.org/integration/autoland/rev/b46f9e6da2f2
Shutdown cleanup issues for Compression Dictionaries r=necko-reviewers,valentin

Comment 6

[amarc]

Backed out for causing multiple failures

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

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/5d0db3ea2e52
https://hg.mozilla.org/integration/autoland/rev/a603a89a97c9
Shutdown cleanup issues for Compression Dictionaries r=necko-reviewers,valentin

Comment 8

[agoloman]

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

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/029f476c058e
https://hg.mozilla.org/integration/autoland/rev/adbfcbccfbb5
Shutdown cleanup issues for Compression Dictionaries r=necko-reviewers,valentin

Comment 10

[Cristian Tuns]

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

Comment 11

[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 12

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/22c04bad814b
https://hg.mozilla.org/integration/autoland/rev/f48c99278e66
Shutdown cleanup issues for Compression Dictionaries r=necko-reviewers,valentin

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/7e3bc66813a0
https://hg.mozilla.org/integration/autoland/rev/b9b61857922d
Shutdown cleanup issues for Compression Dictionaries r=necko-reviewers,valentin

Comment 14

[pstanciu]

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