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

Issue 4360 link

Starred by 54 users

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Feb 2010
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature

Blocked on:
issue 16254
issue 16876
issue 16877
issue 18729
issue 19972
issue 20055
issue 26955

Restricted
  • Only users with Commit permission may comment.



Sign in to add a comment

Enable HTML5 name/value storage APIs (DOM storage/Localstorage)

Project Member Reported by darin@chromium.org, Nov 12 2008

Issue description

Enable HTML5 name/value storage APIs

http://www.w3.org/TR/html5/structured.html#storage

 

Comment 1 by jon@chromium.org, Dec 4 2008

Labels: -Type-Bug Type-Feature
Labels: Mstone-2.0
Status: Started

Comment 4 by jon@chromium.org, Mar 17 2009

Labels: -Mstone-2.0 Mstone-2.1
Moving out of 2.0.

Comment 6 by jorlow@chromium.org, Apr 24 2009

The spec has since moved here: http://dev.w3.org/html5/webstorage/#storage

Comment 7 by jorlow@chromium.org, Apr 24 2009

On the WebKit side, this will require some refactoring 
(https://bugs.webkit.org/show_bug.cgi?id=25376) and v8 bindings.

Comment 8 by jon@chromium.org, May 13 2009

Labels: -Mstone-2.1 Mstone-3.0
Moving out of Mstone:2.1 as there just isn't enough time to work on this 
issue.
Labels: mstone4
Labels: mstone-4
Labels: -mstone4
Blockedon: 14006
http://crbug/14006 is a component of this bug.
https://bugs.webkit.org/show_bug.cgi?id=25376 is another.

Comment 13 by dhw@chromium.org, Jul 1 2009

Summary: Enable HTML5 name/value storage APIs (DOM storage/Localstorage)
Blockedon: 16254
Blockedon: -14006
Most of the upstream changes are now in.

Working on the Chromium side of things.  Work is happening here: 
http://codereview.chromium.org/147248
Blockedon: 16876 16877
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=21059 

------------------------------------------------------------------------
r21059 | jorlow@chromium.org | 2009-07-18 10:02:08 -0700 (Sat, 18 Jul 2009) | 8 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/render_view_host_delegate_helper.cc?r1=21059&r2=21058
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_switches.cc?r1=21059&r2=21058
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_switches.h?r1=21059&r2=21058
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/render_messages.h?r1=21059&r2=21058
   A http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageNamespaceProxy.cpp
   A http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageNamespaceProxy.h
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webpreferences.h?r1=21059&r2=21058
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.cc?r1=21059&r2=21058
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/webkit.gyp?r1=21059&r2=21058

Enable DOM_STORAGE in our build.  Put LocalStorage and SessionStorage behind their own flags.  Add the beginnings of StorageNamespaceProxy since it implements WebCore::StorageNamespace::____StorageNamespace and we'd get link errors otherwise.

--enable-local-storage and --enable-session-storage are the new flags.  If you enable them and try to use DOM Storage, Chromium will crash.

BUG= 4360 
TEST=none

Review URL: http://codereview.chromium.org/149792
------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=21157 

------------------------------------------------------------------------
r21157 | jorlow@chromium.org | 2009-07-20 20:23:40 -0700 (Mon, 20 Jul 2009) | 2 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/render_view_host_delegate_helper.cc?r1=21157&r2=21156
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_switches.cc?r1=21157&r2=21156
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_switches.h?r1=21157&r2=21156
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/render_messages.h?r1=21157&r2=21156
   A http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageNamespaceProxy.cpp
   A http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageNamespaceProxy.h
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webpreferences.h?r1=21157&r2=21156
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.cc?r1=21157&r2=21156
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/webkit.gyp?r1=21157&r2=21156

Enable DOM_STORAGE in our build.  Put LocalStorage and SessionStorage behind their own flags.  Add the beginnings of StorageNamespaceProxy since it implements WebCore::StorageNamespace::____StorageNamespace and we'd get link errors otherwise.--enable-local-storage and --enable-session-storage are the new flags.  If you enable them and try to use DOM Storage, Chromium will crash.Originally Committed in http://src.chromium.org/viewvc/chrome?view=rev&revision=21059 but then backed out due to include path issues.BUG=4360TEST=none
Review URL: http://codereview.chromium.org/159059
------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=21346 

------------------------------------------------------------------------
r21346 | jorlow@chromium.org | 2009-07-22 17:09:02 -0700 (Wed, 22 Jul 2009) | 11 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc?r1=21346&r2=21345
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.h?r1=21346&r2=21345
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/webkit_thread.cc?r1=21346&r2=21345
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/webkit_thread.h?r1=21346&r2=21345
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_dispatcher_host.cc?r1=21346&r2=21345
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_message_filter.cc?r1=21346&r2=21345
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/render_messages_internal.h?r1=21346&r2=21345

One part of many for enabling DOM Storage.

Add browser-process IPC code + tweak the WebKit Thread.  Note that this code can't possibly be called/run yet.

WebKitThread now has a PostIOThreadTask methods that can safely post back to the IO thread.  This should be used for all WebKit thread -> the rest of the world communication.

There are many TODOs in this code that should be resolved before this is shipped without a --enable-local-storage flag.

BUG= 4360 
TEST=none
Review URL: http://codereview.chromium.org/155845
------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=21495 

------------------------------------------------------------------------
r21495 | jorlow@chromium.org | 2009-07-23 18:22:51 -0700 (Thu, 23 Jul 2009) | 10 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/browser_webkitclient_impl.cc?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/browser_webkitclient_impl.h?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_sandbox_host_linux.cc?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome.gyp?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webkitclient_impl.cc?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webkitclient_impl.h?r1=21495&r2=21494
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webstoragearea_impl.cc
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webstoragearea_impl.h
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webstoragenamespace_impl.cc
   A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webstoragenamespace_impl.h
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/worker/worker_webkitclient_impl.cc?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/worker/worker_webkitclient_impl.h?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/public/WebKitClient.h?r1=21495&r2=21494
   A http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageAreaProxy.cpp
   A http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageAreaProxy.h
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageNamespaceProxy.cpp?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/StorageNamespaceProxy.h?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_shell_webkit_init.h?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_worker/test_worker_main.cc?r1=21495&r2=21494
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/webkit.gyp?r1=21495&r2=21494

The final CL for plumbing DOM Storage.

Add webKitClient plumbing for getting/creating storage namespaces.  Add a chromium implementation for WebStorageArea and WebStorageNamespace which communicates via IPC with the dom_storage_dispatcher_host in the browser process.  Flesh out the StorageAreaProxy and StorageNamespaceProxy to use the aforementioned implementations.

The WebStorageArea implementation includes decently aggressive caching optimizations.  There's still a lot of work to do, though.

BUG= 4360 
TEST=none

Review URL: http://codereview.chromium.org/147248
------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22029 

------------------------------------------------------------------------
r22029 | jorlow@chromium.org | 2009-07-29 17:27:38 -0700 (Wed, 29 Jul 2009) | 6 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/layout_tests/test_expectations.txt?r1=22029&r2=22028
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_shell.cc?r1=22029&r2=22028

List individual storage related layout tests rather than excluding them all.  Run all the dom storage tests that don't rely on events (which are still unsupported).  Enable local storage and session storage in the test shell.

TEST=none
BUG= 4360 

Review URL: http://codereview.chromium.org/160355
------------------------------------------------------------------------

Comment 24 by dhw@chromium.org, Aug 4 2009

 Issue 18327  has been merged into this issue.
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22452 

------------------------------------------------------------------------
r22452 | jorlow@chromium.org | 2009-08-04 18:15:54 -0700 (Tue, 04 Aug 2009) | 16 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/browser_webkitclient_impl.cc?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/browser_webkitclient_impl.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.cc?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/in_process_webkit/dom_storage_dispatcher_host.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_sandbox_host_linux.cc?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webkitclient_impl.cc?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_webkitclient_impl.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/worker/worker_webkitclient_impl.cc?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/worker/worker_webkitclient_impl.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/public/WebKitClient.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/ChromiumBridge.cpp?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/WebKit.cpp?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webkitclient_impl.cc?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webkitclient_impl.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_shell_webkit_init.h?r1=22452&r2=22451
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_worker/test_worker_main.cc?r1=22452&r2=22451

This CL is doing a bunch of Misc work to make LocalStorage data persist.

First of all, this allows WebKit clients to specify whether or not the VFS should be used.  In the browser process, we never want it to be.

Next, this allows WebKit clients to specify the behavior of WebKit's FileSystem code.  By default, they should all be NOT_REACHED().  The browser process implements the two of these I need for LocalStorage, but it'll be really easy for the rest to be implemented as needed.

Next, this adds a function that storage routines can call to ensure that lazily initialized stuff that must be initialized on the main WebKit thread is initialized.  Right now, this is just used to initialize the UTF8 conversion tables, but I'm sure there'll be more overt time.

Lastly, this uses the profile directory stored by webkit_context_ to derive the path LocalStorage should use.

This CL also cleans up a few minor style issues as it goes.

TEST=none
BUG= 4360 

Review URL: http://codereview.chromium.org/159778
------------------------------------------------------------------------

Blockedon: 18729
Labels: HTML5
Blockedon: 19972
Blockedon: 20055
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=25669 

------------------------------------------------------------------------
r25669 | japhet@chromium.org | 2009-09-08 15:16:05 -0700 (Tue, 08 Sep 2009) | 10 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/DEPS?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/base/clipboard.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/base/clipboard_linux.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/base/clipboard_mac.mm?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/base/clipboard_unittest.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/base/clipboard_win.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/autocomplete/autocomplete_edit_view_mac.mm?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/autocomplete/autocomplete_edit_view_win.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_utils.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_view_host.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_gtk.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/render_widget_host_view_gtk.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_message_filter.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_message_filter.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/resource_message_filter_gtk.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/render_messages.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/render_messages_internal.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/render_view.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/render_view.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/renderer_glue.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/views/controls/textfield/native_textfield_win.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/views/view_unittest.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/public/WebClipboard.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/public/WebViewClient.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/AssertMatchingEnums.cpp?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/ChromiumBridge.cpp?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webclipboard_impl.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webclipboard_impl.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webkit_glue.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_delegate.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/layout_tests/test_expectations.txt?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/mock_webclipboard_impl.cc?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/mock_webclipboard_impl.h?r1=25669&r2=25668
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/simple_clipboard_impl.cc?r1=25669&r2=25668

Roll webkit deps 48155:48185 and remove a couple of passing tests from test_expectations.txt.

Also, merge in http://codereview.chromium.org/174367 (original author: vandebo@chromium.org), which is the downstream half of r48168.

BUG= 4360 
BUG= 21228 
BUG= 18792 
TEST=none
TBR=eroman

------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=29501 

------------------------------------------------------------------------
r29501 | yutak@chromium.org | 2009-10-20 02:18:27 -0700 (Tue, 20 Oct 2009) | 7 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/layout_tests/test_expectations.txt?r1=29501&r2=29500

Another round of test_expectations clean up.

Remove test expectations that are consistently passing, and add a platform tag to another test.

BUG= 24930 , 4360 
TEST=none
Review URL: http://codereview.chromium.org/306002
------------------------------------------------------------------------

Just as a general status update: Things are shaping up.  I have a couple changes in 
flight that need to land, I need to fix a couple crashers that have been recently 
discovered, and then I need to beef up testing a bit.  But I we're close to this being 
enabled by default in the dev channel.
Labels: LayoutTests
Still left to do: SessionStorage + making LocalStorage not racy.  (The latter is not done 
because we still haven't decided on a solution in standards land.)
Blockedon: 26955
Labels: -mstone-4 Mstone-5
All of the bits intended for mstone 4 have already been implemented or are in their own 
bug.  The rest of this is targeted at mstone 5.

Comment 39 by karen@chromium.org, Nov 23 2009

Labels: Mstone-X
Labels: LTTF
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=34654 

------------------------------------------------------------------------
r34654 | hamaji@chromium.org | 2009-12-15 21:31:06 -0800 (Tue, 15 Dec 2009) | 6 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/layout_tests/test_expectations.txt?r1=34654&r2=34653

Mark documentURI as flaky assertion.

BUG= 4360 
TEST=try

Review URL: http://codereview.chromium.org/506027
------------------------------------------------------------------------

Chrome 4.0.266.0
webStorage is to slow, IE or Safari is faster!

https://bugzilla.mozilla.org/attachment.cgi?id=418994
storage_slow.html
772 bytes Download
<script>
 alert(localStorage)
 alert(sessionStorage) // SessionStorage forgot about?
</script>

Comment 45 by goor...@gmail.com, Dec 29 2009

What the hell we doing?
sessionStorage is a part of webStorage!

http://dev.w3.org/html5/webstorage/#the-sessionstorage-attribute

Do not go this way!

Firefox, Explorer, Safari, Opera 10.5 Aphla support full specification.
Not only localStorage. Also sessionStorage!

Comment 46 by jor...@jorlow.com, Dec 30 2009

sessionStorage is slated for the next milestone and will not happen for mstone 4.  
Patches are welcome if you'd like to take a stab at it.  Will look at the documentURI 
assertion flake.
Tests for this functionality can be found at:
http://samples.msdn.microsoft.com/ietestcenter/html5.htm#

HTML 5 > DOM Storage > Session Storage

Comment 48 by karen@chromium.org, Jan 11 2010

Labels: -HTML5
Status: Fixed
I believe that's the last CL.  This should ship to dev channel within a week.  Please open 
new bugs if you discover problems.
Project Member

Comment 51 by bugdroid1@chromium.org, Oct 12 2012

Blockedon: -chromium:16254 -chromium:16876 -chromium:16877 -chromium:18729 -chromium:19972 -chromium:20055 -chromium:26955 chromium:16254 chromium:16876 chromium:16877 chromium:18729 chromium:19972 chromium:20055 chromium:26955
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 52 by bugdroid1@chromium.org, Mar 11 2013

Labels: -Area-WebKit Cr-Content
Project Member

Comment 53 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content Cr-Blink

Sign in to add a comment