Some minor touch-ups to the SVG content-model test framework

Some minor touch-ups to the SVG content-model test framework

* Update references to filter spec. and make feDropShadow a proper
  filter primitive.
* Rename needChld to needChild.
* Sync tags table and classes with SVG2.
* Minor simplifications to class expansion and child-creation.

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

[fs, 2015-03-13 11:27:01]

fs@opera.com changed reviewers:
+ ed@opera.com

[Erik Dahlström (inactive), 2015-03-13 11:52:55]

lgtm

[fs, 2015-03-13 12:03:05]

On 2015/03/13 11:52:55, Erik Dahlström wrote:
> lgtm

Synced the tags table w/ SVG2 a bit too. PTAL

[Erik Dahlström (inactive), 2015-03-13 12:28:38]

Some nits re patchset #2.

The code changes are still lgtm.

https://codereview.chromium.org/1006863002/diff/20001/LayoutTests/svg/dom/res...
File LayoutTests/svg/dom/resources/svgtags.js (right):

https://codereview.chromium.org/1006863002/diff/20001/LayoutTests/svg/dom/res...
LayoutTests/svg/dom/resources/svgtags.js:124: //
http://www.w3.org/TR/SVG/intro.html#TermLightSourceElement
Since you updated one of the links above, please make all filter links point to
the fxtf spec.

https://codereview.chromium.org/1006863002/diff/20001/LayoutTests/svg/dom/res...
LayoutTests/svg/dom/resources/svgtags.js:129: //
https://svgwg.org/svg2-draft/intro.html#TermPaintServerElement
Note that the term definitions in general have moved (or will be moved) to the
place where they're first used in the spec, this link is broken.

If you want to reference svg2 terms then please update the rest of the CLASS_*
links too.

[fs, 2015-03-13 12:30:05]

https://codereview.chromium.org/1006863002/diff/20001/LayoutTests/svg/dom/res...
File LayoutTests/svg/dom/resources/svgtags.js (right):

https://codereview.chromium.org/1006863002/diff/20001/LayoutTests/svg/dom/res...
LayoutTests/svg/dom/resources/svgtags.js:129: //
https://svgwg.org/svg2-draft/intro.html#TermPaintServerElement
On 2015/03/13 12:28:38, Erik Dahlström wrote:
> Note that the term definitions in general have moved (or will be moved) to the
> place where they're first used in the spec, this link is broken.

Heh, yeah I noticed, and was just about to mention it...

> If you want to reference svg2 terms then please update the rest of the CLASS_*
> links too.

Ok.

[fs, 2015-03-13 12:53:52]

Updated term URLs and synced the classes a bit. If you know of any additional
classes that would be good to have, let me know and I'll add them.

[Erik Dahlström (inactive), 2015-03-13 13:52:27]

On 2015/03/13 12:53:52, fs wrote:
> Updated term URLs and synced the classes a bit. If you know of any additional
> classes that would be good to have, let me know and I'll add them.

Looks fine still.

Classes for text elements maybe, see
https://svgwg.org/svg2-draft/text.html#Introduction.

[fs, 2015-03-13 13:59:15]

On 2015/03/13 13:52:27, Erik Dahlström wrote:
> On 2015/03/13 12:53:52, fs wrote:
> > Updated term URLs and synced the classes a bit. If you know of any
additional
> > classes that would be good to have, let me know and I'll add them.
> 
> Looks fine still.
> 
> Classes for text elements maybe, see
> https://svgwg.org/svg2-draft/text.html#Introduction.

Added.

https://codereview.chromium.org/1006863002/diff/40001/LayoutTests/svg/dom/res...
File LayoutTests/svg/dom/resources/svgtags.js (right):

https://codereview.chromium.org/1006863002/diff/40001/LayoutTests/svg/dom/res...
LayoutTests/svg/dom/resources/svgtags.js:96: 'a', 'defs', 'glyph', 'g',
'marker', 'mask', 'missing-glyph', 'pattern', 'svg', 'switch',
Forgot to clean some of these out... Fixed.

[fs, 2015-03-13 13:59:20]

The CQ bit was checked by fs@opera.com

[fs, 2015-03-13 13:59:21]

The patchset sent to the CQ was uploaded after l-g-t-m from ed@opera.com
Link to the patchset: https://codereview.chromium.org/1006863002/#ps60001
(title: "Add text element classes. Drop some dead containers.")

[commit-bot: I haz the power, 2015-03-13 13:59:32]

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

[commit-bot: I haz the power, 2015-03-13 18:12:00]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191856