Disable most MathML attributes

Disable most MathML attributes

The HTML parser only needs the encoding attribute to
parse MathML content, so do not generate the
other MathML attribute strings to save a bit in binary
size and startup time.

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

[rwlbuis, 2014-09-15 21:26:57]

rob.buis@samsung.com changed reviewers:
+ eseidel@chromium.org

[rwlbuis, 2014-09-15 21:26:58]

PTAL

[eseidel, 2014-09-15 22:14:39]

The parser changes are that we're no longer re-writing definitionURL to the
proper case?  Is that intentional?

[rwlbuis, 2014-09-15 22:39:08]

On 2014/09/15 22:14:39, eseidel wrote:
> The parser changes are that we're no longer re-writing definitionURL to the
> proper case?  Is that intentional?

Oops I forgot about that code! Patch set #2 keeps definitionURL.

[eseidel, 2014-09-16 06:17:45]

I would just delete them all instead of commenting them out.  With the way the
trends of the web are going these days, I think it's unlikely we'll ever
implement MathML in Blink directly, but rather expose all the necessary
primitives so that anyone can implement something as complex as MathML on top of
blink.

Thanks for doing this.  lgtm.

[rwlbuis, 2014-09-16 14:01:03]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-09-16 14:01:25]

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

[commit-bot: I haz the power, 2014-09-16 15:08:02]

Committed patchset #4 (id:60001) as 182055