[Mac/GN] Add BUILD.gn file for //chrome/tools/mac_helpers.

[Mac/GN] Add BUILD.gn file for //chrome/tools/mac_helpers.

BUG=431177
R=thakis@chromium.org

Committed: https://crrev.com/31cad7ed14e279442c9234c19f8c602f70cea8a2
Cr-Commit-Position: refs/heads/master@{#390505}

[Robert Sesek, 2016-04-28 20:07:51]

Patchset #1 (id:1) has been deleted

[Nico, 2016-04-28 20:13:24]

https://codereview.chromium.org/1914233005/diff/20001/chrome/tools/mac_helper...
File chrome/tools/mac_helpers/BUILD.gn (right):

https://codereview.chromium.org/1914233005/diff/20001/chrome/tools/mac_helper...
chrome/tools/mac_helpers/BUILD.gn:9: "infoplist_strings_util.mm",
this includes "chrome/grit/chromium_strings.h" which I think is a file generated
by grit. Does ui_data_pack generate chrome/chromium_strings.h?

[Robert Sesek, 2016-04-28 20:27:52]

https://codereview.chromium.org/1914233005/diff/20001/chrome/tools/mac_helper...
File chrome/tools/mac_helpers/BUILD.gn (right):

https://codereview.chromium.org/1914233005/diff/20001/chrome/tools/mac_helper...
chrome/tools/mac_helpers/BUILD.gn:9: "infoplist_strings_util.mm",
On 2016/04/28 20:13:24, Nico wrote:
> this includes "chrome/grit/chromium_strings.h" which I think is a file
generated
> by grit. Does ui_data_pack generate chrome/chromium_strings.h?

Oops, yes need that too. Thanks. The deps list in GYP was pretty crufty so I
didn't trust it.

[Nico, 2016-04-28 20:30:33]

lgtm

[Nico, 2016-04-28 20:30:55]

(and maybe clean up the deps list in gyp while here -- probably nice if deps
match)

[Robert Sesek, 2016-04-28 20:56:55]

The CQ bit was checked by rsesek@chromium.org

[Robert Sesek, 2016-04-28 20:56:55]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1914233005/#ps60001
(title: "Fix gyp")

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

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

[commit-bot: I haz the power, 2016-04-28 22:14:32]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-04-30 17:21:59]

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

[commit-bot: I haz the power, 2016-04-30 17:57:56]

Description was changed from

==========
[Mac/GN] Add BUILD.gn file for //chrome/tools/mac_helpers.

BUG=431177
R=thakis@chromium.org
==========

to

==========
[Mac/GN] Add BUILD.gn file for //chrome/tools/mac_helpers.

BUG=431177
R=thakis@chromium.org

Committed: https://crrev.com/31cad7ed14e279442c9234c19f8c602f70cea8a2
Cr-Commit-Position: refs/heads/master@{#390505}
==========