Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(275)

Unified Diff: Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp

Issue 13843016: Prepare to remove WebSecurityOrigin from IDB backend (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: remove unused static Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/WebKit/chromium/src/IDBFactoryBackendProxy.h ('k') | Source/WebKit/chromium/src/WebIDBFactoryImpl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp
diff --git a/Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp b/Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp
index 7c6f30c4864e1f85ee72111637f7b05e5e3f278a..9510069bd75feaf6b4b14a2379ff76cc2cb00705 100644
--- a/Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp
+++ b/Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp
@@ -187,48 +187,35 @@ bool IDBFactoryBackendProxy::allowIndexedDB(ScriptExecutionContext* context, con
return allowed;
}
-static WebFrameImpl* getWebFrame(ScriptExecutionContext* context)
-{
- ASSERT_WITH_SECURITY_IMPLICATION(context->isDocument() || context->isWorkerContext());
- if (context->isDocument()) {
- Document* document = static_cast<Document*>(context);
- return WebFrameImpl::fromFrame(document->frame());
- }
- return 0;
-}
-
-void IDBFactoryBackendProxy::getDatabaseNames(PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<SecurityOrigin> securityOrigin, ScriptExecutionContext* context, const String& dataDir)
+void IDBFactoryBackendProxy::getDatabaseNames(PassRefPtr<IDBCallbacks> prpCallbacks, const String& databaseIdentifier, ScriptExecutionContext* context, const String& dataDir)
{
RefPtr<IDBCallbacks> callbacks(prpCallbacks);
- WebSecurityOrigin origin(securityOrigin);
+ WebSecurityOrigin origin(context->securityOrigin());
if (!allowIndexedDB(context, "Database Listing", origin, callbacks))
return;
- WebFrameImpl* webFrame = getWebFrame(context);
- m_webIDBFactory->getDatabaseNames(new WebIDBCallbacksImpl(callbacks), origin, webFrame, dataDir);
+ m_webIDBFactory->getDatabaseNames(new WebIDBCallbacksImpl(callbacks), databaseIdentifier, dataDir);
}
-void IDBFactoryBackendProxy::open(const String& name, int64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<IDBDatabaseCallbacks> prpDatabaseCallbacks, PassRefPtr<SecurityOrigin> securityOrigin, ScriptExecutionContext* context, const String& dataDir)
+void IDBFactoryBackendProxy::open(const String& name, int64_t version, int64_t transactionId, PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<IDBDatabaseCallbacks> prpDatabaseCallbacks, const String& databaseIdentifier, ScriptExecutionContext* context, const String& dataDir)
{
RefPtr<IDBCallbacks> callbacks(prpCallbacks);
RefPtr<IDBDatabaseCallbacks> databaseCallbacks(prpDatabaseCallbacks);
- WebSecurityOrigin origin(securityOrigin);
+ WebSecurityOrigin origin(context->securityOrigin());
if (!allowIndexedDB(context, name, origin, callbacks))
return;
- WebFrameImpl* webFrame = getWebFrame(context);
- m_webIDBFactory->open(name, version, transactionId, new WebIDBCallbacksImpl(callbacks), new WebIDBDatabaseCallbacksImpl(databaseCallbacks), origin, webFrame, dataDir);
+ m_webIDBFactory->open(name, version, transactionId, new WebIDBCallbacksImpl(callbacks), new WebIDBDatabaseCallbacksImpl(databaseCallbacks), databaseIdentifier, dataDir);
}
-void IDBFactoryBackendProxy::deleteDatabase(const String& name, PassRefPtr<IDBCallbacks> prpCallbacks, PassRefPtr<SecurityOrigin> securityOrigin, ScriptExecutionContext* context, const String& dataDir)
+void IDBFactoryBackendProxy::deleteDatabase(const String& name, PassRefPtr<IDBCallbacks> prpCallbacks, const String& databaseIdentifier, ScriptExecutionContext* context, const String& dataDir)
{
RefPtr<IDBCallbacks> callbacks(prpCallbacks);
- WebSecurityOrigin origin(securityOrigin);
+ WebSecurityOrigin origin(context->securityOrigin());
if (!allowIndexedDB(context, name, origin, callbacks))
return;
- WebFrameImpl* webFrame = getWebFrame(context);
- m_webIDBFactory->deleteDatabase(name, new WebIDBCallbacksImpl(callbacks), origin, webFrame, dataDir);
+ m_webIDBFactory->deleteDatabase(name, new WebIDBCallbacksImpl(callbacks), databaseIdentifier, dataDir);
}
} // namespace WebKit
« no previous file with comments | « Source/WebKit/chromium/src/IDBFactoryBackendProxy.h ('k') | Source/WebKit/chromium/src/WebIDBFactoryImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698