WebBlobInfo plumbing for WebIDBDatabaseImpl::put.

content/child/indexed_db/webidbdatabase_impl.cc

Index: content/child/indexed_db/webidbdatabase_impl.cc
diff --git a/content/child/indexed_db/webidbdatabase_impl.cc b/content/child/indexed_db/webidbdatabase_impl.cc
index 459a6fd08be7028f2125ca5e46bf6f04b860f1ec..3687ad9dae511266eed188e127cd82d378dcba0f 100644
--- a/content/child/indexed_db/webidbdatabase_impl.cc
+++ b/content/child/indexed_db/webidbdatabase_impl.cc
@@ -11,11 +11,13 @@
 #include "content/child/thread_safe_sender.h"
 #include "content/child/worker_task_runner.h"
 #include "content/common/indexed_db/indexed_db_messages.h"
+#include "third_party/WebKit/public/platform/WebBlobInfo.h"
 #include "third_party/WebKit/public/platform/WebIDBKeyPath.h"
 #include "third_party/WebKit/public/platform/WebIDBMetadata.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/platform/WebVector.h"
 
+using blink::WebBlobInfo;
 using blink::WebIDBCallbacks;
 using blink::WebIDBCursor;
 using blink::WebIDBDatabase;
@@ -106,6 +108,8 @@ void WebIDBDatabaseImpl::get(long long transaction_id,
                                     callbacks);
 }
 
+// TODO(ericu): Remove this once it's obsolete.  It's only here for the
+// three-sided-patch dance.
 void WebIDBDatabaseImpl::put(long long transaction_id,
                              long long object_store_id,
                              const blink::WebData& value,
@@ -116,10 +120,35 @@ void WebIDBDatabaseImpl::put(long long transaction_id,
                              const WebVector<WebIndexKeys>& web_index_keys) {
   IndexedDBDispatcher* dispatcher =
       IndexedDBDispatcher::ThreadSpecificInstance(thread_safe_sender_.get());
+  const blink::WebVector<WebBlobInfo> webBlobInfo;
   dispatcher->RequestIDBDatabasePut(ipc_database_id_,
                                     transaction_id,
                                     object_store_id,
                                     value,
+                                    webBlobInfo,
+                                    IndexedDBKeyBuilder::Build(key),
+                                    put_mode,
+                                    callbacks,
+                                    web_index_ids,
+                                    web_index_keys);
+}
+
+void WebIDBDatabaseImpl::put(long long transaction_id,
+                             long long object_store_id,
+                             const blink::WebData& value,
+                             const blink::WebVector<WebBlobInfo>& webBlobInfo,

[jsbell, 2014/04/09 20:11:08]

Ugh. Case should match the header, which matches blink names, which would be
camelCase. But the other params are snake_case.

Should probably be snake_case to match.

[ericu, 2014/04/09 21:14:47]

I think the existing code's a reasonable compromise; the header's blink style,
the implementation is chromium style.  I'll match that.

+                             const WebIDBKey& key,
+                             PutMode put_mode,
+                             WebIDBCallbacks* callbacks,
+                             const WebVector<long long>& web_index_ids,
+                             const WebVector<WebIndexKeys>& web_index_keys) {
+  IndexedDBDispatcher* dispatcher =
+      IndexedDBDispatcher::ThreadSpecificInstance(thread_safe_sender_.get());
+  dispatcher->RequestIDBDatabasePut(ipc_database_id_,
+                                    transaction_id,
+                                    object_store_id,
+                                    value,
+                                    webBlobInfo,
                                     IndexedDBKeyBuilder::Build(key),
                                     put_mode,
                                     callbacks,