Disable tests on VS 2017 Preview, code-gen bug

Disable tests on VS 2017 Preview, code-gen bug

VS 2017 has a code-gen bug, discussed and reported here:

https://developercommunity.visualstudio.com/content/problem/40904/bad-code-gen-in-chromes-mojo-public-bindings-unitt.html

As of VS 2017 Update 3 Preview 1 the bug is still not fixed, so this CL
adjusts the checks to continue disabling the failing tests to allow
testing with this VC version. The next preview should have the fix -
probably. crrev.com/2826713003 disabled the tests for VS 2017 RTM.

R=jam@chromium.org
BUG=723893
Review-Url: https://codereview.chromium.org/2909593002
Cr-Commit-Position: refs/heads/master@{#475017}
Committed: https://chromium.googlesource.com/chromium/src/+/6d1398efad86b7c862e83719602aed55c35d0e28

[brucedawson, 2017-05-26 03:14:08]

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

[commit-bot: I haz the power, 2017-05-26 03:14:58]

Dry run: CQ is trying da patch.

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

[brucedawson, 2017-05-26 03:15:36]

This is an update to a previous change to work-around a VS 2017 compiler bug.

This change is needed to let us test VS 2017 Preview - it was tested separately
in crrev.com/2904733004, but I'd rather land it separately. PTAL.

[commit-bot: I haz the power, 2017-05-26 04:31:19]

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

[commit-bot: I haz the power, 2017-05-26 04:31:20]

Dry run: This issue passed the CQ dry run.

[jam, 2017-05-26 14:35:57]

lgtm

[brucedawson, 2017-05-26 15:55:17]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2017-05-26 15:55:39]

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, 2017-05-26 16:00:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495814117373440, "parent_rev":
"e459910b090911bb1fca28aeffa46d16d929925b", "commit_rev":
"6d1398efad86b7c862e83719602aed55c35d0e28"}

[commit-bot: I haz the power, 2017-05-26 16:01:07]

Description was changed from

==========
Disable tests on VS 2017 Preview, code-gen bug

VS 2017 has a code-gen bug, discussed and reported here:

https://developercommunity.visualstudio.com/content/problem/40904/bad-code-ge...

As of VS 2017 Update 3 Preview 1 the bug is still not fixed, so this CL
adjusts the checks to continue disabling the failing tests to allow
testing with this VC version. The next preview should have the fix -
probably. crrev.com/2826713003 disabled the tests for VS 2017 RTM.

R=jam@chromium.org
BUG=723893
==========

to

==========
Disable tests on VS 2017 Preview, code-gen bug

VS 2017 has a code-gen bug, discussed and reported here:

https://developercommunity.visualstudio.com/content/problem/40904/bad-code-ge...

As of VS 2017 Update 3 Preview 1 the bug is still not fixed, so this CL
adjusts the checks to continue disabling the failing tests to allow
testing with this VC version. The next preview should have the fix -
probably. crrev.com/2826713003 disabled the tests for VS 2017 RTM.

R=jam@chromium.org
BUG=723893

Review-Url: https://codereview.chromium.org/2909593002
Cr-Commit-Position: refs/heads/master@{#475017}
Committed:
https://chromium.googlesource.com/chromium/src/+/6d1398efad86b7c862e83719602a...
==========

[commit-bot: I haz the power, 2017-05-26 16:01:07]

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