HeadlessWebContentsImpl incorrectly assumes there is only one renderer per tab |
|||
Issue descriptionIt's going to break when OOPIF lands. We should fix this!
,
Apr 27 2017
Can you point out where we assume there's only one renderer?
,
Apr 27 2017
As noted in https://codereview.chromium.org/2830753004/diff/100001/headless/lib/browser/headless_web_contents_impl.h#newcode132, HeadlessWebContentsImpl::render_process_host_ is used in a way that assumes a WebContents has only one process. That CL is also introducing more code making the same assumption (e.g., the test). Not sure if there are additional places.
,
May 15 2017
,
Sep 28 2017
FYI, this is cl will make headless_web_contents_impl be associated with the main render process host: https://chromium-review.googlesource.com/c/chromium/src/+/673124
,
Jul 31
|
|||
►
Sign in to add a comment |
|||
Comment 1 by creis@chromium.org
, Apr 27 2017Components: Internals>Sandbox>SiteIsolation
Summary: HeadlessWebContentsImpl incorrectly assumes there is only one renderer per tab (was: HeadlessWebContentsImpl incorrectly assumes there is only one renderer)