New issue
Advanced search Search tips

Issue 769461 link

Starred by 1 user
Status: Available
Owner: ----
Cc:
lfg@chromium.org
wjmaclean@chromium.org
mcnee@chromium.org
wfh@chromium.org
creis@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

WebView tag shouldn't track the owner via the *main* frame process

Project Member Reported by lukasza@chromium.org, Sep 27 2017 
This bug is a followup to https://chromium-review.googlesource.com/c/chromium/src/+/673124/10/extensions/browser/guest_view/web_view/web_view_guest.cc#327

All of these places where WebView tag refers to the owner's WebContents main frame's process are wrong in theory.  This is similar to  issue 769385 .

Fixing this is not as trivial though, since it requires refactoring tracking the owner WebContents to tracking the owner frame.

Comment 1 by lfg@chromium.org, Oct 19 2017

Cc: wjmaclean@chromium.org lfg@chromium.org
Owner: ----
Status: Available (was: Untriaged)
Marking as available, as I'm not sure I'll be able to work on this.

This is essential if we want to support embedding guests inside OOPIFs. MimeHandlerViewGuest already has code to work around issues related to this, but we should make this more generic and have the GuestView tracking the frame that owns it instead of the WebContents.
Project Member

Comment 2 by sheriffbot@chromium.org, Oct 19

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by nasko@google.com, Oct 25

Cc: mcnee@chromium.org
Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)
It looks like this bug is still relevant, so removing the Hotlist-Recharge-Cold label.
►

Sign in to add a comment