Resolve console output in postMessage tests with site-per-process.

Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test does it.

BUG=614413
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/6b50e7bb8127b82d23787cb4ee1975a6e3758c82
Cr-Commit-Position: refs/heads/master@{#396055}

[dgozman, 2016-05-25 22:23:57]

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

[dgozman, 2016-05-25 22:23:58]

Could you please take a look?
What's the name of site-per-process bot to use in CQ_INCLUDE_TRYBOTS?

[Łukasz Anforowicz, 2016-05-25 22:35:30]

lgtm with nits.  Thanks!

You can use:
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

I've also tried patching in your CL and postMessage tests are now passing with
--additional-drt-flag=--site-per-process:
$ third_party/WebKit/Tools/Scripts/run-webkit-tests -t gn -v
--additional-drt-flag=--site-per-process --no-retry-failures
http/tests/security/postMessage

https://codereview.chromium.org/2015713002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/security/postMessage/invalid-origin-throws-exception.html
(right):

https://codereview.chromium.org/2015713002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/security/postMessage/invalid-origin-throws-exception.html:8:
// We have different console output in --site-per-process mode.
nit: Could you add a reference to https://crbug.com/614413 ?

https://codereview.chromium.org/2015713002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/security/postMessage/target-origin.html
(right):

https://codereview.chromium.org/2015713002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/security/postMessage/target-origin.html:8:
// We have different console output in --site-per-process mode.
nit: Could you add a reference to https://crbug.com/614413 ?

[alexmos, 2016-05-25 23:04:16]

LGTM

[dgozman, 2016-05-25 23:10:49]

https://codereview.chromium.org/2015713002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/security/postMessage/invalid-origin-throws-exception.html
(right):

https://codereview.chromium.org/2015713002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/security/postMessage/invalid-origin-throws-exception.html:8:
// We have different console output in --site-per-process mode.
On 2016/05/25 22:35:30, Łukasz Anforowicz wrote:
> nit: Could you add a reference to https://crbug.com/614413 ?

Done.

[dgozman, 2016-05-25 23:11:03]

Description was changed from

==========
Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test
does it.

BUG=614413
==========

to

==========
Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test
does it.

BUG=614413
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[dgozman, 2016-05-25 23:11:07]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-05-25 23:11:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
lukasza@chromium.org, alexmos@chromium.org
Link to the patchset: https://codereview.chromium.org/2015713002/#ps20001
(title: "bug reference")

[commit-bot: I haz the power, 2016-05-25 23:11:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2015713002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2015713002/20001

[commit-bot: I haz the power, 2016-05-26 00:35:05]

Description was changed from

==========
Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test
does it.

BUG=614413
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test
does it.

BUG=614413
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-05-26 00:35:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-26 00:38:02]

Description was changed from

==========
Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test
does it.

BUG=614413
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Resolve console output in postMessage tests with site-per-process.

Regular tests do not dump console output anymore, and special same-site test
does it.

BUG=614413
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/6b50e7bb8127b82d23787cb4ee1975a6e3758c82
Cr-Commit-Position: refs/heads/master@{#396055}
==========

[commit-bot: I haz the power, 2016-05-26 00:38:04]

Patchset 2 (id:??) landed as
https://crrev.com/6b50e7bb8127b82d23787cb4ee1975a6e3758c82
Cr-Commit-Position: refs/heads/master@{#396055}