mac: Bump the deployment target to OS X 10.7.

mac: Bump the deployment target to OS X 10.7.

BUG=592663
Committed: https://crrev.com/4bdf946cb5e01a3165fbe4e64ca15ff5aaa2ef5f
Cr-Commit-Position: refs/heads/master@{#385003}

[erikchen, 2016-03-30 20:47:34]

Description was changed from

==========
mac: Bump the deployment target to OS X 10.7.

BUG=
==========

to

==========
mac: Bump the deployment target to OS X 10.7.

BUG=592663
==========

[erikchen, 2016-03-30 20:48:37]

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

[erikchen, 2016-03-30 20:48:38]

mark: Please review.

[Mark Mentovai, 2016-03-30 20:50:35]

LGTM

[erikchen, 2016-04-04 17:39:24]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[erikchen, 2016-04-04 17:39:45]

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

[erikchen, 2016-04-04 17:39:45]

thakis: Please review.

[commit-bot: I haz the power, 2016-04-04 17:40:00]

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

[Nico, 2016-04-04 18:01:24]

Nice! Do the same in gn...looks like that's now in build_overrides/build.gni

https://codereview.chromium.org/1847533004/diff/1/build/common.gypi
File build/common.gypi (right):

https://codereview.chromium.org/1847533004/diff/1/build/common.gypi#newcode1965
build/common.gypi:1965: 'mac_deployment_target%': '10.7',
These can merge again now, right? iOS only cares about "at least 10.7", so no
need to have two of these anymore.

[erikchen, 2016-04-04 18:53:18]

dpranke: Please review.
thakis: PTAL

https://codereview.chromium.org/1847533004/diff/40001/build_overrides/build.gni
File build_overrides/build.gni (right):

https://codereview.chromium.org/1847533004/diff/40001/build_overrides/build.g...
build_overrides/build.gni:9: # requires 10.7.
After this CL lands and sticks, we will still need to keep this file because of
the 10.11 SDK requirement?

[erikchen, 2016-04-04 18:53:34]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-04 18:54:00]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1847533004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1847533004/40001

[Nico, 2016-04-04 19:00:44]

lgtm, thanks

[erikchen, 2016-04-04 19:38:39]

erikchen@chromium.org changed reviewers:
+ dpranke@chromium.org

[erikchen, 2016-04-04 19:38:39]

+dpranke, for real this time.

[commit-bot: I haz the power, 2016-04-04 20:04:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-04 20:04:27]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2016-04-04 20:56:32]

lgtm.

I don't know if we'll need to keep the build_overrides file around or not.

[erikchen, 2016-04-04 20:57:52]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-04-04 20:57:52]

The patchset sent to the CQ was uploaded after l-g-t-m from mark@chromium.org
Link to the patchset: https://codereview.chromium.org/1847533004/#ps40001
(title: "Comments from thakis.")

[commit-bot: I haz the power, 2016-04-04 20:58:42]

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

[commit-bot: I haz the power, 2016-04-04 21:05:13]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-04 21:06:44]

Description was changed from

==========
mac: Bump the deployment target to OS X 10.7.

BUG=592663
==========

to

==========
mac: Bump the deployment target to OS X 10.7.

BUG=592663

Committed: https://crrev.com/4bdf946cb5e01a3165fbe4e64ca15ff5aaa2ef5f
Cr-Commit-Position: refs/heads/master@{#385003}
==========

[commit-bot: I haz the power, 2016-04-04 21:06:45]

Patchset 3 (id:??) landed as
https://crrev.com/4bdf946cb5e01a3165fbe4e64ca15ff5aaa2ef5f
Cr-Commit-Position: refs/heads/master@{#385003}