Don't set MIPS specific target arch CFLAGS for android WebView

Don't set MIPS specific target arch CFLAGS for android WebView

These flags are set by the android build system in the case of
an android webview build, so don't attempt to reset them.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=208907

[benm (inactive), 2013-06-26 22:40:21]

ptal, i think this is what we want?

[Torne, 2013-06-26 22:41:51]

https://codereview.chromium.org/17956002/diff/2001/build/common.gypi
File build/common.gypi (right):

https://codereview.chromium.org/17956002/diff/2001/build/common.gypi#newcode1333
build/common.gypi:1333: 'mips_arch_variant': '',
Do we need to touch this given the other diff fragment? Is it used somewhere
else?

https://codereview.chromium.org/17956002/diff/2001/build/common.gypi#newcode3049
build/common.gypi:3049: }], ['android_webview_build==0 and
mips_arch_variant!="mips32r2"', {
This should be split onto a new line after }],

[benm (inactive), 2013-06-26 22:50:47]

thanks torne, running a test build locally

https://codereview.chromium.org/17956002/diff/2001/build/common.gypi
File build/common.gypi (right):

https://codereview.chromium.org/17956002/diff/2001/build/common.gypi#newcode1333
build/common.gypi:1333: 'mips_arch_variant': '',
On 2013/06/26 22:41:51, Torne wrote:
> Do we need to touch this given the other diff fragment? Is it used somewhere
> else?

removed.

https://codereview.chromium.org/17956002/diff/2001/build/common.gypi#newcode3049
build/common.gypi:3049: }], ['android_webview_build==0 and
mips_arch_variant!="mips32r2"', {
On 2013/06/26 22:41:51, Torne wrote:
> This should be split onto a new line after }],

Done.

[Torne, 2013-06-26 22:53:05]

lgtm

[commit-bot: I haz the power, 2013-06-27 00:00:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benm@chromium.org/17956002/7001

[commit-bot: I haz the power, 2013-06-27 06:51:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benm@chromium.org/17956002/7001

[commit-bot: I haz the power, 2013-06-27 12:58:43]

Change committed as 208907