Ozone Linux: Disable some mash tests causing configuration errors

Ozone Linux: Disable some mash tests causing configuration errors

After [1], new ChromeOS-specific files are built when
enable_package_mash_services=1. This is the case for Ozone Linux and it
caused the following error to happen on the build bot:

  ERROR at //ash/mus/BUILD.gn:11:1: Assertion failed.
    assert(is_chromeos)
      ^-----
        See //services/service_manager/public/service_manifest.gni:136:19: which caused the file to be included.
                  deps += [ manifest_target ]
                                    ^--------------

This CL disables corresponding conditionals of [1] when is_chromeos=0 so
that Ozone Linux builds again.

[1] https://codereview.chromium.org/2680283003/

BUG=664570
Review-Url: https://codereview.chromium.org/2694683002
Cr-Commit-Position: refs/heads/master@{#449961}
Committed: https://chromium.googlesource.com/chromium/src/+/59d468305aab2c224ab5bd34a3415869a3e78351

[fwang, 2017-02-13 08:37:20]

The CQ bit was checked by fwang@igalia.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-13 08:37:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fwang, 2017-02-13 08:38:14]

fwang@igalia.com changed reviewers:
+ sky@chromium.org

[fwang, 2017-02-13 08:38:15]

PTAL

[commit-bot: I haz the power, 2017-02-13 10:02:39]

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

[commit-bot: I haz the power, 2017-02-13 10:02:40]

Dry run: This issue passed the CQ dry run.

[kylechar, 2017-02-13 15:13:23]

kylechar@chromium.org changed reviewers:
+ kylechar@chromium.org

[kylechar, 2017-02-13 15:13:23]

lgtm

[fwang, 2017-02-13 15:18:46]

The CQ bit was checked by fwang@igalia.com

[commit-bot: I haz the power, 2017-02-13 15:19:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[kylechar, 2017-02-13 15:19:54]

Ah sorry, you'll still need an LGTM from sky for the presubmit.

[fwang, 2017-02-13 15:22:18]

On 2017/02/13 15:19:54, kylechar wrote:
> Ah sorry, you'll still need an LGTM from sky for the presubmit.

Mmh, chrome/test/OWNERS has:

# If you're doing structural changes get a review from one of the OWNERS.
per-file BUILD.gn=*

[commit-bot: I haz the power, 2017-02-13 15:23:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486999126390200, "parent_rev":
"ee2192b1f215da8c710bdf47d2643f43220db62d", "commit_rev":
"59d468305aab2c224ab5bd34a3415869a3e78351"}

[kylechar, 2017-02-13 15:23:02]

On 2017/02/13 15:22:18, fwang wrote:
> On 2017/02/13 15:19:54, kylechar wrote:
> > Ah sorry, you'll still need an LGTM from sky for the presubmit.
> 
> Mmh, chrome/test/OWNERS has:
> 
> # If you're doing structural changes get a review from one of the OWNERS.
> per-file BUILD.gn=*

Ah, this is fine then :)

[commit-bot: I haz the power, 2017-02-13 15:23:34]

Description was changed from

==========
Ozone Linux: Disable some mash tests causing configuration errors

After [1], new ChromeOS-specific files are built when
enable_package_mash_services=1. This is the case for Ozone Linux and it
caused the following error to happen on the build bot:

  ERROR at //ash/mus/BUILD.gn:11:1: Assertion failed.
    assert(is_chromeos)
      ^-----
        See //services/service_manager/public/service_manifest.gni:136:19: which
caused the file to be included.
                  deps += [ manifest_target ]
                                    ^--------------

This CL disables corresponding conditionals of [1] when is_chromeos=0 so
that Ozone Linux builds again.

[1] https://codereview.chromium.org/2680283003/

BUG=664570
==========

to

==========
Ozone Linux: Disable some mash tests causing configuration errors

After [1], new ChromeOS-specific files are built when
enable_package_mash_services=1. This is the case for Ozone Linux and it
caused the following error to happen on the build bot:

  ERROR at //ash/mus/BUILD.gn:11:1: Assertion failed.
    assert(is_chromeos)
      ^-----
        See //services/service_manager/public/service_manifest.gni:136:19: which
caused the file to be included.
                  deps += [ manifest_target ]
                                    ^--------------

This CL disables corresponding conditionals of [1] when is_chromeos=0 so
that Ozone Linux builds again.

[1] https://codereview.chromium.org/2680283003/

BUG=664570

Review-Url: https://codereview.chromium.org/2694683002
Cr-Commit-Position: refs/heads/master@{#449961}
Committed:
https://chromium.googlesource.com/chromium/src/+/59d468305aab2c224ab5bd34a341...
==========

[commit-bot: I haz the power, 2017-02-13 15:23:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/59d468305aab2c224ab5bd34a341...

[sky, 2017-02-13 18:01:39]

LGTM