New issue
Advanced search Search tips

Issue 616626 link

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

Blocking:
issue 477150



Sign in to add a comment

OOPIF support in layout tests harness: allow_universal_access_from_file_urls

Project Member Reported by lukasza@chromium.org, Jun 1 2016 
Repro:
$ ninja -C out/gn ... blink_tests
$ third_party/WebKit/Tools/Scripts/run-webkit-tests -t gn -v --no-retry-failures --additional-drt-flag=--site-per-process http/tests/local/drag-over-remote-content.html

Expected behavior: test passes

Actual behavior: test fails

Comment 1 by lukasza@chromium.org, Jun 1 2016 

With --site-per-process, trying to access |frames[0].document.body| (in http/tests/local/drag-over-remote-content.html) throws a javascript exception: Blocked a frame with origin "file://" from accessing a cross-origin frame.

Without --site-per-process things surprisingly work.  I am guessing this is because of allow_universal_access_from_file_urls option used by layout tests.

Note that layout tests under http/tests/local path get served over file:// rather than over http.  I couldn't find where this protocol choice takes place, but I did find comments about this in components/test_runner/test_preferences.cc.
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 3 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8b3aa9314e3f33a6ed7740d78241e1b633595893

commit 8b3aa9314e3f33a6ed7740d78241e1b633595893
Author: lukasza <lukasza@chromium.org>
Date: Fri Jun 03 16:44:11 2016

Adding specific bugs for layout test failures with --site-per-process flag.

BUG= 615156 ,  616608 , 616626

Review-Url: https://codereview.chromium.org/2037743003
Cr-Commit-Position: refs/heads/master@{#397729}

[modify] https://crrev.com/8b3aa9314e3f33a6ed7740d78241e1b633595893/third_party/WebKit/LayoutTests/FlagExpectations/site-per-process

Comment 3 by lukasza@chromium.org, Mar 9 2018

Labels: Test-Layout
Summary: OOPIF support in layout tests harness: allow_universal_access_from_file_urls (was: With --site-per-process layout test cannot access crossSiteFrame.document.body even with allow_universal_access_from_file_urls)
►

Sign in to add a comment