Disable failing/crashing layout tests on Site Isolation bots.

Disable failing/crashing layout tests on Site Isolation bots.

These tests are failing consistently on the Site Isolation FYI bots.
Disabling them while they are investigated and fixed.

BUG=710098, 716085
Review-Url: https://codereview.chromium.org/2845093004
Cr-Commit-Position: refs/heads/master@{#467767}
Committed: https://chromium.googlesource.com/chromium/src/+/94ad18d772d3d2fbf8c084b93af70502432a4256

[nasko, 2017-04-27 17:40:01]

nasko@chromium.org changed reviewers:
+ lukasza@chromium.org

[nasko, 2017-04-27 17:40:01]

Hey Lukasz,
Can you review this CL for me? It disables the currently failing layout tests on
the bots.
Thanks in advance!
Nasko

[Łukasz Anforowicz, 2017-04-27 18:10:01]

LGTM, but I wonder if we should broaden the test exceptions - please see my
comment below.

https://codereview.chromium.org/2845093004/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/FlagExpectations/site-per-process (right):

https://codereview.chromium.org/2845093004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:12:
crbug.com/716085
virtual/mojo-loading/http/tests/feature-policy-experimental-features/vibrate-enabledforself.php
[ Crash ]
I wonder if you also should add [ Failure ] and/or [ Missing ], since I see that
the test also (rarely/intermittently) failed in these modes at
https://storage.googleapis.com/chromium-layout-test-archives/Site_Isolation_L...:

http/tests/feature-policy-experimental-features/vibrate-disabled.php - text
crash crash crash
http/tests/feature-policy-experimental-features/vibrate-enabledforself.php -
missing crash crash crash

[Łukasz Anforowicz, 2017-04-27 18:10:46]

+lunalu@ FYI

[nasko, 2017-04-27 18:50:06]

https://codereview.chromium.org/2845093004/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/FlagExpectations/site-per-process (right):

https://codereview.chromium.org/2845093004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:12:
crbug.com/716085
virtual/mojo-loading/http/tests/feature-policy-experimental-features/vibrate-enabledforself.php
[ Crash ]
On 2017/04/27 18:10:00, Łukasz A. wrote:
> I wonder if you also should add [ Failure ] and/or [ Missing ], since I see
that
> the test also (rarely/intermittently) failed in these modes at
>
https://storage.googleapis.com/chromium-layout-test-archives/Site_Isolation_L...:
> 
> http/tests/feature-policy-experimental-features/vibrate-disabled.php - text
> crash crash crash
> http/tests/feature-policy-experimental-features/vibrate-enabledforself.php -
> missing crash crash crash

Done.

[nasko, 2017-04-27 18:50:18]

The CQ bit was checked by nasko@chromium.org

[nasko, 2017-04-27 18:50:19]

The patchset sent to the CQ was uploaded after l-g-t-m from lukasza@chromium.org
Link to the patchset: https://codereview.chromium.org/2845093004/#ps20001
(title: "Incorporate Lukasz's suggestion.")

[commit-bot: I haz the power, 2017-04-27 18:50:53]

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-04-27 20:29:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493319018756530,
"parent_rev": "42776f8a5b7e07ce63442004737150495eed0db0", "commit_rev":
"94ad18d772d3d2fbf8c084b93af70502432a4256"}

[commit-bot: I haz the power, 2017-04-27 20:29:11]

Description was changed from

==========
Disable failing/crashing layout tests on Site Isolation bots.

These tests are failing consistently on the Site Isolation FYI bots.
Disabling them while they are investigated and fixed.

BUG=710098,716085
==========

to

==========
Disable failing/crashing layout tests on Site Isolation bots.

These tests are failing consistently on the Site Isolation FYI bots.
Disabling them while they are investigated and fixed.

BUG=710098,716085

Review-Url: https://codereview.chromium.org/2845093004
Cr-Commit-Position: refs/heads/master@{#467767}
Committed:
https://chromium.googlesource.com/chromium/src/+/94ad18d772d3d2fbf8c084b93af7...
==========

[commit-bot: I haz the power, 2017-04-27 20:29:12]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/94ad18d772d3d2fbf8c084b93af7...