Mojo: Mojom: Make specialized handle types (e.g., message_pipe) not keywords.

Mojo: Mojom: Make specialized handle types (e.g., message_pipe) not keywords.

Other types (e.g., int32) aren't keywords, so it's a bit anomalous that
the specialized handle types are. (Insisting that they be keywords makes
adding future handle types more difficult, since doing so could break
existing .mojom files.)

R=darin@chromium.org

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

[darin (slow to review), 2014-05-19 21:14:33]

LGTM

https://codereview.chromium.org/295773002/diff/1/mojo/public/tools/bindings/p...
File mojo/public/tools/bindings/pylib/mojom/parse/parser.py (right):

https://codereview.chromium.org/295773002/diff/1/mojo/public/tools/bindings/p...
mojo/public/tools/bindings/pylib/mojom/parse/parser.py:205: if p[3] not in
('data_pipe_consumer', 'data_pipe_producer',
nit: you might list these one per line so that future edits are less disruptive
history wise.

[viettrungluu, 2014-05-19 21:22:34]

Thanks.

https://codereview.chromium.org/295773002/diff/1/mojo/public/tools/bindings/p...
File mojo/public/tools/bindings/pylib/mojom/parse/parser.py (right):

https://codereview.chromium.org/295773002/diff/1/mojo/public/tools/bindings/p...
mojo/public/tools/bindings/pylib/mojom/parse/parser.py:205: if p[3] not in
('data_pipe_consumer', 'data_pipe_producer',
On 2014/05/19 21:14:33, darin wrote:
> nit: you might list these one per line so that future edits are less
disruptive
> history wise.

Done.

[viettrungluu, 2014-05-19 21:22:39]

The CQ bit was checked by viettrungluu@chromium.org

[commit-bot: I haz the power, 2014-05-19 21:23:20]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/295773002/10001

[viettrungluu, 2014-05-20 00:03:21]

Committed patchset #2 manually as r271519 (presubmit successful).