On Wednesday, December 23, 2020 4:30:48 AM CST Boyd Stephen Smith Jr. wrote: > On Wednesday, December 23, 2020 4:19:43 AM CST Michel Le Bihan wrote: > > The Debian patch for #963548 > > https://salsa.debian.org/mimi89999/chromium/-/blob/ece23b4ca107cd968ac9a40 > > 9f > > 40eb11edf8a0266/debian/patches/fixes/serviceworker-double-destruction.pat > > ch is clearly in upstream 87.0.4280.88: > > https://source.chromium.org/chromium/chromium/src/+/refs/tags/87.0.4280.88 > > :c > > ontent/browser/service_worker/service_worker_container_host.cc;l=671-679 > > Different destructor. The old patch is for ServiceWorkerObjectHost, but we > need a new patch for ServiceWorker*Registration*ObjectHost.
In particular, just above where you linked to, at line 629 (https:// source.chromium.org/chromium/chromium/src/+/refs/tags/87.0.4280.88:content/ browser/service_worker/service_worker_container_host.cc;l=629) is the crashing line. The fix for the back trace I finally got is to change that line like so: https://source.chromium.org/chromium/chromium/src/+/ f27ad210062f61d06eb782214ee4fc8c19a1725b:content/browser/service_worker/ service_worker_container_host.cc;l=623-647 The f27ed21 commit does contain the fix for #963548, but lower down at line 689: https://source.chromium.org/chromium/chromium/src/+/ f27ad210062f61d06eb782214ee4fc8c19a1725b:content/browser/service_worker/ service_worker_container_host.cc;l=689-697 -- Boyd Stephen Smith Jr. ,= ,-_-. =. b...@iguanasuicide.net ((_/)o o(\_)) Twitter: @DaTwinkDaddy `-'(. .)`-' http://iguanasuicide.net/ \_/
signature.asc
Description: This is a digitally signed message part.
