Bug 121936 – Move m_style to RenderElement

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Antti Koivisto]

Text renderers always have the same style as their parent.

[Antti Koivisto]

Created attachment 212661 [details]
for bots

[Early Warning System Bot]

Comment on attachment 212661 [details]
for bots

Attachment 212661 [details] did not pass qt-ews (qt):
Output: http://webkit-queues.appspot.com/results/2230134

[Early Warning System Bot]

Comment on attachment 212661 [details]
for bots

Attachment 212661 [details] did not pass qt-wk2-ews (qt-wk2):
Output: http://webkit-queues.appspot.com/results/2186147

[Antti Koivisto]

Created attachment 212666 [details]
another

[WebKit Commit Bot]

Attachment 212666 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCore/css/CSSGradientValue.cpp', u'Source/WebCore/dom/PseudoElement.cpp', u'Source/WebCore/editing/ApplyBlockElementCommand.cpp', u'Source/WebCore/editing/HTMLInterchange.cpp', u'Source/WebCore/editing/InsertLineBreakCommand.cpp', u'Source/WebCore/rendering/RenderBlock.cpp', u'Source/WebCore/rendering/RenderButton.cpp', u'Source/WebCore/rendering/RenderCombineText.cpp', u'Source/WebCore/rendering/RenderElement.cpp', u'Source/WebCore/rendering/RenderElement.h', u'Source/WebCore/rendering/RenderImageResource.cpp', u'Source/WebCore/rendering/RenderImageResourceStyleImage.cpp', u'Source/WebCore/rendering/RenderInline.cpp', u'Source/WebCore/rendering/RenderMenuList.cpp', u'Source/WebCore/rendering/RenderObject.cpp', u'Source/WebCore/rendering/RenderObject.h', u'Source/WebCore/rendering/RenderRegion.cpp', u'Source/WebCore/rendering/RenderSearchField.cpp', u'Source/WebCore/rendering/RenderText.cpp', u'Source/WebCore/rendering/RenderText.h', u'Source/WebCore/rendering/mathml/RenderMathMLOperator.cpp', u'Source/WebCore/rendering/style/ContentData.cpp', u'Source/WebCore/style/StyleResolveTree.cpp', u'Source/WebKit/mac/WebView/WebHTMLRepresentation.mm']" exit_code: 1
Source/WebCore/rendering/RenderImageResource.cpp:34:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 1 in 20 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Build Bot]

Comment on attachment 212666 [details]
another

Attachment 212666 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/2327002

New failing tests:
svg/custom/text-ctm.svg
svg/custom/SVGPoint-matrixTransform.svg
fast/text/decorations-with-text-combine.html
compositing/contents-opaque/visibility-hidden.html
svg/custom/text-hit-test.svg
svg/custom/getTransformToElement.svg
compositing/contents-opaque/hidden-with-visible-text.html
svg/text/append-text-node-to-tspan.html

[Build Bot]

Created attachment 212669 [details]
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-10  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5

[Build Bot]

Comment on attachment 212666 [details]
another

Attachment 212666 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/2169254

New failing tests:
svg/custom/text-ctm.svg
svg/custom/SVGPoint-matrixTransform.svg
fast/text/decorations-with-text-combine.html
compositing/contents-opaque/visibility-hidden.html
svg/custom/text-hit-test.svg
svg/custom/getTransformToElement.svg
compositing/contents-opaque/hidden-with-visible-text.html
svg/text/append-text-node-to-tspan.html

[Build Bot]

Created attachment 212670 [details]
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-08  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Build Bot]

Comment on attachment 212666 [details]
another

Attachment 212666 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/2340008

New failing tests:
svg/custom/text-ctm.svg
svg/custom/SVGPoint-matrixTransform.svg
fast/text/decorations-with-text-combine.html
compositing/contents-opaque/visibility-hidden.html
svg/custom/text-hit-test.svg
svg/custom/getTransformToElement.svg
compositing/contents-opaque/hidden-with-visible-text.html
svg/text/append-text-node-to-tspan.html

[Build Bot]

Created attachment 212673 [details]
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-06  Port: mac-mountainlion  Platform: Mac OS X 10.8.5

[Antti Koivisto]

Created attachment 212776 [details]
patch

[Andreas Kling]

Comment on attachment 212776 [details]
patch

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

This looks great to me!

> Source/WebCore/ChangeLog:66
> +            Change assert. This can get legitimely called during layout for first letter renderers.

Typo, legitimately

> LayoutTests/ChangeLog:8
> +        Progressions from text style updating correctly in all cases.

I'm gonna take your word for this.

[Antti Koivisto]

Created attachment 212779 [details]
include WebKit changes too

[Antti Koivisto]

https://trac.webkit.org/r156527

[Ryosuke Niwa]

Looks like this caused ~3% regression on SVG/SvgCubics:Time?

https://perf.webkit.org/#mode=charts&chartList=%5B%5B%22mac-lion%22%2C%22SVG%2FSvgCubics%3ATime%22%5D%2C%5B%22mac-mountainlion%22%2C%22SVG%2FSvgCubics%3ATime%22%5D%5D&days=56&zoom=%5B1379938406879.8728%2C1380489114314.355%5D

[Roger Fong]

Just noticed that the following test is failing on Windows:
fast\forms\button-first-line-first-letter.html

Assertion failure:     ASSERT(innerStyle->refCount() == 1);

>	WebKit.dll!WebCore::RenderButton::setupInnerStyle(WebCore::RenderStyle * innerStyle)  Line 129	C++
 	WebKit.dll!WebCore::RenderButton::styleDidChange(WebCore::StyleDifference diff, const WebCore::RenderStyle * oldStyle)  Line 111	C++
 	WebKit.dll!WebCore::RenderElement::setStyle(WTF::PassRefPtr<WebCore::RenderStyle> style)  Line 398	C++
 	WebKit.dll!WebCore::RenderElement::setAnimatableStyle(WTF::PassRefPtr<WebCore::RenderStyle> style)  Line 436	C++
 	WebKit.dll!WebCore::Style::resolveLocal(WebCore::Element & current, WebCore::Style::Change inheritedChange)  Line 604 + 0x19 bytes	C++
 	WebKit.dll!WebCore::Style::resolveTree(WebCore::Element & current, WebCore::Style::Change change)  Line 740 + 0xd bytes	C++
 	WebKit.dll!WebCore::Style::resolveTree(WebCore::Element & current, WebCore::Style::Change change)  Line 772 + 0xd bytes	C++
 	WebKit.dll!WebCore::Style::resolveTree(WebCore::Element & current, WebCore::Style::Change change)  Line 772 + 0xd bytes	C++
 	WebKit.dll!WebCore::Style::resolveTree(WebCore::Document & document, WebCore::Style::Change change)  Line 812 + 0xd bytes	C++
 	WebKit.dll!WebCore::Document::recalcStyle(WebCore::Style::Change change)  Line 1729 + 0xd bytes	C++

[Darin Adler]

Lets make a new bug report about that Windows test failure. We should diagnose and fix it!

[Roger Fong]

Filed https://bugs.webkit.org/show_bug.cgi?id=123005 for the Windows failure.