Add expectations for three layout tests failing on Site Isolation bots.

Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 661725, 680249, 680307
NOTRY=true

Review-Url: https://codereview.chromium.org/2625233002
Cr-Commit-Position: refs/heads/master@{#443071}
Committed: https://chromium.googlesource.com/chromium/src/+/96557bfa11c102628943935e7906cff4b3b04a74

[alexmos, 2017-01-11 23:41:29]

Description was changed from

==========
Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 680249, 680307
==========

to

==========
Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 680249, 680307
NOTRY=true
==========

[alexmos, 2017-01-11 23:47:41]

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

[alexmos, 2017-01-11 23:47:41]

Lukasz, can you please take a look?

It didn't look like any of these have a quick fix, unfortunately, so I'm
disabling them to get the bots green.

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

https://codereview.chromium.org/2625233002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:16:
crbug.com/661725
http/tests/security/frameNavigation/xss-ALLOWED-top-navigation-after-postMessage.html
[ Failure Timeout ]
I kept the Timeout here since Nate has a CL in progress which will re-enable his
intervention and make this test fail with a Timeout again (w/
--site-per-process).  This will keep the bot green, and I'll need to revisit
this test anyhow as part of user gesture work.

[Łukasz Anforowicz, 2017-01-11 23:52:12]

LGTM - thanks for taking a look at these failures.

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

https://codereview.chromium.org/2625233002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:16:
crbug.com/661725
http/tests/security/frameNavigation/xss-ALLOWED-top-navigation-after-postMessage.html
[ Failure Timeout ]
On 2017/01/11 23:47:41, alexmos wrote:
> I kept the Timeout here since Nate has a CL in progress which will re-enable
his
> intervention and make this test fail with a Timeout again (w/
> --site-per-process).  This will keep the bot green, and I'll need to revisit
> this test anyhow as part of user gesture work.

Acknowledged.

https://codereview.chromium.org/2625233002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:178:
crbug.com/680307 http/tests/security/mixedContent/blob-url-in-iframe.html [
Failure ]
Do we need to replicate the expectations for virtual/mojo-loading as well?

[alexmos, 2017-01-11 23:56:58]

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

https://codereview.chromium.org/2625233002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:178:
crbug.com/680307 http/tests/security/mixedContent/blob-url-in-iframe.html [
Failure ]
On 2017/01/11 23:52:12, Łukasz Anforowicz wrote:
> Do we need to replicate the expectations for virtual/mojo-loading as well?

Indeed, these also fail for Site Isolation Win (though not Linux).  Done. 
Thanks for noticing!

[alexmos, 2017-01-11 23:58:34]

Description was changed from

==========
Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 680249, 680307
NOTRY=true
==========

to

==========
Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 661725, 680249, 680307
NOTRY=true
==========

[alexmos, 2017-01-12 00:01:19]

The CQ bit was checked by alexmos@chromium.org

[alexmos, 2017-01-12 00:01: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/2625233002/#ps20001
(title: "Add virtual/mojo-loading variants")

[commit-bot: I haz the power, 2017-01-12 00:03:08]

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-01-12 00:14:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484179279375810,
"parent_rev": "ccb031cf7c5a129b7059e6b5855f16af12f2a9f3", "commit_rev":
"96557bfa11c102628943935e7906cff4b3b04a74"}

[commit-bot: I haz the power, 2017-01-12 00:15:23]

Description was changed from

==========
Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 661725, 680249, 680307
NOTRY=true
==========

to

==========
Add expectations for three layout tests failing on Site Isolation bots.

This should hopefully get the bots back to green while individual bugs
are being investigated.

BUG=680201, 661725, 680249, 680307
NOTRY=true

Review-Url: https://codereview.chromium.org/2625233002
Cr-Commit-Position: refs/heads/master@{#443071}
Committed:
https://chromium.googlesource.com/chromium/src/+/96557bfa11c102628943935e7906...
==========

[commit-bot: I haz the power, 2017-01-12 00:15:26]

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