Remove trailing commas from Chromium IDLs

Remove trailing commas from Chromium IDLs

This removes trailing commas in extended attributes from the
2 files that have them.
This is not part of the standard, and doesn't work with strict
parsers, but laxer parsers may not notice.
This removes a barrier to using more standard-compliant parsers
more widely; discovered when rewriting Blink IDL parser.

BUG=242800
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=201779

[Nils Barth (inactive), 2013-05-22 07:19:03]

Hiyas,
A very minor fix -- trailing commas are not allowed in Web IDL grammar, so
they'll cause syntax error if we move to a stricter parser.
This removes the very few places in the Chromium codebase (outside Blink) where
I found them.

[slightlyoff, 2013-05-22 08:25:31]

Have you verified that npchrome_frame.dll still links with this change? If so,
LGTM. If not, please verify.


On 2013/05/22 07:19:03, Nils Barth wrote:
> Hiyas,
> A very minor fix -- trailing commas are not allowed in Web IDL grammar, so
> they'll cause syntax error if we move to a stricter parser.
> This removes the very few places in the Chromium codebase (outside Blink)
where
> I found them.

[Nils Barth (inactive), 2013-05-22 08:34:48]

On 2013/05/22 08:25:31, slightlyoff wrote:
> Have you verified that npchrome_frame.dll still links with this change? If so,
> LGTM. If not, please verify.

Thanks Alex!
I haven't verified on Windows (developing on Linux);
is there a try bot I can use to check this?
(Is win_layout_rel enough?)

[slightlyoff, 2013-05-22 09:10:23]

Robert would know, but I can try this change for you later today.
On 22 May 2013 09:34, <nbarth@chromium.org> wrote:

> On 2013/05/22 08:25:31, slightlyoff wrote:
>
>> Have you verified that npchrome_frame.dll still links with this change?
>> If so,
>> LGTM. If not, please verify.
>>
>
> Thanks Alex!
> I haven't verified on Windows (developing on Linux);
> is there a try bot I can use to check this?
> (Is win_layout_rel enough?)
>
>
https://codereview.chromium.**org/15679004/<https://codereview.chromium.org/1...
>

[Nils Barth (inactive), 2013-05-22 10:26:52]

On 2013/05/22 09:10:23, slightlyoff wrote:
> Robert would know, but I can try this change for you later today.

Thanks Alex -- probably safest that way.

[grt (UTC plus 2), 2013-05-22 12:57:59]

On 2013/05/22 08:34:48, Nils Barth wrote:
> On 2013/05/22 08:25:31, slightlyoff wrote:
> > Have you verified that npchrome_frame.dll still links with this change? If
so,
> > LGTM. If not, please verify.
> 
> Thanks Alex!
> I haven't verified on Windows (developing on Linux);
> is there a try bot I can use to check this?
> (Is win_layout_rel enough?)

win, win_rel, and win_cf will all build GCF and run the various tests. I've just
sent the change to the win bot to see how it goes. Otherwise, LGTM. Thanks for
taking care of this.

[slightlyoff, 2013-05-22 13:04:58]

Links fine. LGTM++

On 2013/05/22 12:57:59, grt wrote:
> On 2013/05/22 08:34:48, Nils Barth wrote:
> > On 2013/05/22 08:25:31, slightlyoff wrote:
> > > Have you verified that npchrome_frame.dll still links with this change? If
> so,
> > > LGTM. If not, please verify.
> > 
> > Thanks Alex!
> > I haven't verified on Windows (developing on Linux);
> > is there a try bot I can use to check this?
> > (Is win_layout_rel enough?)
> 
> win, win_rel, and win_cf will all build GCF and run the various tests. I've
just
> sent the change to the win bot to see how it goes. Otherwise, LGTM. Thanks for
> taking care of this.

[commit-bot: I haz the power, 2013-05-23 02:12:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/15679004/1

[commit-bot: I haz the power, 2013-05-23 02:27:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/15679004/22001

[commit-bot: I haz the power, 2013-05-23 13:11:06]

Change committed as 201779