Consolidate key, primary key, value cursor messages.

content/renderer/renderer_webidbcursor_impl.h

Index: content/renderer/renderer_webidbcursor_impl.h
diff --git a/content/renderer/renderer_webidbcursor_impl.h b/content/renderer/renderer_webidbcursor_impl.h
index d7f4541aea650a80bfac3b688372687643a3847b..8ad703234160da56b0bda101a47022d0c87fc1df 100644
--- a/content/renderer/renderer_webidbcursor_impl.h
+++ b/content/renderer/renderer_webidbcursor_impl.h
@@ -19,9 +19,6 @@ class RendererWebIDBCursorImpl : public WebKit::WebIDBCursor {
   virtual ~RendererWebIDBCursorImpl();
 
   virtual unsigned short direction() const;
-  virtual WebKit::WebIDBKey key() const;
-  virtual WebKit::WebIDBKey primaryKey() const;
-  virtual WebKit::WebSerializedScriptValue value() const;

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

This is an example of that 'difficult to review' thing mentioned in email.

Why are these methods being removed? The WebCore::IDBCursor class still needs to
retrieve these values in someway. Aren't these methods that way?

[dgrogan, 2011/09/07 01:15:19]

The webkit patch stores key, primary key, and value in the WebCore::IDBCursor
frontend object.  So calls to key(), primaryKey(), and value() don't have to go
over the wire.

   virtual void update(const WebKit::WebSerializedScriptValue& value,
                       WebKit::WebIDBCallbacks* callback,
                       WebKit::WebExceptionCode& ec);