Simplify xcode_settings block in common.gypi a bit.

Simplify xcode_settings block in common.gypi a bit.

- Mac builds are always 64-bit nowaydays
- Collapse two adjacent OS==ios conditions

No intended behavior change.

BUG=none

Committed: https://crrev.com/ea6bdacc04396d48bd80fc40cf490ad238d88e4d
Cr-Commit-Position: refs/heads/master@{#367314}

[Nico, 2016-01-03 22:19:22]

thakis@chromium.org changed reviewers:
+ mark@chromium.org

[Mark Mentovai, 2016-01-04 16:15:49]

LGTM

https://codereview.chromium.org/1557823002/diff/1/build/common.gypi
File build/common.gypi (left):

https://codereview.chromium.org/1557823002/diff/1/build/common.gypi#oldcode6412
build/common.gypi:6412: 'i386'
Double-check with someone from remoting that they’re not still building their
pref pane as 32-bit (or fat).

[Nico, 2016-01-04 16:29:54]

https://codereview.chromium.org/1557823002/diff/1/build/common.gypi
File build/common.gypi (left):

https://codereview.chromium.org/1557823002/diff/1/build/common.gypi#oldcode6412
build/common.gypi:6412: 'i386'
On 2016/01/04 16:15:49, Mark Mentovai wrote:
> Double-check with someone from remoting that they’re not still building their
> pref pane as 32-bit (or fat).

Would that be a problem? They used to locally set ARCHS in their target (lhs of
https://codereview.chromium.org/1547533002/diff/160001/remoting/remoting_host...
-- they currently don't set ARCHS but that change looks wrong to me)

[Mark Mentovai, 2016-01-04 16:31:01]

I didn’t know if they were setting ARCHS or if they had some
double-build-plus-lipo thing.

LGTM, it looks like you won’t break them (and it looks like they may already be
broken anyway).

[Nico, 2016-01-04 16:31:23]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-01-04 16:33:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1557823002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1557823002/1

[commit-bot: I haz the power, 2016-01-04 16:38:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-04 16:39:30]

Description was changed from

==========
Simplify xcode_settings block in common.gypi a bit.

- Mac builds are always 64-bit nowaydays
- Collapse two adjacent OS==ios conditions

No intended behavior change.

BUG=none
==========

to

==========
Simplify xcode_settings block in common.gypi a bit.

- Mac builds are always 64-bit nowaydays
- Collapse two adjacent OS==ios conditions

No intended behavior change.

BUG=none

Committed: https://crrev.com/ea6bdacc04396d48bd80fc40cf490ad238d88e4d
Cr-Commit-Position: refs/heads/master@{#367314}
==========

[commit-bot: I haz the power, 2016-01-04 16:39:32]

Patchset 1 (id:??) landed as
https://crrev.com/ea6bdacc04396d48bd80fc40cf490ad238d88e4d
Cr-Commit-Position: refs/heads/master@{#367314}