IndexedDB: Pass origin to platform/IPC, rather than DatabaseIdentifier

content/common/indexed_db/indexed_db_messages.h

Index: content/common/indexed_db/indexed_db_messages.h
diff --git a/content/common/indexed_db/indexed_db_messages.h b/content/common/indexed_db/indexed_db_messages.h
index 4c211a7d18c5e9140424894486135adbf5437d2d..f0ec336ad9cd756767f6831d842899da7413623f 100644
--- a/content/common/indexed_db/indexed_db_messages.h
+++ b/content/common/indexed_db/indexed_db_messages.h
@@ -47,8 +47,8 @@ IPC_STRUCT_BEGIN(IndexedDBHostMsg_FactoryGetDatabaseNames_Params)
   // The response should have these ids.
   IPC_STRUCT_MEMBER(int32_t, ipc_thread_id)
   IPC_STRUCT_MEMBER(int32_t, ipc_callbacks_id)
-  // The string id of the origin doing the initiating.
-  IPC_STRUCT_MEMBER(std::string, database_identifier)
+  // The origin doing the initiating.
+  IPC_STRUCT_MEMBER(GURL, origin)

[Mike West, 2016/03/03 05:49:08]

I think I'm missing something here... If you're changing these messages anyway,
why not change them to `url::Origin`?

[jsbell, 2016/03/08 17:17:11]

On the content/browser side ("back end") this is used/passed around/serialized
as a GURL. If we don't mind converting url::Origin to GURL that's fine.
Otherwise I'd prefer to do this incrementally (switch to GURL first to be
consistent end-to-end, then convert everything to url::Origin)

 IPC_STRUCT_END()
 
 // Used to open an indexed database.
@@ -59,8 +59,8 @@ IPC_STRUCT_BEGIN(IndexedDBHostMsg_FactoryOpen_Params)
   IPC_STRUCT_MEMBER(int32_t, ipc_callbacks_id)
   // Identifier for database callbacks
   IPC_STRUCT_MEMBER(int32_t, ipc_database_callbacks_id)
-  // The string id of the origin doing the initiating.
-  IPC_STRUCT_MEMBER(std::string, database_identifier)
+  // The origin doing the initiating.
+  IPC_STRUCT_MEMBER(GURL, origin)
   // The name of the database.
   IPC_STRUCT_MEMBER(base::string16, name)
   // The transaction id used if a database upgrade is needed.
@@ -74,8 +74,8 @@ IPC_STRUCT_BEGIN(IndexedDBHostMsg_FactoryDeleteDatabase_Params)
   // The response should have these ids.
   IPC_STRUCT_MEMBER(int32_t, ipc_thread_id)
   IPC_STRUCT_MEMBER(int32_t, ipc_callbacks_id)
-  // The string id of the origin doing the initiating.
-  IPC_STRUCT_MEMBER(std::string, database_identifier)
+  // The origin doing the initiating.
+  IPC_STRUCT_MEMBER(GURL, origin)
   // The name of the database.
   IPC_STRUCT_MEMBER(base::string16, name)
 IPC_STRUCT_END()