[Presentation API] Move sandbox check to PresentationRequest constructor.

[Presentation API] Move sandbox check to PresentationRequest constructor.

Fix PresentationRequest_sandboxing_error.html.

[Patched from https://codereview.chromium.org/2780583003]

BUG=705170
Review-Url: https://codereview.chromium.org/2782663002
Cr-Commit-Position: refs/heads/master@{#460460}
Committed: https://chromium.googlesource.com/chromium/src/+/e29fdab2cf168e2a4de8709aea221cf5ef65cf5f

[mark a. foltz, 2017-03-28 18:20:11]

Description was changed from

==========
[Presentation API] Move sandbox check to PresentationRequest constructor.

Fix PresentationRequest_sandboxing_error.html.

BUG=705170
==========

to

==========
[Presentation API] Move sandbox check to PresentationRequest constructor.

Fix PresentationRequest_sandboxing_error.html.

[Patched from https://codereview.chromium.org/2780583003]

BUG=705170
==========

[mark a. foltz, 2017-03-28 18:21:05]

mfoltz@chromium.org changed reviewers:
+ imcheng@chromium.org

[mark a. foltz, 2017-03-28 18:34:22]

We can probably remove the iframe*.html tests.  They look redundant with the
tests added in this patch.

[imcheng, 2017-03-28 18:50:22]

lgtm

[mark a. foltz, 2017-03-28 20:01:07]

On 2017/03/28 at 18:50:22, imcheng wrote:
> lgtm

PTAL at the current patch which replaces the existing tests.

[imcheng, 2017-03-28 20:24:33]

still lgtm

https://codereview.chromium.org/2782663002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/presentation/presentation-request-iframe-default-success.html
(right):

https://codereview.chromium.org/2782663002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/presentation/presentation-request-iframe-default-success.html:16:
window.addEventListener("message", t.step_func_done(e => {
nit: use single quote to be consistent here and below. Same for other files.

[mark a. foltz, 2017-03-28 21:39:54]

The CQ bit was checked by mfoltz@chromium.org

[mark a. foltz, 2017-03-28 21:39:54]

The patchset sent to the CQ was uploaded after l-g-t-m from imcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2782663002/#ps40001
(title: "Respond to imcheng@ comments")

[commit-bot: I haz the power, 2017-03-28 21:40:11]

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

[mark a. foltz, 2017-03-28 21:40:11]

https://codereview.chromium.org/2782663002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/presentation/presentation-request-iframe-default-success.html
(right):

https://codereview.chromium.org/2782663002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/presentation/presentation-request-iframe-default-success.html:16:
window.addEventListener("message", t.step_func_done(e => {
On 2017/03/28 at 20:24:32, imcheng wrote:
> nit: use single quote to be consistent here and below. Same for other files.

Done.

[commit-bot: I haz the power, 2017-03-28 22:07:26]

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

[commit-bot: I haz the power, 2017-03-28 22:07:27]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[mark a. foltz, 2017-03-28 22:29:16]

The CQ bit was checked by mfoltz@chromium.org

[commit-bot: I haz the power, 2017-03-28 22:30:03]

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-03-28 22:53:32]

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

[commit-bot: I haz the power, 2017-03-28 22:53:33]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[mark a. foltz, 2017-03-29 17:04:44]

The CQ bit was checked by mfoltz@chromium.org

[commit-bot: I haz the power, 2017-03-29 17:05:13]

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-03-29 18:20:12]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490807084523850,
"parent_rev": "6d71d9c5b9a95b7476beefaa9ace1fc8b54d5b43", "commit_rev":
"e29fdab2cf168e2a4de8709aea221cf5ef65cf5f"}

[commit-bot: I haz the power, 2017-03-29 18:21:48]

Description was changed from

==========
[Presentation API] Move sandbox check to PresentationRequest constructor.

Fix PresentationRequest_sandboxing_error.html.

[Patched from https://codereview.chromium.org/2780583003]

BUG=705170
==========

to

==========
[Presentation API] Move sandbox check to PresentationRequest constructor.

Fix PresentationRequest_sandboxing_error.html.

[Patched from https://codereview.chromium.org/2780583003]

BUG=705170

Review-Url: https://codereview.chromium.org/2782663002
Cr-Commit-Position: refs/heads/master@{#460460}
Committed:
https://chromium.googlesource.com/chromium/src/+/e29fdab2cf168e2a4de8709aea22...
==========

[commit-bot: I haz the power, 2017-03-29 18:21:49]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/e29fdab2cf168e2a4de8709aea22...