Making the AIDL GN target support multiple include directories.

Making the AIDL GN target support multiple include directories.

This is needed by a follow up patch that adds Java Parcelable's
support to Mojo.

BUG=699288
TBR=phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2738743003
Cr-Commit-Position: refs/heads/master@{#455504}
Committed: https://chromium.googlesource.com/chromium/src/+/3e97d4db7e7a6bd09f0dea98fb347ce61bdcda05

[Jay Civelli, 2017-03-07 23:07:39]

Patchset #1 (id:1) has been deleted

[Jay Civelli, 2017-03-07 23:12:56]

Description was changed from

==========
Makking the aidl GN tag support multiple include directories.

BUG=
==========

to

==========
Makking the aidl GN tag support multiple include directories.

BUG=699288
==========

[Jay Civelli, 2017-03-07 23:12:56]

jcivelli@chromium.org changed reviewers:
+ dpranke@chromium.org

[Jay Civelli, 2017-03-07 23:13:19]

Description was changed from

==========
Makking the aidl GN tag support multiple include directories.

BUG=699288
==========

to

==========
Making the AIDL GN target support multiple include directories.

BUG=699288
==========

[Jay Civelli, 2017-03-07 23:14:13]

Description was changed from

==========
Making the AIDL GN target support multiple include directories.

BUG=699288
==========

to

==========
Making the AIDL GN target support multiple include directories.

This is needed by a follow up patch that adds Java Parcelable's
support to Mojo.

BUG=699288
==========

[Jay Civelli, 2017-03-07 23:14:19]

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

[commit-bot: I haz the power, 2017-03-07 23:15:01]

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

[Dirk Pranke, 2017-03-08 02:02:16]

lgtm

[commit-bot: I haz the power, 2017-03-08 02:10:22]

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

[commit-bot: I haz the power, 2017-03-08 02:10:23]

Dry run: This issue passed the CQ dry run.

[Jay Civelli, 2017-03-08 17:32:31]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2017-03-08 17:32:58]

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, 2017-03-08 17:40:41]

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

[commit-bot: I haz the power, 2017-03-08 17:40:42]

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

[Jay Civelli, 2017-03-08 17:44:22]

jcivelli@chromium.org changed reviewers:
+ boliu@chromium.org, hanxi@chromium.org, phajdan.jr@chromium.org

[Jay Civelli, 2017-03-08 17:46:39]

Adding OWNER reviewer, could you please take a look at:
boliu@ android_webview/BUILD.gn and content/public/android/BUILD.gn
phajdan.jr@ base/test/BUILD.gn
hanxi@ chrome/android/webapk/libs/runtime_library/BUILD.gn

[boliu, 2017-03-08 17:55:31]

On 2017/03/08 17:46:39, Jay Civelli wrote:
> Adding OWNER reviewer, could you please take a look at:
> boliu@ android_webview/BUILD.gn and content/public/android/BUILD.gn

rs lgtm

> phajdan.jr@ base/test/BUILD.gn
> hanxi@ chrome/android/webapk/libs/runtime_library/BUILD.gn

[Xi Han, 2017-03-08 17:59:04]

WebAPK lgtm

[Jay Civelli, 2017-03-08 18:43:17]

Description was changed from

==========
Making the AIDL GN target support multiple include directories.

This is needed by a follow up patch that adds Java Parcelable's
support to Mojo.

BUG=699288
==========

to

==========
Making the AIDL GN target support multiple include directories.

This is needed by a follow up patch that adds Java Parcelable's
support to Mojo.

BUG=699288
TBR=phajdan.jr@chromium.org
==========

[Jay Civelli, 2017-03-08 18:43:32]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2017-03-08 18:44:17]

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, 2017-03-08 18:52:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488998612131790,
"parent_rev": "61393e61922a94637ea7d9463690db0f5113e598", "commit_rev":
"3e97d4db7e7a6bd09f0dea98fb347ce61bdcda05"}

[commit-bot: I haz the power, 2017-03-08 18:52:52]

Description was changed from

==========
Making the AIDL GN target support multiple include directories.

This is needed by a follow up patch that adds Java Parcelable's
support to Mojo.

BUG=699288
TBR=phajdan.jr@chromium.org
==========

to

==========
Making the AIDL GN target support multiple include directories.

This is needed by a follow up patch that adds Java Parcelable's
support to Mojo.

BUG=699288
TBR=phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2738743003
Cr-Commit-Position: refs/heads/master@{#455504}
Committed:
https://chromium.googlesource.com/chromium/src/+/3e97d4db7e7a6bd09f0dea98fb34...
==========

[commit-bot: I haz the power, 2017-03-08 18:52:53]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/3e97d4db7e7a6bd09f0dea98fb34...