2010-11-29 Jeremy Orlow <jorlow@chromium.org>
authorjorlow@chromium.org <jorlow@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Dec 2010 16:14:21 +0000 (16:14 +0000)
committerjorlow@chromium.org <jorlow@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Dec 2010 16:14:21 +0000 (16:14 +0000)
        Reviewed by Darin Fisher.

        [Chromium] Clean up IndexedDB 2 sided roll bits
        https://bugs.webkit.org/show_bug.cgi?id=50160

        * public/WebIDBDatabase.h:
        (WebKit::WebIDBDatabase::version):
        (WebKit::WebIDBDatabase::objectStoreNames):
        (WebKit::WebIDBDatabase::deleteObjectStore):
        * public/WebIDBFactory.h:
        (WebKit::WebIDBFactory::open):
        * public/WebIDBKeyRange.h:
        * public/WebIDBObjectStore.h:
        (WebKit::WebIDBObjectStore::deleteFunction):
        * src/IDBDatabaseProxy.cpp:
        * src/IDBDatabaseProxy.h:
        * src/WebIDBKeyRange.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@73264 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/chromium/ChangeLog
WebKit/chromium/public/WebIDBDatabase.h
WebKit/chromium/public/WebIDBFactory.h
WebKit/chromium/public/WebIDBKeyRange.h
WebKit/chromium/public/WebIDBObjectStore.h
WebKit/chromium/src/IDBDatabaseProxy.cpp
WebKit/chromium/src/IDBDatabaseProxy.h
WebKit/chromium/src/WebIDBKeyRange.cpp

index 614b4c6..c93c262 100644 (file)
@@ -1,3 +1,23 @@
+2010-11-29  Jeremy Orlow  <jorlow@chromium.org>
+
+        Reviewed by Darin Fisher.
+
+        [Chromium] Clean up IndexedDB 2 sided roll bits
+        https://bugs.webkit.org/show_bug.cgi?id=50160
+
+        * public/WebIDBDatabase.h:
+        (WebKit::WebIDBDatabase::version):
+        (WebKit::WebIDBDatabase::objectStoreNames):
+        (WebKit::WebIDBDatabase::deleteObjectStore):
+        * public/WebIDBFactory.h:
+        (WebKit::WebIDBFactory::open):
+        * public/WebIDBKeyRange.h:
+        * public/WebIDBObjectStore.h:
+        (WebKit::WebIDBObjectStore::deleteFunction):
+        * src/IDBDatabaseProxy.cpp:
+        * src/IDBDatabaseProxy.h:
+        * src/WebIDBKeyRange.cpp:
+
 2010-12-02  Chris Rogers  <crogers@google.com>
 
         Reviewed by Darin Fisher.
index 1588e23..5ed8052 100644 (file)
@@ -47,26 +47,22 @@ public:
         WEBKIT_ASSERT_NOT_REACHED();
         return WebString();
     }
-    // FIXME: remove after roll.
-    virtual WebString description() const
+    virtual WebString version() const
     {
+        WEBKIT_ASSERT_NOT_REACHED();
         return WebString();
     }
-    virtual WebString version() const
+    virtual WebDOMStringList objectStoreNames() const
     {
         WEBKIT_ASSERT_NOT_REACHED();
-        return WebString();
+        return WebDOMStringList();
     }
-    virtual WebDOMStringList objectStores() const { return objectStoreNames(); } // FIXME: Remove after roll.
-    virtual WebDOMStringList objectStoreNames() const { return objectStores(); } // FIXME: Assert not reached after roll.
     virtual WebIDBObjectStore* createObjectStore(const WebString& name, const WebString& keyPath, bool autoIncrement, const WebIDBTransaction&, WebExceptionCode&)
     { 
         WEBKIT_ASSERT_NOT_REACHED();
         return 0;
     }
-    virtual void deleteObjectStore(const WebString& name, const WebIDBTransaction& transaction, WebExceptionCode& ec) { removeObjectStore(name, transaction, ec); }
-    // FIXME: remove after roll.
-    virtual void removeObjectStore(const WebString& name, const WebIDBTransaction& transaction, WebExceptionCode& ec) { deleteObjectStore(name, transaction, ec); }
+    virtual void deleteObjectStore(const WebString& name, const WebIDBTransaction& transaction, WebExceptionCode& ec) { WEBKIT_ASSERT_NOT_REACHED(); }
     virtual void setVersion(const WebString& version, WebIDBCallbacks* callbacks, WebExceptionCode&) { WEBKIT_ASSERT_NOT_REACHED(); }
     // Transfers ownership of the WebIDBTransaction to the caller.
     virtual WebIDBTransaction* transaction(const WebDOMStringList& names, unsigned short mode, unsigned long timeout, WebExceptionCode&)
index 5e3337e..10a134a 100755 (executable)
@@ -51,17 +51,10 @@ public:
 
     virtual ~WebIDBFactory() { }
 
-    // FIXME: Remove after roll.
-    virtual void open(const WebString& name, const WebString& description, WebIDBCallbacks* callbacks, const WebSecurityOrigin& origin, WebFrame* webFrame, const WebString& dataDir, unsigned long long maximumSize)
-    {
-        open(name, callbacks, origin, webFrame, dataDir, maximumSize);
-    }
-
     // The WebKit implementation of open ignores the WebFrame* parameter.
     virtual void open(const WebString& name, WebIDBCallbacks* callbacks, const WebSecurityOrigin& origin, WebFrame* webFrame, const WebString& dataDir, unsigned long long maximumSize)
     {
-        // WEBKIT_ASSERT_NOT_REACHED();
-        open(name, "", callbacks, origin, webFrame, dataDir, maximumSize);
+        WEBKIT_ASSERT_NOT_REACHED();
     }
 };
 
