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
Created attachment 8746437 [details] [diff] [review] bug_1268147_remove_nsscriptloader_shouldloadscript.patch 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+
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.