Factor out common code from RenderSVGResource*::postApplyResource

Factor out common code from RenderSVGResource*::postApplyResource

Factor the fill/stroke code out of the various paint-server
RenderSVGResource subclasses and move it to SVGRenderSupport.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181907

[fs, 2014-09-12 12:02:55]

fs@opera.com changed reviewers:
+ fmalita@chromium.org, pdr@chromium.org

[kouhei (in TOK), 2014-09-12 14:07:08]

lgtm

[f(malita), 2014-09-12 14:10:35]

lgtm!

(i've been thinking we need to refine the resource type hierarchy:
gradient/pattern/solidcolor/markers really are used differently from
clips/masks/filters. in a refactored hierarchy maybe fillOrStrokePrimitive would
belong in a gradient/pattern/solidcolor superclass)

[fs, 2014-09-12 14:32:09]

On 2014/09/12 14:10:35, Florin Malita wrote:
> lgtm!
> 
> (i've been thinking we need to refine the resource type hierarchy:
> gradient/pattern/solidcolor/markers really are used differently from
> clips/masks/filters. in a refactored hierarchy maybe fillOrStrokePrimitive
would
> belong in a gradient/pattern/solidcolor superclass)

Yes, I've been strolling along the same lines (again...) I've not yet come up
with a nice, decomposable way to get there though. Also, I don't think
fillOrStrokePrimitive will survive that long ;-)

[fs, 2014-09-12 14:32:19]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2014-09-12 14:32:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/571463002/1

[commit-bot: I haz the power, 2014-09-12 14:37:51]

Committed patchset #1 (id:1) as 181907