Remove Chrome OS-related GN build configuration

Remove Chrome OS-related GN build configuration

We don't support Chrome OS, so no need for build configuration.

R=viettrungluu@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/57ca940d8410a27fdf517b6194d7ecb0f29ecc2a

[Petr Hosek, 2016-01-12 19:11:09]

Description was changed from

==========
Remove Chrome OS-related GN build configuration

We no longer support Chrome OS.

R=vtl@chromium.org
==========

to

==========
Remove Chrome OS-related GN build configuration

We don't support Chrome OS, so no need for build configuration.

R=vtl@chromium.org
==========

[Petr Hosek, 2016-01-12 19:11:37]

Description was changed from

==========
Remove Chrome OS-related GN build configuration

We don't support Chrome OS, so no need for build configuration.

R=vtl@chromium.org
==========

to

==========
Remove Chrome OS-related GN build configuration

We don't support Chrome OS, so no need for build configuration.

R=viettrungluu@chromium.org
==========

[Petr Hosek, 2016-01-12 19:11:38]

phosek@chromium.org changed reviewers:
+ viettrungluu@chromium.org
- vtl@chromium.org

[viettrungluu, 2016-01-12 22:47:11]

LGTM, though I think you can also delete all of //build/toolchain/cros.

https://codereview.chromium.org/1581753002/diff/1/build/config/BUILDCONFIG.gn
File build/config/BUILDCONFIG.gn (right):

https://codereview.chromium.org/1581753002/diff/1/build/config/BUILDCONFIG.gn...
build/config/BUILDCONFIG.gn:97: is_desktop_linux = current_os == "linux"
We should probably get rid of "is_desktop_linux" in that case. (You don't need
to do that here.

[Petr Hosek, 2016-01-13 00:33:20]

Patchset #2 (id:20001) has been deleted

[Petr Hosek, 2016-01-13 00:35:55]

Removed.

https://codereview.chromium.org/1581753002/diff/1/build/config/BUILDCONFIG.gn
File build/config/BUILDCONFIG.gn (right):

https://codereview.chromium.org/1581753002/diff/1/build/config/BUILDCONFIG.gn...
build/config/BUILDCONFIG.gn:97: is_desktop_linux = current_os == "linux"
On 2016/01/12 22:47:11, viettrungluu wrote:
> We should probably get rid of "is_desktop_linux" in that case. (You don't need
> to do that here.

I'll do that in a separate change.

[Petr Hosek, 2016-01-13 02:26:41]

Description was changed from

==========
Remove Chrome OS-related GN build configuration

We don't support Chrome OS, so no need for build configuration.

R=viettrungluu@chromium.org
==========

to

==========
Remove Chrome OS-related GN build configuration

We don't support Chrome OS, so no need for build configuration.

R=viettrungluu@chromium.org

Committed:
https://chromium.googlesource.com/external/mojo/+/57ca940d8410a27fdf517b6194d...
==========

[Petr Hosek, 2016-01-13 02:26:42]

Committed patchset #2 (id:40001) manually as
57ca940d8410a27fdf517b6194d7ecb0f29ecc2a.