Tab crash when creating fetch/XHR from isolated world execution context
Reported by
boris.ok...@ub.io,
Aug 30
|
|||||||
Issue description
UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.106 Safari/537.36
Steps to reproduce the problem:
1. create an isolated world execution context with Page.createIsolatedWorld
2. open DevTools, Console, switch to isolated world execution context
3. send `fetch('https://github.com')`
4. observe the tab crash
What is the expected behavior?
Tab should not crash, the promise should either resolve or reject, the behaviour should match the default execution context
What went wrong?
Tab crashed
Did this work before? No
Chrome version: 70.0.3537.0 Channel: canary
OS Version: OS X 10.13.4
Flash Version:
,
Aug 31
Thanks for filing the issue! @Reporter: Could you please provide sample Test file/URL that reproduces the issue which helps us in further triaging it in better way. Thanks!
,
Aug 31
,
Sep 3
,
Sep 14
,
Sep 19
,
Sep 20
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b8d3bfaa306273d85c542c0aca9352d767a95e33 commit b8d3bfaa306273d85c542c0aca9352d767a95e33 Author: Joey Arhar <jarhar@chromium.org> Date: Thu Sep 20 18:42:19 2018 [DevTools] Fix isolated world crash by always adding a SecurityOrigin Bug: 879219 Change-Id: I6cf01b36326a8b7727a870f2f7e2d061774d1c5b Reviewed-on: https://chromium-review.googlesource.com/1226797 Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Joey Arhar <jarhar@chromium.org> Cr-Commit-Position: refs/heads/master@{#592888} [add] https://crrev.com/b8d3bfaa306273d85c542c0aca9352d767a95e33/third_party/WebKit/LayoutTests/http/tests/inspector-protocol/page/createIsolatedWorld-fetch-expected.txt [add] https://crrev.com/b8d3bfaa306273d85c542c0aca9352d767a95e33/third_party/WebKit/LayoutTests/http/tests/inspector-protocol/page/createIsolatedWorld-fetch.js [modify] https://crrev.com/b8d3bfaa306273d85c542c0aca9352d767a95e33/third_party/blink/renderer/core/inspector/inspector_page_agent.cc
,
Sep 20
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by vamshi.kommuri@chromium.org
, Aug 31