Introduce new entry-points for non-paint-server RenderSVGResources

Introduce new entry-points for non-paint-server RenderSVGResources

The applyResource/postApplyResource entry-points defined in the
RenderSVGResource interface are not of much use to non-paint-server
resource-types (clipPath, mask, filter, marker), since they share
very little semantic-wise. This also mean that they impede changes
that could benefit the "other" type of resource.
Make the mentioned methods be used only for the paint-server
resources, and move the current functionality for mask/filter
(clipPath already has an interface of its own, as does marker)
to methods with the names prepareEffect/finishEffect.

This removes all actual uses of the RenderSVGResourceMode
ApplyToDefaultMode "flag", so drop it.

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

[fs, 2014-10-03 16:28:23]

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

[fs, 2014-10-03 17:06:23]

Bikeshed territory...

[pdr., 2014-10-03 17:22:53]

On 2014/10/03 at 17:06:23, fs wrote:
> Bikeshed territory...

I think this looks good and is the right approach. LGTM

(here, or in a followup) What do you think of renaming applyResource to
applyPaintServerResource or similar?

[fs, 2014-10-06 07:58:40]

On 2014/10/03 17:22:53, pdr wrote:
> On 2014/10/03 at 17:06:23, fs wrote:
> > Bikeshed territory...
> 
> I think this looks good and is the right approach. LGTM
> 
> (here, or in a followup) What do you think of renaming applyResource to
> applyPaintServerResource or similar?

On my work-branch for this issue it's currently named "preparePaintServer" (the
"post..." method disappears).

[fs, 2014-10-06 09:31:53]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2014-10-06 09:32:28]

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

[commit-bot: I haz the power, 2014-10-06 11:14:20]

Committed patchset #1 (id:1) as 183261