Summary: | Clean up SVGPatternElement::collectPatternAttributes | ||
---|---|---|---|
Product: | WebKit | Reporter: | Gyuyoung Kim <gyuyoung.kim> |
Component: | SVG | Assignee: | Gyuyoung Kim <gyuyoung.kim> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | buildbot, commit-queue, d-r, fmalita, gyuyoung.kim, pdr, rniwa, schenney, zimmermann |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Attachments: |
Description
Gyuyoung Kim
2014-01-25 00:12:38 PST
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. |