Make chrome/BUILD.gn use symlink.gni rather than symlink.py directly

Make chrome/BUILD.gn use symlink.gni rather than symlink.py directly

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

[agrieve, 2016-09-16 02:03:19]

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

[Dirk Pranke, 2016-09-16 03:26:48]

lgtm, but you'll need to copy some other owner as well, as I don't own this
file.

[agrieve, 2016-09-16 13:03:16]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-09-16 13:03:35]

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

[commit-bot: I haz the power, 2016-09-16 13:03:35]

This CL has an open dependency (Issue 2348793002 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[agrieve, 2016-09-16 13:03:42]

On 2016/09/16 03:26:48, Dirk Pranke wrote:
> lgtm, but you'll need to copy some other owner as well, as I don't own this
> file.

Looks like //build owners have per-file on //chrome/BUILD.gn

[agrieve, 2016-09-16 13:49:44]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-09-16 13:50:05]

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-09-16 14:55:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-16 14:57:04]

Description was changed from

==========
Make chrome/BUILD.gn use symlink.gni rather than symlink.py directly
==========

to

==========
Make chrome/BUILD.gn use symlink.gni rather than symlink.py directly

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

[commit-bot: I haz the power, 2016-09-16 14:57:05]

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

[afakhry, 2016-09-16 16:47:38]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2344223002/ by afakhry@chromium.org.

The reason for reverting is: Breaks BuildPackages due to GN error
(https://bugs.chromium.org/p/chromium/issues/detail?id=647706).

BUG=647706.

[Dirk Pranke, 2016-09-16 19:45:28]

On 2016/09/16 13:03:42, agrieve wrote:
> On 2016/09/16 03:26:48, Dirk Pranke wrote:
> > lgtm, but you'll need to copy some other owner as well, as I don't own this
> > file.
> 
> Looks like //build owners have per-file on //chrome/BUILD.gn

Ah, right, I forgot ;).