Remove 'char' from IDL

Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715
Committed: https://crrev.com/509e71d7781b783c01728263fe9ffd52123cdbe3
Cr-Commit-Position: refs/heads/master@{#423524}

[Yeol Park, 2016-10-05 04:58:49]

Description was changed from

==========
Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715
==========

to

==========
Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715
==========

[Yeol Park, 2016-10-05 04:58:49]

peary2@gmail.com changed reviewers:
+ bbudge@chromium.org, noelallen@chromium.org, raymes@chromium.org

[Yeol Park, 2016-10-05 04:59:20]

PTAL :)
for Clean UP

[bbudge, 2016-10-05 14:07:34]

lgtm w/comments

https://codereview.chromium.org/2394773002/diff/1/ppapi/api/pp_input_event.idl
File ppapi/api/pp_input_event.idl (right):

https://codereview.chromium.org/2394773002/diff/1/ppapi/api/pp_input_event.id...
ppapi/api/pp_input_event.idl:115: /* (brettw) figure out exactly what this
means.*/
Just delete the comment.

https://codereview.chromium.org/2394773002/diff/1/ppapi/c/pp_input_event.h
File ppapi/c/pp_input_event.h (right):

https://codereview.chromium.org/2394773002/diff/1/ppapi/c/pp_input_event.h#ne...
ppapi/c/pp_input_event.h:124: /* (brettw) figure out exactly what this means. */
Delete comment

[Yeol Park, 2016-10-06 03:04:05]

On 2016/10/05 14:07:34, bbudge wrote:
> lgtm w/comments
> 
> https://codereview.chromium.org/2394773002/diff/1/ppapi/api/pp_input_event.idl
> File ppapi/api/pp_input_event.idl (right):
> 
>
https://codereview.chromium.org/2394773002/diff/1/ppapi/api/pp_input_event.id...
> ppapi/api/pp_input_event.idl:115: /* (brettw) figure out exactly what this
> means.*/
> Just delete the comment.
> 
> https://codereview.chromium.org/2394773002/diff/1/ppapi/c/pp_input_event.h
> File ppapi/c/pp_input_event.h (right):
> 
>
https://codereview.chromium.org/2394773002/diff/1/ppapi/c/pp_input_event.h#ne...
> ppapi/c/pp_input_event.h:124: /* (brettw) figure out exactly what this means.
*/
> Delete comment

I update patch set for delete comment.
Could you run build bot & reviews? :)

[bbudge, 2016-10-06 12:54:04]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-06 12:54:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-06 13:46:14]

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

[commit-bot: I haz the power, 2016-10-06 13:46:15]

Dry run: This issue passed the CQ dry run.

[Yeol Park, 2016-10-06 14:27:50]

The CQ bit was checked by peary2@gmail.com

[Yeol Park, 2016-10-06 14:27:50]

The patchset sent to the CQ was uploaded after l-g-t-m from bbudge@chromium.org
Link to the patchset: https://codereview.chromium.org/2394773002/#ps20001
(title: "Remove 'char' from IDL")

[commit-bot: I haz the power, 2016-10-06 14:28:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-06 14:33:56]

Description was changed from

==========
Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715
==========

to

==========
Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715
==========

[commit-bot: I haz the power, 2016-10-06 14:33:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-06 14:35:47]

Description was changed from

==========
Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715
==========

to

==========
Remove 'char' from IDL

We should just use int8_t or uint8_t instead of char in the IDL.

BUG=255715

Committed: https://crrev.com/509e71d7781b783c01728263fe9ffd52123cdbe3
Cr-Commit-Position: refs/heads/master@{#423524}
==========

[commit-bot: I haz the power, 2016-10-06 14:35:48]

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