Framing for a DOMStorage backend that does not depend on in-process-webkit.

webkit/dom_storage/dom_storage_host.cc

Index: webkit/dom_storage/dom_storage_host.cc
===================================================================
--- webkit/dom_storage/dom_storage_host.cc	(revision 0)
+++ webkit/dom_storage/dom_storage_host.cc	(revision 0)
@@ -0,0 +1,104 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "webkit/dom_storage/dom_storage_host.h"
+
+#include "googleurl/src/gurl.h"
+#include "webkit/dom_storage/dom_storage_area.h"
+#include "webkit/dom_storage/dom_storage_context.h"
+#include "webkit/dom_storage/dom_storage_namespace.h"
+
+namespace dom_storage {
+
+DomStorageHost::DomStorageHost(DomStorageContext* context)
+    : last_connection_id_(0),
+      context_(context) {
+}
+
+DomStorageHost::~DomStorageHost() {
+}
+
+// TODO(michaeln): have the caller pass in the 'connection_id' value
+// instead of generating them here, avoids a sync ipc on open.
+int DomStorageHost::OpenStorageArea(int namespace_id, const GURL& origin) {
+  DomStorageNamespace* name_space = context_->GetStorageNamespace(namespace_id);
+  if (!name_space)
+    return -1;
+  DomStorageArea* area = name_space->GetStorageArea(origin);
+  if (!area)
+    return -1;
+  int id = ++last_connection_id_;
+  connections_[id] = area;
+  return id;
+}
+
+void DomStorageHost::CloseStorageArea(int connection_id) {
+  connections_.erase(connection_id);
+}
+
+int DomStorageHost::GetAreaLength(int connection_id) {
+  DomStorageArea* area = GetOpenArea(connection_id);
+  if (!area)
+    return 0;
+  return area->Length();
+}
+
+NullableString16 DomStorageHost::GetAreaKey(int connection_id, int index) {
+  DomStorageArea* area = GetOpenArea(connection_id);
+  if (!area)
+    return NullableString16();

[benm (inactive), 2012/01/31 18:03:38]

NullableString16 is not null by default... think you want to pass true here?

[michaeln, 2012/01/31 21:17:45]

Done here and below

+  return area->Key(index);
+}
+
+NullableString16 DomStorageHost::GetAreaItem(int connection_id,
+                                             const string16& key) {
+  DomStorageArea* area = GetOpenArea(connection_id);
+  if (!area)
+    return NullableString16();
+  return area->GetItem(key);
+}
+
+bool DomStorageHost::SetAreaItem(
+    int connection_id, const string16& key,
+    const string16& value, const GURL& page_url,
+    NullableString16* old_value) {
+  DomStorageArea* area = GetOpenArea(connection_id);
+  if (!area)
+    return false;
+  if (!area->SetItem(key, value, old_value))
+    return false;
+  context_->NotifyItemSet(area, key, value, *old_value, page_url);
+  return true;
+}
+
+bool DomStorageHost::RemoveAreaItem(int connection_id, const string16& key,
+                                const GURL& page_url,
+                                string16* old_value) {
+  DomStorageArea* area = GetOpenArea(connection_id);
+  if (!area)
+    return false;
+  if (!area->RemoveItem(key, old_value))
+    return false;
+  context_->NotifyItemRemoved(area, key, *old_value, page_url);
+  return true;
+}
+
+bool DomStorageHost::ClearArea(int connection_id, const GURL& page_url) {
+  DomStorageArea* area = GetOpenArea(connection_id);
+  if (!area)
+    return false;
+  if (!area->Clear())
+    return false;
+  context_->NotifyAreaCleared(area, page_url);
+  return true;
+}
+
+DomStorageArea* DomStorageHost::GetOpenArea(int connection_id) {
+  AreaMap::iterator found = connections_.find(connection_id);
+  if (found == connections_.end())
+    return NULL;
+  return found->second;
+}
+
+}  // namespace dom_storage
Property changes on: webkit\dom_storage\dom_storage_host.cc
___________________________________________________________________
Added: svn:eol-style
   + LF