Re-do r15244: Fix crash in ~TestWebViewDelegate caused by

shell_ being null.  Moved the RevokeDragDrop call to the
TestShell destructor instead.

Eliminate webkit/glue/webhistoryitem* in favor of adding a
NavigateBackForwardSoon method WebViewDelegate.  This moves
all of the hacky details of how we intercept "history.{back,
forward,go}" into the webkit layer.  My eventual plan is to
teach WebCore how to make this not hacky.

BUG=11423
R=mpcomplete

Review URL: http://codereview.chromium.org/108004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15278 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 5 deletions

diff --git a/webkit/glue/webview_delegate.h b/webkit/glue/webview_delegate.h
index 29c31be..d6a36a6 100644
--- a/webkit/glue/webview_delegate.h
+++ b/webkit/glue/webview_delegate.h
@@ -53,7 +53,6 @@ class SkBitmap;
 class WebDevToolsAgentDelegate;
 class WebError;
 class WebFrame;
-class WebHistoryItem;
 class WebMediaPlayerDelegate;
 class WebPluginDelegate;
 class WebRequest;
@@ -695,10 +694,9 @@ class WebViewDelegate : virtual public WebWidgetDelegate {
 
   // History Related ---------------------------------------------------------
 
-  // Returns the session history entry at a distance |offset| relative to the
-  // current entry.  Returns NULL on failure.
-  virtual WebHistoryItem* GetHistoryEntryAtOffset(int offset) {
-    return NULL;
+  // Tells the embedder to navigate back or forward in session history by the
+  // given offset (relative to the current position in session history).
+  virtual void NavigateBackForwardSoon(int offset) {
   }
 
   // Returns how many entries are in the back and forward lists, respectively.