remove deprecated armv7 gyp setting now that all uses have been removed

remove deprecated armv7 gyp setting now that all uses have been removed

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

[Mostyn Bramley-Moore, 2013-12-23 22:33:44]

@Nico: PTAL

[Nico, 2013-12-24 01:35:12]

Lgtm

Good luck!
On Dec 23, 2013 2:33 PM, <mostynb@opera.com> wrote:

> Reviewers: Nico,
>
> Message:
> @Nico: PTAL
>
> Description:
> remove deprecated armv7 gyp setting now that all uses have been removed
>
> BUG=234135
>
> Please review this at https://codereview.chromium.org/120123004/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files (+0, -10 lines):
>   M build/common.gypi
>
>
> Index: build/common.gypi
> diff --git a/build/common.gypi b/build/common.gypi
> index 734c9e78d4aba89411ec70873c804e7408db1460..
> 7772b76928ded567209d83201da2dc78e930307a 100644
> --- a/build/common.gypi
> +++ b/build/common.gypi
> @@ -527,15 +527,6 @@
>            'os_bsd%': 0,
>          }],
>
> -        # Set armv7 for backward compatibility.
> -        # TODO(mostynb@opera.com): remove armv7 once all uses are
> -        # removed http://crbug.com/234135
> -        ['arm_version==7', {
> -          'armv7': 1,
> -        }, {
> -          'armv7': 0,
> -        }],
> -
>          # NSS usage.
>          ['(OS=="linux" or OS=="freebsd" or OS=="openbsd" or
> OS=="solaris") and use_openssl==0', {
>            'use_nss%': 1,
> @@ -914,7 +905,6 @@
>      'tracing_like_official_build%': '<(tracing_like_official_build)',
>      'python_ver%': '<(python_ver)',
>      'arm_version%': '<(arm_version)',
> -    'armv7%': '<(armv7)',
>      'arm_neon%': '<(arm_neon)',
>      'arm_neon_optional%': '<(arm_neon_optional)',
>      'sysroot%': '<(sysroot)',
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-12-24 08:29:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/120123004/1

[commit-bot: I haz the power, 2013-12-24 11:08:48]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-12-24 13:12:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mostynb@opera.com/120123004/1

[commit-bot: I haz the power, 2013-12-24 15:08:04]

Change committed as 242428