Add missing IDL parser support for 'stringifier readonly attribute'.

Add missing IDL parser support for 'stringifier readonly attribute'.

R=
BUG=306606
Committed: https://crrev.com/ffb5cc6ac6951e3b311b6d275da2d333f4e44243
Cr-Commit-Position: refs/heads/master@{#388484}

[sof, 2016-04-20 07:51:48]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, jl@opera.com

[sof, 2016-04-20 07:51:49]

please take a look.

[haraken, 2016-04-20 07:52:20]

LGTM

[sof, 2016-04-20 08:06:15]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-04-20 08:06:25]

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

[Jens Widell, 2016-04-20 08:09:22]

LGTM

[commit-bot: I haz the power, 2016-04-20 08:14:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-20 08:14:23]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[sof, 2016-04-20 08:19:59]

sigbjornf@opera.com changed reviewers:
+ thakis@chromium.org

[sof, 2016-04-20 08:19:59]

Who owns idl_parser/ more?

Adding thakis@ for tools/idl_parser/ approval.

[haraken, 2016-04-20 08:27:22]

c.f., adding tools/idl_parser/OWNERS:
https://codereview.chromium.org/1897293004/

[Jens Widell, 2016-04-20 08:32:07]

https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser.py
File tools/idl_parser/idl_parser.py (right):

https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser...
tools/idl_parser/idl_parser.py:563: elif p[1] != ';':
BTW, this ought to be dead code now; len(p) will be 3 in both cases.

[sof, 2016-04-20 08:49:27]

https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser.py
File tools/idl_parser/idl_parser.py (right):

https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser...
tools/idl_parser/idl_parser.py:563: elif p[1] != ';':
On 2016/04/20 08:32:07, Jens Widell wrote:
> BTW, this ought to be dead code now; len(p) will be 3 in both cases.

i wondered about that, but didn't take it further :)

[sof, 2016-04-20 11:42:51]

On 2016/04/20 08:49:27, sof wrote:
>
https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser.py
> File tools/idl_parser/idl_parser.py (right):
> 
>
https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser...
> tools/idl_parser/idl_parser.py:563: elif p[1] != ';':
> On 2016/04/20 08:32:07, Jens Widell wrote:
> > BTW, this ought to be dead code now; len(p) will be 3 in both cases.
> 
> i wondered about that, but didn't take it further :)

Addressed now.

[haraken, 2016-04-20 11:45:35]

On 2016/04/20 11:42:51, sof wrote:
> On 2016/04/20 08:49:27, sof wrote:
> >
>
https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser.py
> > File tools/idl_parser/idl_parser.py (right):
> > 
> >
>
https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser...
> > tools/idl_parser/idl_parser.py:563: elif p[1] != ';':
> > On 2016/04/20 08:32:07, Jens Widell wrote:
> > > BTW, this ought to be dead code now; len(p) will be 3 in both cases.
> > 
> > i wondered about that, but didn't take it further :)
> 
> Addressed now.

I guess you can now land with my LG.

[sof, 2016-04-20 11:47:19]

On 2016/04/20 11:45:35, haraken wrote:
> On 2016/04/20 11:42:51, sof wrote:
> > On 2016/04/20 08:49:27, sof wrote:
> > >
> >
>
https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser.py
> > > File tools/idl_parser/idl_parser.py (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/1900873006/diff/1/tools/idl_parser/idl_parser...
> > > tools/idl_parser/idl_parser.py:563: elif p[1] != ';':
> > > On 2016/04/20 08:32:07, Jens Widell wrote:
> > > > BTW, this ought to be dead code now; len(p) will be 3 in both cases.
> > > 
> > > i wondered about that, but didn't take it further :)
> > 
> > Addressed now.
> 
> I guess you can now land with my LG.

Yes, thanks (x2)

[sof, 2016-04-20 11:47:37]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-04-20 11:47:38]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, jl@opera.com
Link to the patchset: https://codereview.chromium.org/1900873006/#ps20001
(title: "remove dead code")

[commit-bot: I haz the power, 2016-04-20 11:47:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1900873006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1900873006/20001

[commit-bot: I haz the power, 2016-04-20 13:18:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:23:03]

Description was changed from

==========
Add missing IDL parser support for 'stringifier readonly attribute'.

R=
BUG=306606
==========

to

==========
Add missing IDL parser support for 'stringifier readonly attribute'.

R=
BUG=306606

Committed: https://crrev.com/ffb5cc6ac6951e3b311b6d275da2d333f4e44243
Cr-Commit-Position: refs/heads/master@{#388484}
==========

[commit-bot: I haz the power, 2016-04-22 19:23:03]

Patchset 2 (id:??) landed as
https://crrev.com/ffb5cc6ac6951e3b311b6d275da2d333f4e44243
Cr-Commit-Position: refs/heads/master@{#388484}