Upstream: Make expat compiled in Android

Upstream: Make expat compiled in Android


BUG=None
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111206

[michaelbai, 2011-11-21 23:30:43]

Please help to review this

[nsylvain, 2011-11-22 00:48:36]

I have no idea what expat is. Is there anyone else who could review this?

The change itself looks good though...

[Peter Beverloo, 2011-11-22 15:07:27]

SGTM as well. Nicolas, you reviewed the initial expat import in issue 203022,
it's a "lightweight xml parser".

+Nico (from the trivial gyp-change gang :-))

[Nico, 2011-11-22 15:25:42]

The gyp change looks fine. I assume there's no system libexpat you could use
instead?

[michaelbai, 2011-11-22 19:06:33]

On 2011/11/22 15:25:42, Nico wrote:
> The gyp change looks fine. I assume there's no system libexpat you could use
> instead?

Yes, NDK doesn't provide this.

[commit-bot: I haz the power, 2011-11-22 19:10:48]

No LGTM from valid reviewers yet.

[Nico, 2011-11-22 19:11:08]

LGTM

[commit-bot: I haz the power, 2011-11-22 19:17:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/michaelbai@chromium.org/8620004/1

[commit-bot: I haz the power, 2011-11-22 20:47:51]

Change committed as 111206