This patch refactors SVGPatternElement::collectPatternAttributes() by extracting the setPatternAttributes logic into a new function. This patch also changes the while loop to be easier to understand and similar to collectGradientAttributes.
Created attachment 222195 [details] Patch
Though I think this patch isn't big improvement, this patch will help to understand code readability.
Comment on attachment 222195 [details] Patch Attachment 222195 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/5436325394120704 New failing tests: svg/custom/js-update-pattern.svg svg/dynamic-updates/SVGPatternElement-dom-width-attr.html svg/custom/js-update-pattern-child.svg svg/dynamic-updates/SVGPatternElement-dom-patternUnits-attr.html http/tests/misc/slow-loading-image-in-pattern.html svg/W3C-SVG-1.1/pservers-pattern-01-b.svg svg/W3C-SVG-1.1/pservers-grad-06-b.svg svg/custom/absolute-root-position-masking.xhtml svg/transforms/text-with-pattern-with-svg-transform.svg svg/transforms/text-with-pattern-inside-transformed-html.xhtml svg/dynamic-updates/SVGPatternElement-dom-x-attr.html svg/W3C-SVG-1.1/pservers-grad-03-b.svg svg/dom/viewport-container-crash.svg svg/W3C-SVG-1.1-SE/pservers-pattern-04-f.svg svg/custom/js-late-pattern-and-object-creation.svg svg/animations/animateTransform-pattern-transform.html svg/dynamic-updates/SVGPatternElement-dom-height-attr.html svg/animations/svgenum-animation-1.html svg/css/composite-shadow-text.svg svg/stroke/non-scaling-stroke-pattern.svg svg/W3C-SVG-1.1/coords-units-01-b.svg svg/custom/deep-dynamic-updates.svg svg/dynamic-updates/SVGPatternElement-dom-patternTransform-attr.html svg/batik/paints/patternRegionA.svg svg/W3C-SVG-1.1-SE/pservers-pattern-03-f.svg svg/batik/paints/patternRegions.svg svg/batik/paints/patternPreserveAspectRatioA.svg svg/dynamic-updates/SVGPatternElement-dom-patternContentUnits-attr.html svg/transforms/transformed-text-fill-pattern.html svg/custom/js-late-pattern-creation.svg
Created attachment 222198 [details] Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-15 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Comment on attachment 222195 [details] Patch Attachment 222195 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/4790797513785344 New failing tests: svg/custom/js-update-pattern.svg svg/dynamic-updates/SVGPatternElement-dom-width-attr.html svg/custom/js-update-pattern-child.svg svg/dynamic-updates/SVGPatternElement-dom-patternUnits-attr.html http/tests/misc/slow-loading-image-in-pattern.html svg/W3C-SVG-1.1/pservers-pattern-01-b.svg svg/W3C-SVG-1.1/pservers-grad-06-b.svg svg/custom/absolute-root-position-masking.xhtml svg/transforms/text-with-pattern-with-svg-transform.svg svg/transforms/text-with-pattern-inside-transformed-html.xhtml svg/dynamic-updates/SVGPatternElement-dom-x-attr.html svg/W3C-SVG-1.1/pservers-grad-03-b.svg svg/dom/viewport-container-crash.svg svg/W3C-SVG-1.1-SE/pservers-pattern-04-f.svg svg/custom/js-late-pattern-and-object-creation.svg svg/animations/animateTransform-pattern-transform.html svg/dynamic-updates/SVGPatternElement-dom-height-attr.html svg/animations/svgenum-animation-1.html svg/css/composite-shadow-text.svg svg/stroke/non-scaling-stroke-pattern.svg svg/W3C-SVG-1.1/coords-units-01-b.svg svg/custom/deep-dynamic-updates.svg svg/dynamic-updates/SVGPatternElement-dom-patternTransform-attr.html svg/batik/paints/patternRegionA.svg svg/W3C-SVG-1.1-SE/pservers-pattern-03-f.svg svg/batik/paints/patternRegions.svg svg/batik/paints/patternPreserveAspectRatioA.svg svg/dynamic-updates/SVGPatternElement-dom-patternContentUnits-attr.html svg/transforms/transformed-text-fill-pattern.html svg/custom/js-late-pattern-creation.svg
Created attachment 222200 [details] Archive of layout-test-results from webkit-ews-03 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-03 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Comment on attachment 222195 [details] Patch Attachment 222195 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5153050087915520 New failing tests: svg/custom/js-update-pattern.svg svg/dynamic-updates/SVGPatternElement-dom-width-attr.html svg/custom/js-update-pattern-child.svg svg/dynamic-updates/SVGPatternElement-dom-patternUnits-attr.html http/tests/misc/slow-loading-image-in-pattern.html svg/W3C-SVG-1.1/pservers-pattern-01-b.svg svg/W3C-SVG-1.1/pservers-grad-06-b.svg svg/custom/absolute-root-position-masking.xhtml svg/transforms/text-with-pattern-with-svg-transform.svg svg/transforms/text-with-pattern-inside-transformed-html.xhtml svg/dynamic-updates/SVGPatternElement-dom-x-attr.html svg/W3C-SVG-1.1/pservers-grad-03-b.svg svg/dom/viewport-container-crash.svg svg/W3C-SVG-1.1-SE/pservers-pattern-04-f.svg svg/custom/js-late-pattern-and-object-creation.svg svg/animations/animateTransform-pattern-transform.html svg/dynamic-updates/SVGPatternElement-dom-height-attr.html svg/animations/svgenum-animation-1.html svg/css/composite-shadow-text.svg svg/stroke/non-scaling-stroke-pattern.svg svg/W3C-SVG-1.1/coords-units-01-b.svg svg/custom/deep-dynamic-updates.svg svg/dynamic-updates/SVGPatternElement-dom-patternTransform-attr.html svg/batik/paints/patternRegionA.svg svg/W3C-SVG-1.1-SE/pservers-pattern-03-f.svg svg/batik/paints/patternRegions.svg svg/batik/paints/patternPreserveAspectRatioA.svg svg/dynamic-updates/SVGPatternElement-dom-patternContentUnits-attr.html svg/transforms/transformed-text-fill-pattern.html svg/custom/js-late-pattern-creation.svg
Created attachment 222203 [details] Archive of layout-test-results from webkit-ews-07 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-07 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Created attachment 222209 [details] Patch
Comment on attachment 222209 [details] Patch Clearing flags on attachment: 222209 Committed r162792: <http://trac.webkit.org/changeset/162792>
All reviewed patches have been landed. Closing bug.