Consolidate key, primary key, value cursor messages.

content/renderer/indexed_db_dispatcher.cc

Index: content/renderer/indexed_db_dispatcher.cc
diff --git a/content/renderer/indexed_db_dispatcher.cc b/content/renderer/indexed_db_dispatcher.cc
index eb00ac395dd1db889e8a8510a0ae96eba2ae2319..2a30e73c1042747f1c399bd739494cc891e2e3a7 100644
--- a/content/renderer/indexed_db_dispatcher.cc
+++ b/content/renderer/indexed_db_dispatcher.cc
@@ -392,10 +392,12 @@ void IndexedDBDispatcher::OnSuccessSerializedScriptValue(
 }
 
 void IndexedDBDispatcher::OnSuccessOpenCursor(int32 repsonse_id,
-                                              int32 object_id) {
+    int32 object_id, const IndexedDBKey& key, const IndexedDBKey& primaryKey,
+    const SerializedScriptValue& value) {
   WebIDBCallbacks* callbacks =
       pending_callbacks_.Lookup(repsonse_id);
-  callbacks->onSuccess(new RendererWebIDBCursorImpl(object_id));
+  callbacks->onSuccess(new RendererWebIDBCursorImpl(object_id), key,

[michaeln, 2011/09/04 23:08:16]

Might be nice if the Cursor class contained a single method to update these
three values all in one go, is there any reason to set one w/o setting the
others...

   cursor = new Cursor(id);
   cursor->setCurrentKeyAndValue(key, primary_key, value);
   onSuccess(cursor);

That same method could be useful in the continue/advance case as well.

+                       primaryKey, value);
   pending_callbacks_.Remove(repsonse_id);
 }