Chromecast: use more recent Android support lib if available.

Chromecast: use more recent Android support lib if available.

R=lcwu@chromium.org,byungchul@chromium.org
BUG=400876
Committed: https://crrev.com/cbdaf76256d60e43e0859ad6e3d893e879610df2
Cr-Commit-Position: refs/heads/master@{#302308}

[byungchul, 2014-10-31 17:36:22]

https://codereview.chromium.org/694853002/diff/1/chromecast/chromecast.gyp
File chromecast/chromecast.gyp (right):

https://codereview.chromium.org/694853002/diff/1/chromecast/chromecast.gyp#ne...
chromecast/chromecast.gyp:355:
'../third_party/android_tools_internal/android_tools.gyp:android_support_v13_javalib_L',
It is not available in ToT. Any other way to replace this?

[gunsch, 2014-10-31 18:02:25]

https://codereview.chromium.org/694853002/diff/1/chromecast/chromecast.gyp
File chromecast/chromecast.gyp (right):

https://codereview.chromium.org/694853002/diff/1/chromecast/chromecast.gyp#ne...
chromecast/chromecast.gyp:355:
'../third_party/android_tools_internal/android_tools.gyp:android_support_v13_javalib_L',
On 2014/10/31 17:36:22, byungchul wrote:
> It is not available in ToT. Any other way to replace this?

Neither are any of the internal GYP references that we guard with the same GYP
condition. Why do you think this case is a problem?

[byungchul, 2014-10-31 18:25:36]

On 2014/10/31 18:02:25, gunsch wrote:
> https://codereview.chromium.org/694853002/diff/1/chromecast/chromecast.gyp
> File chromecast/chromecast.gyp (right):
> 
>
https://codereview.chromium.org/694853002/diff/1/chromecast/chromecast.gyp#ne...
> chromecast/chromecast.gyp:355:
>
'../third_party/android_tools_internal/android_tools.gyp:android_support_v13_javalib_L',
> On 2014/10/31 17:36:22, byungchul wrote:
> > It is not available in ToT. Any other way to replace this?
> 
> Neither are any of the internal GYP references that we guard with the same GYP
> condition. Why do you think this case is a problem?

You cannot build cast_shell_apk only with Chrome brand. Why not replace a sdk
with gyp variables?

[gunsch, 2014-10-31 18:35:33]

Sounds good, done.

[byungchul, 2014-10-31 18:44:39]

lgtm

[gunsch, 2014-10-31 18:45:16]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2014-10-31 18:48:20]

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

[commit-bot: I haz the power, 2014-10-31 18:48:21]

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

[commit-bot: I haz the power, 2014-10-31 18:48:23]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[lcwu1, 2014-10-31 18:50:56]

lgtm

[lcwu1, 2014-10-31 18:51:13]

The CQ bit was checked by lcwu@chromium.org

[gunsch, 2014-10-31 18:51:47]

The CQ bit was unchecked by gunsch@chromium.org

[gunsch, 2014-10-31 18:52:04]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2014-10-31 18:54:33]

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

[commit-bot: I haz the power, 2014-10-31 19:38:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-31 19:38:45]

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