Bug 27992 – [V8] DOM Storage + Isolated worlds don't play well together (at least in Chromium layout tests)

• Component: New Bugs
• Status: RESOLVED FIXED

[Jeremy Orlow]

The following has only been observed in the Chromium test shell.

LayoutTests/http/tests/security/isolatedworld/all-window-properties.html and LayoutTests/http/tests/security/isolatedworld/all-window-prototypes.html are failing 
when I enable local storage and session storage in the test shell.  It's not clear whether this is a real issue or not.

The expected result is simply printing "Done.".  Anything else that's printed are properties of the window object which shouldn't be there.

Here's a diff from one of the two results:
--- src/webkit/Debug/layout-test-results/LayoutTests/http/tests/security/isolatedworld/all-window-properties-expected.txt 
+++ src/webkit/Debug/layout-test-results/LayoutTests/http/tests/security/isolatedworld/all-window-properties-actual.txt 
@@ -1,1 +1,3 @@
+localStorage: FAIL: Visible in isolated world.
+sessionStorage: FAIL: Visible in isolated world.
 Done.


I'm going to exclude these two files in the test exceptions for now.

Chromium bug: http://code.google.com/p/chromium/issues/detail?id=18412

[Jeremy Orlow]

Ping (because I'm concerned this could be a somewhat serious bug...but I just don't understand enough about isolated worlds to know for sure).

[Adam Barth]

Yeah, sorry.  I've been working on other stuff.  I need to do another round of isolated worlds work...  It might be a couple weeks though.

[Yaar Schnitman]

Created attachment 38639 [details]
New tests and tests re-baselining

It was decided that isolated scripts should be able to read properties of local and session storage set by normal scripts.
 
We had to fix the layout tests and add new ones:
1. isolatedWorld/all-window-properties.html: skips localStorage and sessionStorage
2. isolatedWorld/storage-properties.htmL: tests that properties can indeed be read.
3. isolatedWorld/storage-prototype.htmL: tests that properties on the prototype CAN NOT be read.

Also added new files to the platform-specific Skipped files.

[Adam Barth]

Comment on attachment 38639 [details]
New tests and tests re-baselining

Great!  Thanks.

I'm marking this for the commit queue, but it's not likely to work because the patch isn't based in the right directory.  In the future, consider using "WebKitTools/Scripts/svn-create-patch" or "WebKitTools/Script/bugzilla-tool post-diff" to create the patch.

[Eric Seidel (no email)]

Comment on attachment 38639 [details]
New tests and tests re-baselining

Rejecting patch 38639 from commit-queue.  This patch will require manual commit.

Patch https://bugs.webkit.org/attachment.cgi?id=38639 from bug 27992 failed to download and apply.

[Eric Seidel (no email)]

patch -p0 "http/tests/security/isolatedWorld/all-window-properties.html" returned 1.  Pass --force to ignore patch failures.

[Adam Barth]

Yeah, ok.  I'll land this manually.

[Adam Barth]

Ack!  Hans!  The ChangeLog contains tabs!

http://trac.webkit.org/changeset/47809