Do not fire load events from frames with scripting disabled

Do not fire load events from frames with scripting disabled

This patch fixes a crash where an SVG CSS background image could cause
all pending load events to fire. This became problematic once data uri
images started firing synchronously. The events we are interested in
preventing are global pending load events because they are cross-
document and will be fired from an inner (sandboxed) SVG document.

This patch disables load events from frames where scripting is disabled.

BUG=248843
R=japhet@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153029

[Nate Chapin, 2013-06-25 20:43:22]

https://codereview.chromium.org/17682003/diff/4001/Source/core/dom/Document.cpp
File Source/core/dom/Document.cpp (right):

https://codereview.chromium.org/17682003/diff/4001/Source/core/dom/Document.c...
Source/core/dom/Document.cpp:2278: if (!f ||
f->script()->canExecuteScripts(NotAboutToExecuteScript)) {
Should we be firing events for a detached document?

[pdr., 2013-06-25 21:15:08]

On 2013/06/25 20:43:22, Nate Chapin wrote:
>
https://codereview.chromium.org/17682003/diff/4001/Source/core/dom/Document.cpp
> File Source/core/dom/Document.cpp (right):
> 
>
https://codereview.chromium.org/17682003/diff/4001/Source/core/dom/Document.c...
> Source/core/dom/Document.cpp:2278: if (!f ||
> f->script()->canExecuteScripts(NotAboutToExecuteScript)) {
> Should we be firing events for a detached document?

That's a reasonable idea and the tests seem to pass as well.

PTAL

[Nate Chapin, 2013-06-25 21:22:30]

Seems reasonable, so long as the tests still pass. :) LGTM

[pdr., 2013-06-25 21:54:25]

Committed patchset #3 manually as r153029 (presubmit successful).