diff --git a/LayoutTests/fast/dom/HTMLBodyElement/body-inserting-iframe-crash-expected.txt b/LayoutTests/fast/dom/HTMLBodyElement/body-inserting-iframe-crash-expected.txt new file mode 100644 index 00000000000..fbfeed92c6e --- /dev/null +++ b/LayoutTests/fast/dom/HTMLBodyElement/body-inserting-iframe-crash-expected.txt @@ -0,0 +1 @@ +PASS if not crashed. diff --git a/LayoutTests/fast/dom/HTMLBodyElement/body-inserting-iframe-crash.html b/LayoutTests/fast/dom/HTMLBodyElement/body-inserting-iframe-crash.html new file mode 100644 index 00000000000..35cece8d531 --- /dev/null +++ b/LayoutTests/fast/dom/HTMLBodyElement/body-inserting-iframe-crash.html @@ -0,0 +1,25 @@ + + + diff --git a/Source/core/html/HTMLBodyElement.cpp b/Source/core/html/HTMLBodyElement.cpp index 17a8b8edd18..f7e7f72afe3 100644 --- a/Source/core/html/HTMLBodyElement.cpp +++ b/Source/core/html/HTMLBodyElement.cpp @@ -155,22 +155,24 @@ void HTMLBodyElement::parseAttribute(const QualifiedName& name, const AtomicStri Node::InsertionNotificationRequest HTMLBodyElement::insertedInto(ContainerNode* insertionPoint) { HTMLElement::insertedInto(insertionPoint); - if (insertionPoint->inDocument()) { - // FIXME: It's surprising this is web compatible since it means a marginwidth - // and marginheight attribute can magically appear on the of all documents - // embedded through