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

Unified Diff: content/browser/indexed_db/indexed_db_dispatcher_host.cc

Issue 1755343002: IndexedDB: Pass origin to platform/IPC, rather than DatabaseIdentifier (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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
Index: content/browser/indexed_db/indexed_db_dispatcher_host.cc
diff --git a/content/browser/indexed_db/indexed_db_dispatcher_host.cc b/content/browser/indexed_db/indexed_db_dispatcher_host.cc
index 6e83583e7ea1987472740722e9c8c1629603fd46..969ce78e978a3d94d2bfe9a4ab3d10945e8ea768 100644
--- a/content/browser/indexed_db/indexed_db_dispatcher_host.cc
+++ b/content/browser/indexed_db/indexed_db_dispatcher_host.cc
@@ -312,13 +312,10 @@ void IndexedDBDispatcherHost::OnIDBFactoryGetDatabaseNames(
DCHECK(indexed_db_context_->TaskRunner()->RunsTasksOnCurrentThread());
base::FilePath indexed_db_path = indexed_db_context_->data_path();
- GURL origin_url =
- storage::GetOriginFromIdentifier(params.database_identifier);
-
context()->GetIDBFactory()->GetDatabaseNames(
new IndexedDBCallbacks(this, params.ipc_thread_id,
params.ipc_callbacks_id),
- origin_url, indexed_db_path, request_context_);
+ params.origin, indexed_db_path, request_context_);
}
void IndexedDBDispatcherHost::OnIDBFactoryOpen(
@@ -327,20 +324,13 @@ void IndexedDBDispatcherHost::OnIDBFactoryOpen(
base::TimeTicks begin_time = base::TimeTicks::Now();
base::FilePath indexed_db_path = indexed_db_context_->data_path();
- GURL origin_url =
- storage::GetOriginFromIdentifier(params.database_identifier);
-
int64_t host_transaction_id = HostTransactionId(params.transaction_id);
// TODO(dgrogan): Don't let a non-existing database be opened (and therefore
// created) if this origin is already over quota.
- scoped_refptr<IndexedDBCallbacks> callbacks =
- new IndexedDBCallbacks(this,
- params.ipc_thread_id,
- params.ipc_callbacks_id,
- params.ipc_database_callbacks_id,
- host_transaction_id,
- origin_url);
+ scoped_refptr<IndexedDBCallbacks> callbacks = new IndexedDBCallbacks(
+ this, params.ipc_thread_id, params.ipc_callbacks_id,
+ params.ipc_database_callbacks_id, host_transaction_id, params.origin);
callbacks->SetConnectionOpenStartTime(begin_time);
scoped_refptr<IndexedDBDatabaseCallbacks> database_callbacks =
new IndexedDBDatabaseCallbacks(
@@ -352,21 +342,19 @@ void IndexedDBDispatcherHost::OnIDBFactoryOpen(
params.version);
DCHECK(request_context_);
context()->GetIDBFactory()->Open(params.name, connection, request_context_,
- origin_url, indexed_db_path);
+ params.origin, indexed_db_path);
}
void IndexedDBDispatcherHost::OnIDBFactoryDeleteDatabase(
const IndexedDBHostMsg_FactoryDeleteDatabase_Params& params) {
DCHECK(indexed_db_context_->TaskRunner()->RunsTasksOnCurrentThread());
- GURL origin_url =
- storage::GetOriginFromIdentifier(params.database_identifier);
base::FilePath indexed_db_path = indexed_db_context_->data_path();
DCHECK(request_context_);
context()->GetIDBFactory()->DeleteDatabase(
params.name, request_context_,
new IndexedDBCallbacks(this, params.ipc_thread_id,
params.ipc_callbacks_id),
- origin_url, indexed_db_path);
+ params.origin, indexed_db_path);
}
// OnPutHelper exists only to allow us to hop threads while holding a reference
« no previous file with comments | « no previous file | content/child/indexed_db/indexed_db_dispatcher.h » ('j') | content/common/indexed_db/indexed_db_messages.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698