Remove skia_arch_width, fold into skia_arch_type.

Remove skia_arch_width, fold into skia_arch_type.
DOCS_PREVIEW= https://skia.org/?cl=1228553010

BUG=skia:4042
R=borenet@google.com, mtklein@google.com

Committed: https://skia.googlesource.com/skia/+/4239fc2f69bdbd1a1f01ee66112f46033b47b743

[bsalomon, 2015-07-08 18:58:08]

bsalomon@google.com changed reviewers:
+ borenet@google.com, mtklein@google.com

[bsalomon, 2015-07-08 18:58:08]

Needs some corresponding changes to the slave scripts (eliminating
skia_arch_width specifications and changing skia_arch_type from mips to mips32
and mip64 on mips builders).

[mtklein, 2015-07-09 16:29:57]

lgtm (mod bot issues)

https://codereview.chromium.org/1228553010/diff/1/gyp/everything.gyp
File gyp/everything.gyp (right):

https://codereview.chromium.org/1228553010/diff/1/gyp/everything.gyp#newcode23
gyp/everything.gyp:23: ['skia_os in ("ios", "android", "chromeos") or (skia_os
== "mac" and skia_arch_type == "x86")', {
Let's kill off this clause?  We don't do 32-bit Mac builds anymore.

https://codereview.chromium.org/1228553010/diff/1/gyp/libjpeg-turbo.gyp
File gyp/libjpeg-turbo.gyp (right):

https://codereview.chromium.org/1228553010/diff/1/gyp/libjpeg-turbo.gyp#newco...
gyp/libjpeg-turbo.gyp:197: [ 'skia_arch_type == "mips64"', {
This is a lot nicer.

[borenet, 2015-07-13 13:44:36]

LGTM as well.  I suggest landing like this:

1. Create a change to gyp_defs_from_builder_dict in this file:
https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave...
2. Close the tree and wait for builds to cycle. I suggest closing the tree in
the evening and waiting until the following morning.
3. Land the change from #1. All builds will fail until after #4, but hopefully
no builds start.
4. Land this change, ensure that everything is working properly.
5. Open the tree.

[bsalomon, 2015-07-14 15:33:36]

Patchset #2 (id:20001) has been deleted

[bsalomon, 2015-07-14 15:38:37]

bsalomon@google.com changed reviewers:
+ scroggo@google.com

[bsalomon, 2015-07-14 15:39:23]

On 2015/07/13 13:44:36, borenet wrote:
> LGTM as well.  I suggest landing like this:
> 
> 1. Create a change to gyp_defs_from_builder_dict in this file:
>
https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave...
> 2. Close the tree and wait for builds to cycle. I suggest closing the tree in
> the evening and waiting until the following morning.
> 3. Land the change from #1. All builds will fail until after #4, but hopefully
> no builds start.
> 4. Land this change, ensure that everything is working properly.
> 5. Open the tree.

I will try this tonight if the tree is in relatively good shape. Adding the
current sheriff, Leon.

[bsalomon, 2015-07-14 17:51:58]

On 2015/07/13 13:44:36, borenet wrote:
> LGTM as well.  I suggest landing like this:
> 
> 1. Create a change to gyp_defs_from_builder_dict in this file:
>
https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave...


The config.py CL is here: https://codereview.chromium.org/1237833003/

[bsalomon, 2015-07-27 13:10:46]

Committed patchset #4 (id:80001) manually as
4239fc2f69bdbd1a1f01ee66112f46033b47b743.

[bsalomon, 2015-07-27 13:11:04]

https://codereview.chromium.org/1228553010/diff/1/gyp/everything.gyp
File gyp/everything.gyp (right):

https://codereview.chromium.org/1228553010/diff/1/gyp/everything.gyp#newcode23
gyp/everything.gyp:23: ['skia_os in ("ios", "android", "chromeos") or (skia_os
== "mac" and skia_arch_type == "x86")', {
On 2015/07/09 16:29:57, mtklein wrote:
> Let's kill off this clause?  We don't do 32-bit Mac builds anymore.

Done.