New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 740984 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug
Proj-XR
Proj-XR-VR

Blocked on:
issue 793430



Sign in to add a comment

VR: Cannot interact with cross-origin iframes while in VR browsing mode

Project Member Reported by mthiesse@chromium.org, Jul 11 2017

Issue description

Cross-origin iframes cannot be interacted with using the VR controller. Clicking on them does nothing, and attempting to scroll them scrolls the main page.
 
Cc: asimjour@chromium.org
Cc: -asimjour@chromium.org
Owner: asimjour@chromium.org
Components: -Internals>VR UI>Browser>VR
Status: Started (was: Available)
Blockedon: 743059
Labels: -Pri-1 -M-61 M-62 Pri-2
It only happens with the runtime flag --site-per-process.
I change the priority 1->2 M61->M62, but will keep working on it.
Labels: -M-62 M-63
Cc: asimjour@chromium.org dcheng@chromium.org
Components: Internals>Sandbox>SiteIsolation
Labels: -Pri-2 -M-63 M-64 Pri-1
Owner: ----
Status: Available (was: Started)
We need to fix this as site isolation is being gradually turned on.

Comment 9 by nasko@chromium.org, Oct 11 2017

Cc: kenrb@chromium.org
Labels: -M-64 M-65
The Android rollout will follow desktop, but we should still fix this soon.

Comment 11 by kenrb@chromium.org, Nov 16 2017

asimjour@ was looking into this, was there any progress toward a solution?

Comment 12 by bshe@chromium.org, Dec 7 2017

Labels: -Proj-VR-Shell Proj-VR
Owner: asimjour@chromium.org
Amir, just wondering if there is any update on this?
Also, has --site-per-process turned on by default on Android?
RE: has --site-per-process turned on by default on Android?

Up to date there are no OOPIFs / isolation modes shipping on Android.  The recently announced site-per-process enterprise policy [1] is restricted to desktops (ChromeOS, Windows, Mac, Linux).

[1] https://www.blog.google/topics/connected-workspaces/security-enhancements-and-more-enterprise-chrome-browser-customers/
One more data point: --site-per-process does however work well on Android, so it definitely can be experimented/worked with. chrome://flags/#enable-site-per-process
Owner: mthiesse@chromium.org
Status: Started (was: Available)
I'm changing how our input is routed (to do it through Android UI). This *should* fix any issues we have here.
I agree that's the best way to fix it. If input is arriving via calls to ui::ViewClient, then from the point of view of content, they are just regular user input and can be handled the same.
Related "PSA: Please make sure your code works with OOPIFs": https://groups.google.com/a/chromium.org/d/topic/blink-dev/fCTPcKBlmJ4/discussion
Blockedon: -743059 793430
Labels: -M-65 M-66
Won't get this in for 65. Was there any reason this was urgent for 65?
RE: prioritization / context question from #19

Site Isolation (which leads to OOPIFs) is part of Google's response to the Spectre attack.  After the Spectre announcements (e.g. https://support.google.com/faqs/answer/7622138#chrome) there is an increase in the number of Site Isolation users and we expect to see more usage growth going forward.  AFAIK, the Oculus browser is based on Chromium and I think it also is considering using Site Isolation.
lukasza, is Site Isolation still behind a flag for M-65?

If so, I don't think it's worth us rushing our fix for M-65, as it's coming in way too hot and is likely to cause more problems than it solves if we're not careful.
Yes, Site Isolation is behind an opt-in flag for M65 (and/or behind an opt-in Enterprise Policy, but this particular policy only applies to desktop platforms, not to Android).
Status: Fixed (was: Started)
Labels: Test-Manual
User Scenario: You have to enable the "Top document isolation" flag, and then you can go to a page like https://jsfiddle.net/fsbhjy9t/1/ and make sure you can interact with the wikipedia window in the corner.
Labels: -Test-Manual Test-Complete
Status: Verified (was: Fixed)
Added test case "Interact with content in iframes" to "VR Browsing manual test plan" document.  Verified that the feature was working as expected on build 66.0.3359.10.

Sign in to add a comment