[gn] Switch mac to gn

[gn] Switch mac to gn

BUG=chromium:474921
Committed: https://crrev.com/c2d6d983dee014f3661fd56029669d61cc1443bd
Cr-Commit-Position: refs/heads/master@{#38011}

[Michael Achenbach, 2016-07-25 09:40:56]

Description was changed from

==========
[gn] Switch mac to gn

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

to

==========
[gn] Switch mac to gn

BUG=chromium:474921
==========

[Michael Achenbach, 2016-07-25 09:41:10]

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

[commit-bot: I haz the power, 2016-07-25 09:41:22]

Dry run: 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-25 09:44:10]

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

[commit-bot: I haz the power, 2016-07-25 09:44:11]

Dry run: Try jobs failed on following builders:
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/5571)

[Michael Achenbach, 2016-07-25 09:58:04]

PTAL

Comparison links:
https://build.chromium.org/p/client.v8/builders/V8%20Mac/builds/9207/steps/co...
https://build.chromium.org/p/client.v8/builders/V8%20Mac%20-%20debug/builds/8...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64/builds/10238/steps...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64%20-%20debug/builds...

-fstrict-aliasing has always been off so we don't pass the flag anymore now. See
also: http://crbug.com/628560

Another difference is this feature:
https://cs.chromium.org/chromium/src/v8/testing/gtest.gyp?q=%22GTEST_HAS_RTTI...

This looks quite outdated and I think we should only port it if it becomes a
problem. It leads to skipping -DGTEST_HAS_RTTI=0 in GN builds.

[Michael Achenbach, 2016-07-25 09:58:37]

machenbach@chromium.org changed reviewers:
+ jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-07-25 09:58:37]

And now with reviewers. Please see comment above.

[Michael Achenbach, 2016-07-25 09:59:17]

Hold on for now. Need to resolve a build error on mac.

[Michael Achenbach, 2016-07-25 10:17:15]

PTAL now.

https://codereview.chromium.org/2182443002/diff/40001/build_overrides/gtest.gni
File build_overrides/gtest.gni (right):

https://codereview.chromium.org/2182443002/diff/40001/build_overrides/gtest.g...
build_overrides/gtest.gni:9: gtest_include_platform_test = false
Use same switches as pdfium now. This gets us the same source files we had with
gyp.

[vogelheim, 2016-07-25 10:38:41]

lgtm

[Michael Achenbach, 2016-07-25 11:16:25]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-07-25 11:16:29]

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-25 11:35:59]

Description was changed from

==========
[gn] Switch mac to gn

BUG=chromium:474921
==========

to

==========
[gn] Switch mac to gn

BUG=chromium:474921
==========

[commit-bot: I haz the power, 2016-07-25 11:36:00]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-25 11:38:15]

Description was changed from

==========
[gn] Switch mac to gn

BUG=chromium:474921
==========

to

==========
[gn] Switch mac to gn

BUG=chromium:474921

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

[commit-bot: I haz the power, 2016-07-25 11:38:16]

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