Changeset 85954 in webkit

Timestamp:

May 6, 2011 10:04:39 AM (13 years ago)

Author:

inferno@chromium.org

Message:

2011-05-06 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r85919.
​http://trac.webkit.org/changeset/85919
​https://bugs.webkit.org/show_bug.cgi?id=60378

broke chromium database tests (Requested by inferno-sec on
#webkit).

• public/WebPermissionClient.h:
• src/IDBFactoryBackendProxy.cpp: (WebKit::IDBFactoryBackendProxy::open):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• public/WebPermissionClient.h (modified) (2 diffs)
• src/IDBFactoryBackendProxy.cpp (modified) (2 diffs)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2011-05-06  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r85919.
+        http://trac.webkit.org/changeset/85919
+        https://bugs.webkit.org/show_bug.cgi?id=60378
+
+        broke chromium database tests (Requested by inferno-sec on
+        #webkit).
+
+        * public/WebPermissionClient.h:
+        * src/IDBFactoryBackendProxy.cpp:
+        (WebKit::IDBFactoryBackendProxy::open):
+
 2011-05-04  Philippe Normand  <pnormand@igalia.com>
 

trunk/Source/WebKit/chromium/public/WebPermissionClient.h

@@ -35 +35 @@
 
 class WebFrame;
-class WebSecurityOrigin;
 class WebString;
 
@@ -45 +44 @@
     // Controls whether images are allowed for this frame.
     virtual bool allowImages(WebFrame*, bool enabledPerSettings) { return enabledPerSettings; }
-
-    // Controls whether access to Indexed DB are allowed for this frame.
-    virtual bool allowIndexedDB(WebFrame*, const WebString& name, const WebSecurityOrigin&) { return true; }
 
     // Controls whether plugins are allowed for this frame.

trunk/Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp

@@ -43 +43 @@
 #include "WebKit.h"
 #include "WebKitClient.h"
-#include "WebPermissionClient.h"
 #include "WebVector.h"
-#include "WebViewImpl.h"
 
 using namespace WebCore;
@@ -67 +65 @@
 void IDBFactoryBackendProxy::open(const String& name, PassRefPtr<IDBCallbacks> callbacks, PassRefPtr<SecurityOrigin> origin, Frame* frame, const String& dataDir, int64_t maximumSize, BackingStoreType backingStoreType)
 {
-    WebFrameImpl* webFrame = WebFrameImpl::fromFrame(frame);
-    WebViewImpl* webView = webFrame->viewImpl();
-    if (webView->permissionClient() && !webView->permissionClient()->allowIndexedDB(webFrame, name, origin)) {
-        callbacks->onError(WebIDBDatabaseError(0, "The user denied permission to access the database."));
-        return;
-    }
-
+    WebFrame* webFrame = WebFrameImpl::fromFrame(frame);
     m_webIDBFactory->open(name, new WebIDBCallbacksImpl(callbacks), origin, webFrame, dataDir, maximumSize, static_cast<WebIDBFactory::BackingStoreType>(backingStoreType));
 }