nephele
/
haikuwebkit
1
0
Fork 0
Code Issues Releases Activity
haikuwebkit/LayoutTests/plugins/plugin-remove-readystatecha...

3 lines
206 B
Plaintext
Raw Permalink Normal View History

Source/WebCore: ASSERT(!eventDispatchForbidden()) fires when removed plugin re-inserted as part of readyStateChange. https://bugs.webkit.org/show_bug.cgi?id=93639 Reviewed by Ryosuke Niwa. Removing a plugin causes a detach which can cancel the last remaining load on a page, resulting in a readyStateChange event during a time when things are inconsisent. Defer the detach which triggers this chain of events until after the node is fully removed from the document's elementsById map. Test: plugins/plugin-remove-readystatechange.html * dom/ContainerNode.cpp: (WebCore::ContainerNode::removeChild): (WebCore::ContainerNode::removeChildren): LayoutTests: ASSERT(!eventDispatchForbidden()) firest when removed plugin re-inserted as part of readyStateChange. https://bugs.webkit.org/show_bug.cgi?id=93639 Reviewed by Ryosuke Niwa. Add a new testcase to cover this issue. Test passes if assert doesn't fire in debug builds. * plugins/plugin-remove-readystatechange-expected.txt: Added. * plugins/plugin-remove-readystatechange.html: Added. Canonical link: https://commits.webkit.org/114610@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@128524 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2012-09-13 23:12:03 +00:00
ALERT: PASS: element could not be re-appended
This test passes if it does not trip an assert in debug builds. It ensures a readystatechange event can't get dispatched until after a plugin is fully removed.