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

content/common/indexed_db/indexed_db_messages.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Message definition file, included multiple times, hence no include guard.
 
 #include <stdint.h>
 
 #include <string>
 #include <utility>
@@ skipping 29 matching lines @@
 IPC_ENUM_TRAITS_MAX_VALUE(blink::WebIDBTransactionMode,
                           blink::WebIDBTransactionModeLast)
 
 IPC_ENUM_TRAITS_MAX_VALUE(blink::WebIDBDataLoss, blink::WebIDBDataLossTotal)
 
 // Used to enumerate indexed databases.
 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.
 IPC_STRUCT_BEGIN(IndexedDBHostMsg_FactoryOpen_Params)
   // The response should have these ids.
   IPC_STRUCT_MEMBER(int32_t, ipc_thread_id)
   // Identifier of the request
   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.
   IPC_STRUCT_MEMBER(int64_t, transaction_id)
   // The requested version of the database.
   IPC_STRUCT_MEMBER(int64_t, version)
 IPC_STRUCT_END()
 
 // Used to delete an indexed database.
 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()
 
 IPC_STRUCT_BEGIN(IndexedDBHostMsg_DatabaseCreateTransaction_Params)
   IPC_STRUCT_MEMBER(int32_t, ipc_thread_id)
   // The database the object store belongs to.
   IPC_STRUCT_MEMBER(int32_t, ipc_database_id)
   // The transaction id as minted by the frontend.
   IPC_STRUCT_MEMBER(int64_t, transaction_id)
@@ skipping 481 matching lines @@
                      int64_t) /* transaction_id */
 
 // WebIDBDatabase::commit() message.
 IPC_MESSAGE_CONTROL2(IndexedDBHostMsg_DatabaseCommit,
                      int32_t, /* ipc_database_id */
                      int64_t) /* transaction_id */
 
 // WebIDBDatabase::~WebIDBCursor() message.
 IPC_MESSAGE_CONTROL1(IndexedDBHostMsg_CursorDestroyed,
                      int32_t /* ipc_cursor_id */)