mac: Remove IsOSMavericksOrLater()

mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org

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

[Mark Mentovai, 2016-07-08 18:29:18]

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

[Mark Mentovai, 2016-07-08 18:29:19]

We’re always Mavericks or later. We’re also always 64-bit.

[erikchen, 2016-07-08 18:31:25]

lgtm

[Nico, 2016-07-08 18:32:39]

lgtm, thanks!

https://codereview.chromium.org/2137453003/diff/1/content/app/mac/mac_init.mm
File content/app/mac/mac_init.mm (right):

https://codereview.chromium.org/2137453003/diff/1/content/app/mac/mac_init.mm...
content/app/mac/mac_init.mm:27: // TODO: Remove this when we build with the 10.9
SDK.
If this actually means "build with 10.9 SDK" and not "use 10.9 deployment
target", maybe this can be removed now too (not in this cl)

[Mark Mentovai, 2016-07-08 18:38:42]

https://codereview.chromium.org/2137453003/diff/1/content/app/mac/mac_init.mm
File content/app/mac/mac_init.mm (right):

https://codereview.chromium.org/2137453003/diff/1/content/app/mac/mac_init.mm...
content/app/mac/mac_init.mm:27: // TODO: Remove this when we build with the 10.9
SDK.
Nico wrote:
> If this actually means "build with 10.9 SDK" and not "use 10.9 deployment
> target", maybe this can be removed now too (not in this cl)

Good point, I filed https://crbug.com/626733 to ask for Chris’ input, since this
was his thing.

[Mark Mentovai, 2016-07-08 20:21:14]

Description was changed from

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
==========

to

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org
==========

[Mark Mentovai, 2016-07-08 20:21:28]

Description was changed from

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org
==========

to

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org
==========

[Mark Mentovai, 2016-07-08 20:21:28]

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

[Mark Mentovai, 2016-07-08 20:21:35]

brett for OWNERS

[Mark Mentovai, 2016-07-08 20:21:40]

The CQ bit was checked by mark@chromium.org

[Mark Mentovai, 2016-07-08 20:21:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
erikchen@chromium.org, thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/2137453003/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2016-07-08 20:22:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 23:41:36]

Description was changed from

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org
==========

to

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org
==========

[commit-bot: I haz the power, 2016-07-08 23:41:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-08 23:41:50]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-08 23:43:34]

Description was changed from

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org
==========

to

==========
mac: Remove IsOSMavericksOrLater()

BUG=579255
R=erikchen@chromium.org,thakis@chromium.org
TBR=brettw@chromium.org

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

[commit-bot: I haz the power, 2016-07-08 23:43:35]

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