16% of total time loading html5 spec (PerformanceTests/Parser/html5-full-render.html) is spent under updateWidgetIfNecessary

• Component: WebKit Misc.
• Status: NEW ---

[Eric Seidel (no email)]

16% of total time loading html5 spec (PerformanceTests/Parser/html5-full-render.html) is spent under updateWidgetIfNecessary

Running (Self)		Symbol Name
4690.0ms   21.7%	 	             WebCore::HTMLConstructionSite::insertHTMLElement(WebCore::AtomicHTMLToken&)
4546.0ms   21.0%	 	              WTF::PassRefPtr<WebCore::Element> WebCore::HTMLConstructionSite::attach<WebCore::Element>(WebCore::ContainerNode*, WTF::PassRefPtr<WebCore::Element>)
4348.0ms   20.1%	 	               WebCore::Element::attach()
3491.0ms   16.1%	 	                WebCore::ContainerNode::resumePostAttachCallbacks()
3486.0ms   16.1%	 	                 WebCore::ContainerNode::dispatchPostAttachCallbacks()
3485.0ms   16.1%	 	                  WebCore::HTMLPlugInImageElement::updateWidgetIfNecessary()
3485.0ms   16.1%	 	                   WebCore::Document::updateStyleIfNeeded()
3485.0ms   16.1%	 	                    WebCore::Document::recalcStyle(WebCore::Node::StyleChange)

basically we're updating style more often than we need to due to *two* <object> tags in the spec.

Post attach-callbacks for the loss.

[Eric Seidel (no email)]

Created attachment 110814 [details]
Patch

[WebKit Review Bot]

Comment on attachment 110814 [details]
Patch

Attachment 110814 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10056165

New failing tests:
svg/custom/getsvgdocument.html

[Simon Fraser (smfr)]

Comment on attachment 110814 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=110814&action=review

> Source/WebCore/html/HTMLPlugInImageElement.cpp:171
>      if (!needsWidgetUpdate() || useFallbackContent() || isImageType())

Do any of these require that style has been updated?

[Adam Barth]

> Do any of these require that style has been updated?

I asked Eric that last night and he said no.

[Alexey Proskuryakov]

The test failure on EWS looks related to the change.

[Eric Seidel (no email)]

Must be one of them.  I will revert the change.

I guess that's why I shouldn't write changes at 2am. :)

[Adam Barth]

> I will revert the change.

The change never landed, as far as I can tell.

[Eric Seidel (no email)]

Comment on attachment 110814 [details]
Patch

This patch can't work it seems.  I'm not sure what race the current code is depending on, but removing this style update will cause the load never to get kicked off for an <object> tag, and thus not correctly delay the onload event for the window, causing the getSVGDocument() test to fail.

[Eric Seidel (no email)]

I spent some time investigating if I could disentangle the load from the renderer to no avail.  I'll come back to this, but after I finish NRWT.  This is part of the general idea of finally making display: none plugins load properly.