Fix multiple assignment of "visibility" in generate_library_loader.gni

Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.

TBR=thakis

Committed: https://crrev.com/6ad2f470bac33901adc68c86e1863cde5985838e
Cr-Commit-Position: refs/heads/master@{#377090}

[spang, 2016-02-22 22:10:03]

spang@chromium.org changed reviewers:
+ phajdan.jr@chromium.org

[Paweł Hajdan Jr., 2016-02-23 17:02:40]

LGTM

[spang, 2016-02-23 17:48:13]

The CQ bit was checked by spang@chromium.org

[commit-bot: I haz the power, 2016-02-23 17:52:48]

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

[commit-bot: I haz the power, 2016-02-23 18:19:45]

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

[commit-bot: I haz the power, 2016-02-23 18:19:45]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[spang, 2016-02-23 21:04:52]

Description was changed from

==========
Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.
==========

to

==========
Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.

TBR=thakis
==========

[spang, 2016-02-23 21:05:06]

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

[spang, 2016-02-23 21:05:13]

The CQ bit was checked by spang@chromium.org

[commit-bot: I haz the power, 2016-02-23 21:12:01]

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

[commit-bot: I haz the power, 2016-02-23 21:24:40]

Description was changed from

==========
Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.

TBR=thakis
==========

to

==========
Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.

TBR=thakis
==========

[commit-bot: I haz the power, 2016-02-23 21:24:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-23 21:25:39]

Description was changed from

==========
Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.

TBR=thakis
==========

to

==========
Fix multiple assignment of "visibility" in generate_library_loader.gni

If the invoker has visibility set, it will be assigned twice, and
rejected by GN. Remove the second assignment.

TBR=thakis

Committed: https://crrev.com/6ad2f470bac33901adc68c86e1863cde5985838e
Cr-Commit-Position: refs/heads/master@{#377090}
==========

[commit-bot: I haz the power, 2016-02-23 21:25:40]

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

[Nico, 2016-02-23 21:39:00]

can you create a tools/generate_library_loader/OWNER file please?