Remove the HistoryState property of WebRequest....

chrome/renderer/render_view.cc

 // Copyright (c) 2009 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.
 
 #include "chrome/renderer/render_view.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 776 matching lines @@
 
   WebFrame* main_frame = webview()->GetMainFrame();
   if (is_reload && !main_frame->HasCurrentHistoryState()) {
     // We cannot reload if we do not have any history state.  This happens, for
     // example, when recovering from a crash.  Our workaround here is a bit of
     // a hack since it means that reload after a crashed tab does not cause an
     // end-to-end cache validation.
     is_reload = false;
   }
 
-  WebRequestCachePolicy cache_policy;
-  if (is_reload) {
-    cache_policy = WebRequestReloadIgnoringCacheData;
-  } else if (params.page_id != -1 || main_frame->GetInViewSourceMode()) {
-    cache_policy = WebRequestReturnCacheDataElseLoad;
-  } else {
-    cache_policy = WebRequestUseProtocolCachePolicy;
-  }
-
-  scoped_ptr<WebRequest> request(WebRequest::Create(params.url));
-  request->SetCachePolicy(cache_policy);
-
   // A navigation resulting from loading a javascript URL should not be treated
   // as a browser initiated event.  Instead, we want it to look as if the page
   // initiated any load resulting from JS execution.
   if (!params.url.SchemeIs(chrome::kJavaScriptScheme)) {
     pending_navigation_state_.reset(new NavigationState(
         params.page_id, params.transition, params.request_time));
   }
 
-  // If we are reloading, then WebKit will use the state of the current page.
-  // Otherwise, we give it the state to navigate to.
-  if (!is_reload)
-    request->SetHistoryState(params.state);
+  // If we are reloading, then WebKit will use the history state of the current
+  // page, so we should just ignore any given history state.  Otherwise, if we
+  // have history state, then we need to navigate to it, which corresponds to a
+  // back/forward navigation event.
+  if (!is_reload && !params.state.empty()) {
+    // We must know the page ID of the page we are navigating back to.
+    DCHECK(params.page_id != -1);
+    main_frame->LoadHistoryState(params.state);
+  } else {
+    // Navigate to the given URL.
+    scoped_ptr<WebRequest> request(WebRequest::Create(params.url));
 
-  if (params.referrer.is_valid()) {
-    request->SetHttpHeaderValue("Referer",
-                                params.referrer.spec());
+    // TODO(darin): WebFrame should just have a Reload method.
+
+    WebRequestCachePolicy cache_policy;
+    if (is_reload) {
+      cache_policy = WebRequestReloadIgnoringCacheData;
+    } else {
+      // A session history navigation should have been accompanied by state.
+      DCHECK(params.page_id == -1);
+      if (main_frame->GetInViewSourceMode()) {
+        cache_policy = WebRequestReturnCacheDataElseLoad;
+      } else {
+        cache_policy = WebRequestUseProtocolCachePolicy;
+      }
+    }
+    request->SetCachePolicy(cache_policy);
+
+    if (params.referrer.is_valid())
+      request->SetHttpHeaderValue("Referer", params.referrer.spec());
+
+    main_frame->LoadRequest(request.get());
   }
 
-  main_frame->LoadRequest(request.get());
-
   // In case LoadRequest failed before DidCreateDataSource was called.
   pending_navigation_state_.reset();
 }
 
 // Stop loading the current page
 void RenderView::OnStop() {
   if (webview())
     webview()->StopLoading();
 }
 
@@ skipping 2230 matching lines @@
   // If id is valid, alert the browser side that an accessibility focus change
   // occurred.
   if (acc_obj_id >= 0)
     Send(new ViewHostMsg_AccessibilityFocusChange(routing_id_, acc_obj_id));
 
 #else  // defined(OS_WIN)
   // TODO(port): accessibility not yet implemented
   NOTIMPLEMENTED();
 #endif
 }