Chromium Code Reviews
DescriptionRevert of Inline Document::allowExecutingScripts logic (patchset #3 id:40001 of https://codereview.chromium.org/2640983005/ )
Reason for revert:
This caused memory leaks in Linux Leak Detector:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/1655
Original issue's description:
> Inline Document::allowExecutingScripts logic
>
> This CL removes and inlines Document::allowExecutingScripts predicate
> to ScriptLoader::prepareScript, where it was only called.
>
> BUG=None
>
> Review-Url: https://codereview.chromium.org/2640983005
> Cr-Commit-Position: refs/heads/master@{#445023}
> Committed: https://chromium.googlesource.com/chromium/src/+/6432181413b6162f3e78704987fe0f7c61e7c185
TBR=yukishiino@chromium.org,hiroshige@chromium.org,kouhei@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None
Review-Url: https://codereview.chromium.org/2648753002
Cr-Commit-Position: refs/heads/master@{#445039}
Committed: https://chromium.googlesource.com/chromium/src/+/f116503fb64582fe3c85a1589a1898570976d3cb
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|
|||||||||||||||||||||||||||||||||||||