Patch incoming.
Created attachment 157819 [details] A patch apart
Attachment 157819 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/rendering/style/NinePieceImage.h:61: The parameter name "o" adds no information, so it should be removed. [readability/parameter_name] [5] Source/WebCore/rendering/style/NinePieceImage.h:95: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] Total errors found: 2 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 157819 [details] A patch apart View in context: https://bugs.webkit.org/attachment.cgi?id=157819&action=review > Source/WebCore/rendering/style/NinePieceImage.h:37 > -class NinePieceImage { > +class NinePieceImageData { I think it would be better to use the normal RenderStyle DataRef<>/SET_VAR pattern here. The advantage is that we can share the structure in cases where it already exists and some other surround property is modified. It is also more consistent. We should also have separate version of SET_VAR for these two-level data structure cases as currently we end up creating unnecessary structures when a variable is set to the current value.
Comment on attachment 157819 [details] A patch apart Actually DataRefs in substructures don't make sense for non-inherited data as there is never anything to share (except the default value). Forgot what I mumbled above, the patch is fine.
It might be good idea to see if NinePieceImages are so rare compared to other properties in SurroundData it should go to rare data instead.
Created attachment 158101 [details] Patch for moon landing
Comment on attachment 158101 [details] Patch for moon landing Clearing flags on attachment: 158101 Committed r125465: <http://trac.webkit.org/changeset/125465>
All reviewed patches have been landed. Closing bug.