Bug 80413 - Add SVGRenderingContext and move there the context related code from SVGRenderSupport
Summary: Add SVGRenderingContext and move there the context related code from SVGRende...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-06 05:54 PST by Zoltan Herczeg
Modified: 2012-03-09 21:23 PST (History)
5 users (show)

See Also:


Attachments
patch (38.46 KB, patch)
2012-03-06 06:10 PST, Zoltan Herczeg
zimmermann: review+
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
fixing EWS failures (39.20 KB, patch)
2012-03-08 06:26 PST, Zoltan Herczeg
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
another attempt (38.54 KB, patch)
2012-03-09 03:03 PST, Zoltan Herczeg
zimmermann: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zoltan Herczeg 2012-03-06 05:54:07 PST
First step of refactoring this part.
Comment 1 Zoltan Herczeg 2012-03-06 06:10:05 PST
Created attachment 130368 [details]
patch
Comment 2 Nikolas Zimmermann 2012-03-06 07:07:41 PST
Comment on attachment 130368 [details]
patch

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

Looks great! r=me, assuming no regressions.

> Source/WebCore/rendering/svg/SVGRenderingContext.h:38
> +#if ENABLE(FILTERS)

No need to guard class forwards IIRC.

> Source/WebCore/rendering/svg/SVGRenderingContext.h:75
> +    bool isRenderingPrepared() { return m_renderingFlags & RenderingPrepared; }

Could be const.

> Source/WebCore/rendering/svg/SVGRenderingContext.h:79
> +    // we record all successful steps.

No need to wrap lines here, eh?

> Source/WebCore/rendering/svg/SVGRenderingContext.h:91
> +        | EndOpacityLayer | EndShadowLayer | EndFilterLayer;

Ditto.
Comment 3 WebKit Review Bot 2012-03-06 07:17:10 PST
Comment on attachment 130368 [details]
patch

Attachment 130368 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11839392

New failing tests:
svg/filters/subRegion-in-userSpace.svg
svg/dynamic-updates/SVGFEColorMatrixElement-svgdom-values-prop.html
svg/W3C-SVG-1.1/filters-tile-01-b.svg
svg/dynamic-updates/SVGFEBlendElement-dom-mode-attr.html
svg/dynamic-updates/SVGFEColorMatrixElement-dom-type-attr.html
svg/dynamic-updates/SVGFEBlendElement-svgdom-mode-prop.html
Comment 4 Zoltan Herczeg 2012-03-06 12:27:58 PST
I will check the failing tests tomorrow.
Comment 5 Zoltan Herczeg 2012-03-08 06:26:15 PST
Created attachment 130816 [details]
fixing EWS failures

Let's wait for the EWS.
Comment 6 WebKit Review Bot 2012-03-09 02:08:35 PST
Comment on attachment 130816 [details]
fixing EWS failures

Attachment 130816 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/11911471

New failing tests:
svg/dynamic-updates/SVGFECompositeElement-svgdom-k2-prop.html
svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-bias-attr.html
svg/dynamic-updates/SVGFECompositeElement-svgdom-k1-prop.html
svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-divisor-attr.html
svg/dynamic-updates/SVGFEColorMatrixElement-dom-values-attr.html
http/tests/inspector/inspect-element.html
svg/dynamic-updates/SVGFECompositeElement-dom-k3-attr.html
accessibility/aria-disabled.html
fast/loader/text-document-wrapping.html
svg/dynamic-updates/SVGFECompositeElement-svgdom-operator-prop.html
svg/dynamic-updates/SVGFECompositeElement-dom-operator-attr.html
svg/dynamic-updates/SVGFECompositeElement-dom-k1-attr.html
svg/dynamic-updates/SVGFEBlendElement-dom-mode-attr.html
compositing/reflections/reflection-ordering.html
svg/dynamic-updates/SVGFEConvolveMatrixElement-dom-edgeMode-attr.html
svg/dynamic-updates/SVGFECompositeElement-svgdom-k3-prop.html
svg/dynamic-updates/SVGFEColorMatrixElement-dom-type-attr.html
svg/dynamic-updates/SVGFECompositeElement-dom-k2-attr.html
http/tests/xmlhttprequest/xmlhttprequest-unsafe-redirect.html
svg/dynamic-updates/SVGFEColorMatrixElement-svgdom-values-prop.html
svg/dynamic-updates/SVGFEColorMatrixElement-svgdom-type-prop.html
compositing/masks/direct-image-mask.html
svg/dynamic-updates/SVGFECompositeElement-dom-k4-attr.html
css3/filters/effect-contrast-hw.html
svg/dynamic-updates/SVGFECompositeElement-svgdom-k4-prop.html
svg/dynamic-updates/SVGFEBlendElement-svgdom-mode-prop.html
Comment 7 Nikolas Zimmermann 2012-03-09 02:53:12 PST
(In reply to comment #6)
Your patch looks nice, can you confirm the EWS results are correct, and stuff got broken?
Comment 8 Zoltan Herczeg 2012-03-09 03:03:24 PST
Created attachment 131020 [details]
another attempt
Comment 9 Zoltan Herczeg 2012-03-09 03:05:12 PST
> Your patch looks nice, can you confirm the EWS results are correct, and stuff got broken?

Yeah. Still the return value of filter initialization. I misunderstood it. False means you don't need to draw anyithing, not initialization failed. Rather confusing :(
Comment 10 Nikolas Zimmermann 2012-03-09 03:59:47 PST
Comment on attachment 131020 [details]
another attempt

The cr-linux failures now look unrelated, r=me. The previously failing tests are all gone, so we think this is safe.
Comment 11 Zoltan Herczeg 2012-03-09 04:04:03 PST
Landed as http://trac.webkit.org/changeset/110285
Comment 12 Patrick R. Gansterer 2012-03-09 21:23:19 PST
Landed build fix in http://trac.webkit.org/changeset/110319