Closed Bug 1268147 Opened 5 years ago Closed 5 years ago
Script Loader::Should Load Script is never called
nsScriptLoader::ShouldLoadScript has no callers.
nsScriptLoader was converted to use asyncOpen2() within Bug 1194526. At the moment I can't recall why I didn't remove that function. I would assume there was a reason. If it is indeed not called anymore, we can definitely delete it.
Assignee: nobody → ckerschb
Status: NEW → ASSIGNED
Component: DOM → DOM: Security
Jonas, please note that we can remove nsScriptLoader::ShouldLoadScript() but we can't remove nsScriptLoader::CheckContentPolicy() as suggested within  because we still need to call content policies on preloaded scripts .  https://bugzilla.mozilla.org/show_bug.cgi?id=1194526#c27  http://mxr.mozilla.org/mozilla-central/source/dom/base/nsScriptLoader.cpp#531
Attachment #8746437 - Flags: review?(jonas)
Attachment #8746437 - Flags: review?(jonas) → review+
You need to log in before you can comment on or make changes to this bug.