index 6a9c6da..922d4ed 100644 (file)
@@ -43,13 +43,6 @@ public:
     WebIDBKeyRange(const WebIDBKeyRange& keyRange) { assign(keyRange); }
     WebIDBKeyRange(const WebIDBKey& lower, const WebIDBKey& upper, bool lowerOpen, bool upperOpen) { assign(lower, upper, lowerOpen, upperOpen); }
 
-    // FIXME: Remove next 3 methods after next roll.
-    WebIDBKeyRange(const WebIDBKey& lower, const WebIDBKey& upper, unsigned short flags) { assign(lower, upper, flags); }
-    WEBKIT_API void assign(const WebIDBKey& lower, const WebIDBKey& upper, unsigned short flags);
-    WEBKIT_API unsigned short flags() const;
-    WEBKIT_API WebIDBKey left() const;
-    WEBKIT_API WebIDBKey right() const;
-
     WEBKIT_API WebIDBKey lower() const;
     WEBKIT_API WebIDBKey upper() const;
     WEBKIT_API bool lowerOpen() const;
index 17562ca..8f2247f 100755 (executable)
@@ -61,9 +61,7 @@ public:
 
     virtual void get(const WebIDBKey&, WebIDBCallbacks*, const WebIDBTransaction&, WebExceptionCode&) { WEBKIT_ASSERT_NOT_REACHED(); }
     virtual void put(const WebSerializedScriptValue&, const WebIDBKey&, bool addOnly, WebIDBCallbacks*, const WebIDBTransaction&, WebExceptionCode&) { WEBKIT_ASSERT_NOT_REACHED(); }
-    // FIXME: Remove after roll.
-    virtual void remove(const WebIDBKey& key, WebIDBCallbacks* callbacks, const WebIDBTransaction& transaction, WebExceptionCode& ec) { deleteFunction(key, callbacks, transaction, ec); }
-    virtual void deleteFunction(const WebIDBKey& key, WebIDBCallbacks* callbacks, const WebIDBTransaction& transaction, WebExceptionCode& ec) { remove(key, callbacks, transaction, ec); }
+    virtual void deleteFunction(const WebIDBKey& key, WebIDBCallbacks* callbacks, const WebIDBTransaction& transaction, WebExceptionCode& ec) { WEBKIT_ASSERT_NOT_REACHED(); }
     virtual WebIDBIndex* createIndex(const WebString& name, const WebString& keyPath, bool unique, const WebIDBTransaction&, WebExceptionCode&)
     {
         WEBKIT_ASSERT_NOT_REACHED();
index 685f53c..f070d64 100644 (file)
@@ -61,11 +61,6 @@ String IDBDatabaseProxy::name() const
     return m_webIDBDatabase->name();
 }
 
-String IDBDatabaseProxy::description() const
-{
-    return m_webIDBDatabase->description();
-}
-
 String IDBDatabaseProxy::version() const
 {
     return m_webIDBDatabase->version();
index 84693f2..c51a604 100644 (file)
@@ -43,7 +43,6 @@ public:
     virtual ~IDBDatabaseProxy();
 
     virtual String name() const;
-    virtual String description() const;
     virtual String version() const;
     virtual PassRefPtr<DOMStringList> objectStoreNames() const;
 
index ec5b7c2..517ff00 100644 (file)
@@ -47,33 +47,11 @@ void WebIDBKeyRange::assign(const WebIDBKey& lower, const WebIDBKey& upper, bool
         m_private = IDBKeyRange::create(lower, upper, lowerOpen, upperOpen);
 }
 
-// FIXME: Remove this after next roll.
-void WebIDBKeyRange::assign(const WebIDBKey& lower, const WebIDBKey& upper, unsigned short flags)
-{
-    bool lowerOpen = !!(flags & 1);
-    bool upperOpen = !!(flags & 2);
-
-    if (lower.type() == WebIDBKey::InvalidType && upper.type() == WebIDBKey::InvalidType)
-        m_private = 0;
-    else
-        m_private = IDBKeyRange::create(lower, upper, lowerOpen, upperOpen);
-}
-
 void WebIDBKeyRange::reset()
 {
     m_private.reset();
 }
 
-WebIDBKey WebIDBKeyRange::left() const
-{
-    return lower();
-}
-
-WebIDBKey WebIDBKeyRange::right() const
-{
-    return upper();
-}
-
 WebIDBKey WebIDBKeyRange::lower() const
 {
     if (!m_private.get())
@@ -98,20 +76,6 @@ bool WebIDBKeyRange::upperOpen() const
     return m_private.get() && m_private->upperOpen();
 }
 
-// FIXME: Remove this after next roll.
-unsigned short WebIDBKeyRange::flags() const
-{
-    if (!m_private.get())
-        return 0;
-
-    unsigned short flags = 0;
-    if (m_private->lowerOpen())
-        flags |= 1;
-    if (m_private->upperOpen())
-        flags |= 2;
-    return flags;
-}
-
 WebIDBKeyRange::WebIDBKeyRange(const PassRefPtr<IDBKeyRange>& value)
     : m_private(value)
 {