Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532
Committed: https://crrev.com/fd78509dac4927f0406a0947a18c5b15fb0862ee
Cr-Commit-Position: refs/heads/master@{#367172}

[michaelbai, 2015-12-30 00:13:54]

Description was changed from

==========
Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532
==========

to

==========
Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532
==========

[michaelbai, 2015-12-30 00:13:55]

michaelbai@chromium.org changed reviewers:
+ agrieve@chromium.org

[agrieve, 2015-12-30 15:12:16]

lgtm

https://codereview.chromium.org/1551863002/diff/20001/build/config/android/ru...
File build/config/android/rules.gni (right):

https://codereview.chromium.org/1551863002/diff/20001/build/config/android/ru...
build/config/android/rules.gni:582: #
nit: Remove blank line (feel free to remove the one above shared_resources as
well)

https://codereview.chromium.org/1551863002/diff/20001/build/config/android/ru...
build/config/android/rules.gni:1569: if
(defined(invoker.alternative_android_sdk_jar)) {
nit: use forward_variables_from()

[michaelbai, 2015-12-30 18:54:16]

https://codereview.chromium.org/1551863002/diff/20001/build/config/android/ru...
File build/config/android/rules.gni (right):

https://codereview.chromium.org/1551863002/diff/20001/build/config/android/ru...
build/config/android/rules.gni:582: #
On 2015/12/30 15:12:16, agrieve wrote:
> nit: Remove blank line (feel free to remove the one above shared_resources as
> well)

Done.

https://codereview.chromium.org/1551863002/diff/20001/build/config/android/ru...
build/config/android/rules.gni:1569: if
(defined(invoker.alternative_android_sdk_jar)) {
On 2015/12/30 15:12:16, agrieve wrote:
> nit: use forward_variables_from()

Done.

[michaelbai, 2015-12-30 18:54:43]

The CQ bit was checked by michaelbai@chromium.org

[michaelbai, 2015-12-30 18:54:43]

The patchset sent to the CQ was uploaded after l-g-t-m from agrieve@chromium.org
Link to the patchset: https://codereview.chromium.org/1551863002/#ps40001
(title: "address comments")

[commit-bot: I haz the power, 2015-12-30 18:55:43]

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

[commit-bot: I haz the power, 2015-12-30 19:27:43]

Description was changed from

==========
Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532
==========

to

==========
Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532
==========

[commit-bot: I haz the power, 2015-12-30 19:27:44]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-30 19:28:39]

Description was changed from

==========
Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532
==========

to

==========
Port android_aapt_path, app_as_shared_lib and alternative_sdk to GN

alternative_sdk_jar is used in proguard

BUG=565532

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

[commit-bot: I haz the power, 2015-12-30 19:28:40]

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