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
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).
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.
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.
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.
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.
patch -p0 "http/tests/security/isolatedWorld/all-window-properties.html" returned 1. Pass --force to ignore patch failures.
Yeah, ok. I'll land this manually.
Ack! Hans! The ChangeLog contains tabs! http://trac.webkit.org/changeset/47809