This patch is to enable x86 android build.

This patch is to enable x86 android build.

patch includes two parts:
1. gyp files modification to include x86 android build.
2. add android config files for yasm which is copied from linux ones. enable android x86 build

This patch is cloned from the issue #10198027 for ownership issue. 
https://chromiumcodereview.appspot.com/10198027/

BUG=125160
TEST=


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

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

[Wei James(wistoch), 2012-05-02 01:28:15]

this patch is cloned from https://chromiumcodereview.appspot.com/10198027/ for
ownership issue. 

could you help to review it? thanks

[vandebo (ex-Chrome), 2012-05-02 01:32:32]

skia LGTM

[willchan no longer on Chromium, 2012-05-02 01:37:40]

https://chromiumcodereview.appspot.com/10271039/diff/1/base/base.gypi
File base/base.gypi (right):

https://chromiumcodereview.appspot.com/10271039/diff/1/base/base.gypi#newcode687
base/base.gypi:687: ['target_arch == "ia32"', {
I'm a bit confused by this. Can you explain?

[Wei James(wistoch), 2012-05-02 01:48:42]

https://chromiumcodereview.appspot.com/10271039/diff/1/base/base.gypi
File base/base.gypi (right):

https://chromiumcodereview.appspot.com/10271039/diff/1/base/base.gypi#newcode687
base/base.gypi:687: ['target_arch == "ia32"', {
On 2012/05/02 01:37:41, willchan wrote:
> I'm a bit confused by this. Can you explain?

thanks for the review. 

this is to include the source files for atomicops with x86 android build.
without these files, build error with "AtomicOps_Internalx86CPUFeatures not
defined".

[Wei James(wistoch), 2012-05-02 03:20:36]

as LGTM in https://chromiumcodereview.appspot.com/10198027/, I'd like to commit
it if no objection. is it ok? thanks

[commit-bot: I haz the power, 2012-05-02 03:23:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/james.wei@intel.com/10271039/1

[commit-bot: I haz the power, 2012-05-02 03:23:42]

Presubmit check for 10271039-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    base

Presubmit checks took 6.4s to calculate.

[Wei James(wistoch), 2012-05-02 03:28:19]

could anyone of base OWNER provide a LGTM for base directories? 

there is a LGTM from mark at https://chromiumcodereview.appspot.com/10198027/. 

the patch here is cloned from that one. thanks

[Mark Mentovai, 2012-05-02 13:10:46]

https://chromiumcodereview.appspot.com/10271039/diff/1/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/10271039/diff/1/build/common.gypi#newc...
build/common.gypi:811: # Android uses x86 instead of ia32 for their target_arch
designation.
Keep comment lines within 80 characters.

[Wei James(wistoch), 2012-05-02 14:32:39]

https://chromiumcodereview.appspot.com/10271039/diff/1/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/10271039/diff/1/build/common.gypi#newc...
build/common.gypi:811: # Android uses x86 instead of ia32 for their target_arch
designation.
On 2012/05/02 13:10:46, Mark Mentovai wrote:
> Keep comment lines within 80 characters.

thanks. fixed.

[Mark Mentovai, 2012-05-02 15:26:13]

LGTM

[commit-bot: I haz the power, 2012-05-02 15:35:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/james.wei@intel.com/10271039/11001

[brettw, 2012-05-02 19:23:15]

Please don't add so many reviewers. Just add the ones you need to review your
patch.

[commit-bot: I haz the power, 2012-05-03 00:50:36]

Try job failure for 10271039-11001 (retry) (retry) on linux_rel for step
"browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[Marc-Antoine Ruel (Google), 2012-05-03 00:53:57]

I agree with Brett. Removed John Grabowski, Yaron, Hironori Bono, Nico, Paweł
Hajdan Jr.

I'll set the CQ bit again, the error didn't seem related.

[commit-bot: I haz the power, 2012-05-03 00:54:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/james.wei@intel.com/10271039/11001

[Wei James(wistoch), 2012-05-03 00:56:47]

On 2012/05/03 00:53:57, Marc-Antoine Ruel (Google) wrote:
> I agree with Brett. Removed John Grabowski, Yaron, Hironori Bono, Nico, Paweł
> Hajdan Jr.
> 
> I'll set the CQ bit again, the error didn't seem related.

thanks.

[commit-bot: I haz the power, 2012-05-03 04:00:43]

Change committed as 135087

[Nico, 2012-05-03 05:02:16]

http://codereview.chromium.org/10271039/diff/11001/build/common.gypi
File build/common.gypi (right):

http://codereview.chromium.org/10271039/diff/11001/build/common.gypi#newcode2381
build/common.gypi:2381: 'conditions': [
This…

http://codereview.chromium.org/10271039/diff/11001/build/common.gypi#newcode2398
build/common.gypi:2398: 'conditions': [
…overrides the condition block here, which is a gyp error. You need to merge
these two condition blocks.

[Wei James(wistoch), 2012-05-03 05:18:33]

https://chromiumcodereview.appspot.com/10271039/diff/11001/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/10271039/diff/11001/build/common.gypi#...
build/common.gypi:2398: 'conditions': [
On 2012/05/03 05:02:16, Nico wrote:
> …overrides the condition block here, which is a gyp error. You need to merge
> these two condition blocks.

done. thanks

[Nico, 2012-05-03 05:21:45]

On 2012/05/03 05:18:33, James Wei wrote:
> https://chromiumcodereview.appspot.com/10271039/diff/11001/build/common.gypi
> File build/common.gypi (right):
> 
>
https://chromiumcodereview.appspot.com/10271039/diff/11001/build/common.gypi#...
> build/common.gypi:2398: 'conditions': [
> On 2012/05/03 05:02:16, Nico wrote:
> > …overrides the condition block here, which is a gyp error. You need to merge
> > these two condition blocks.
> 
> done. thanks

Feel free to reland then. If you can, be on #chromium on free node when this
lands.

[commit-bot: I haz the power, 2012-05-03 05:22:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/james.wei@intel.com/10271039/29003

[Wei James(wistoch), 2012-05-03 05:25:32]

On 2012/05/03 05:21:45, Nico wrote:
> On 2012/05/03 05:18:33, James Wei wrote:
> > https://chromiumcodereview.appspot.com/10271039/diff/11001/build/common.gypi
> > File build/common.gypi (right):
> > 
> >
>
https://chromiumcodereview.appspot.com/10271039/diff/11001/build/common.gypi#...
> > build/common.gypi:2398: 'conditions': [
> > On 2012/05/03 05:02:16, Nico wrote:
> > > …overrides the condition block here, which is a gyp error. You need to
merge
> > > these two condition blocks.
> > 
> > done. thanks
> 
> Feel free to reland then. If you can, be on #chromium on free node when this
> lands.

thanks. I am on #chromium as wistoch. 

https://chromiumcodereview.appspot.com/10316014/

[commit-bot: I haz the power, 2012-05-03 09:17:48]

Change committed as 135110