Reduce the crazy number of parameters to RenderLayer painting member functions
Created attachment 173561 [details] Patch
Attachment 173561 [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/RenderReplica.cpp:76: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Source/WebCore/rendering/RenderReplica.cpp:77: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Source/WebCore/rendering/RenderLayer.cpp:3063: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] Source/WebCore/rendering/RenderLayer.cpp:3064: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] Source/WebCore/rendering/RenderLayer.cpp:3065: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] Source/WebCore/rendering/RenderLayer.cpp:3066: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] Total errors found: 6 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 173659 [details] Patch
I'm open to better names than LayerPaintingInfo. LayerPaintingContext (sounds too much like GraphicsContext)? LayerPaintingState (sounds like it would be pushed/popped like state)? PaintingInfo is too much like PaintInfo.
This caused a regression that was fixed via https://bugs.webkit.org/show_bug.cgi?id=102002.
http://trac.webkit.org/changeset/134311
<rdar://problem/12705354>