Forward visibility explicitly in GN component.

Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in components, where before it did not apply.

Fixes //ui/ozone:ozone's visibility which was not being applied because of this. I believe the intent of the ozone target is to be public and the rest should be file-local.

BUG=594610
Committed: https://crrev.com/58c2b3a90b634d71ccc8de68417789f21a59a6eb
Cr-Commit-Position: refs/heads/master@{#381043}

[brettw, 2016-03-14 18:11:16]

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

[brettw, 2016-03-14 18:11:31]

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

[commit-bot: I haz the power, 2016-03-14 18:12:04]

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

[yzshen1, 2016-03-14 18:13:03]

lgtm

[commit-bot: I haz the power, 2016-03-14 18:20:17]

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

[commit-bot: I haz the power, 2016-03-14 18:20:18]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[brettw, 2016-03-14 18:25:26]

Description was changed from

==========
Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in
components, where before it did not apply.

BUG=594610
==========

to

==========
Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in
components, where before it did not apply.

Fixes //ui/ozone:ozone's visibility which was not being applied because of this.
I believe the intent of the ozone target is to be public and the rest should be
file-local.

BUG=594610
==========

[brettw, 2016-03-14 18:25:29]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-03-14 18:25:30]

The patchset sent to the CQ was uploaded after l-g-t-m from yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/1800703002/#ps20001
(title: "ozone")

[commit-bot: I haz the power, 2016-03-14 18:25:48]

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

[commit-bot: I haz the power, 2016-03-14 19:36:07]

Description was changed from

==========
Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in
components, where before it did not apply.

Fixes //ui/ozone:ozone's visibility which was not being applied because of this.
I believe the intent of the ozone target is to be public and the rest should be
file-local.

BUG=594610
==========

to

==========
Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in
components, where before it did not apply.

Fixes //ui/ozone:ozone's visibility which was not being applied because of this.
I believe the intent of the ozone target is to be public and the rest should be
file-local.

BUG=594610
==========

[commit-bot: I haz the power, 2016-03-14 19:36:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-14 19:37:35]

Description was changed from

==========
Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in
components, where before it did not apply.

Fixes //ui/ozone:ozone's visibility which was not being applied because of this.
I believe the intent of the ozone target is to be public and the rest should be
file-local.

BUG=594610
==========

to

==========
Forward visibility explicitly in GN component.

As described in the added comment, this picks up file-scope visibility in
components, where before it did not apply.

Fixes //ui/ozone:ozone's visibility which was not being applied because of this.
I believe the intent of the ozone target is to be public and the rest should be
file-local.

BUG=594610

Committed: https://crrev.com/58c2b3a90b634d71ccc8de68417789f21a59a6eb
Cr-Commit-Position: refs/heads/master@{#381043}
==========

[commit-bot: I haz the power, 2016-03-14 19:37:36]

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