New issue
Advanced search Search tips

Issue 772367 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 2
Type: Bug



Sign in to add a comment

[Missing Test]: Failing to load installed scripts should terminate the worker thread

Project Member Reported by susanjuniab@chromium.org, Oct 6 2017

Issue description

Automated tests for the below commit have been missing.Please add test coverage ASAP to avoid regressions in future.

CL: 
----
https://chromium.googlesource.com/chromium/src.git/+/e4404c2f93e2d2ccacf15b07c834bff8051ddb63

Ref Bug: 
---------
https://bugs.chromium.org/p/chromium/issues/detail?id=755054

Thank you...!!
 
 
Labels: -OS-iOS

Comment 2 by falken@chromium.org, Dec 15 2017

Status: WontFix (was: Assigned)
This CL doesn't seem to cause a behavior change: "For cleanup, this CL moves InstalledScriptsManager access from
WorkerThread::InitializeOnWorkerThread() to
ServiceWorkerGlobalScope::EvaluateClassicScript()"

Closing.

Sign in to add a comment