Create and store persistent unique ids for sessionStorage.

webkit/dom_storage/dom_storage_session.h

Index: webkit/dom_storage/dom_storage_session.h
diff --git a/webkit/dom_storage/dom_storage_session.h b/webkit/dom_storage/dom_storage_session.h
index c1add408e883ef23cfa07e9b11d11443f913a52b..91e18695262e3f04b2b8b234f22eacb0d6ae0dee 100644
--- a/webkit/dom_storage/dom_storage_session.h
+++ b/webkit/dom_storage/dom_storage_session.h
@@ -6,6 +6,8 @@
 #define WEBKIT_DOM_STORAGE_DOM_STORAGE_SESSION_H_
 #pragma once
 
+#include <string>
+
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 
@@ -21,7 +23,12 @@ class DomStorageSession
     : public base::RefCountedThreadSafe<DomStorageSession> {
  public:
   explicit DomStorageSession(DomStorageContext* context);

[michaeln, 2012/06/14 22:53:48]

The constructors for this are not so obvious (here and in the content library).
Might help to add a comments on each ctor about the behavior|intended_use.

[marja, 2012/06/15 09:17:43]

Done. (In the header.)

+  DomStorageSession(DomStorageContext* context,
+                    const std::string& persistent_namespace_id);
   int64 namespace_id() const { return namespace_id_; }
+  std::string persistent_namespace_id() const {

[michaeln, 2012/06/14 22:53:48]

ditto const ref question

[marja, 2012/06/15 09:17:43]

Done.

+    return persistent_namespace_id_;
+  }
   DomStorageSession* Clone();
 
   static DomStorageSession* CloneFrom(DomStorageContext* context,
@@ -30,11 +37,14 @@ class DomStorageSession
  private:
   friend class base::RefCountedThreadSafe<DomStorageSession>;
 
-  DomStorageSession(DomStorageContext* context, int64 namespace_id);
+  DomStorageSession(DomStorageContext* context,
+                    int64 namespace_id,
+                    const std::string& persistent_namespace_id);
   ~DomStorageSession();
 
   scoped_refptr<DomStorageContext> context_;
   int64 namespace_id_;
+  std::string persistent_namespace_id_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(DomStorageSession);
 };