Comment on attachment 26312[details]
This patch is addressing comment #2.
This patch is looking great! Sorry it took so long to get a review. Marking as r- for now to get some things fixed based on the comments below.
> + page->inspectorController()->didUseDomStorage(storageArea.get(), false, m_frame);
As Sam mentioned, we prefer the spelling DOM to Dom when in the middle of name, or dom if it is at the beginning. There are a lot of occurrences of this that should be changed to DOM.
> + this.localStorageListTreeElement = new WebInspector.SidebarSectionTreeElement(WebInspector.UIString("LOCAL-STORAGE"), {}, true);
> + this.sessionStorageListTreeElement = new WebInspector.SidebarSectionTreeElement(WebInspector.UIString("SESSION-STORAGE"), {}, true);
I think these would read better without the hyphen, so "LOCAL STORAGE" and "SESSION STORAGE".
> + content: url(Images/domStorage.png);
What does this icon look like? Who created it? Can it be freely added to WebKit under the BSD license?
> -.database-view {
> +.database-view, .domstorage-view {
I think it might be better to rename .database-view to something generic like .storage-view, so all the selectors don't need to be complex multi-selectors.
You also need to add the new files to the WebCore.vcproj.

Created attachment 27321[details]
Addressing comment #7
This patch is addressing comment #7.
WRT the image, I drew it based on databasetable.png. Nobody should claim ownership for it and it should be safe to add the image to the project.