Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8588)

Unified Diff: chrome/browser/sessions/base_session_service.cc

Issue 9999010: Store original request URL in NavigationEntry (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebasing Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/sessions/session_service_test_helper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sessions/base_session_service.cc
diff --git a/chrome/browser/sessions/base_session_service.cc b/chrome/browser/sessions/base_session_service.cc
index 418a021cac506ead155746c4bdc179fd3846869b..54ec8ba501498994d9985e1c0d7769891708564e 100644
--- a/chrome/browser/sessions/base_session_service.cc
+++ b/chrome/browser/sessions/base_session_service.cc
@@ -187,7 +187,10 @@ SessionCommand* BaseSessionService::CreateUpdateTabNavigationCommand(
entry.GetReferrer().url.spec() : std::string());
pickle.WriteInt(entry.GetReferrer().policy);
- // Adding more data? Be sure and update TabRestoreService too.
+ WriteStringToPickle(pickle, &bytes_written, max_state_size,
+ entry.GetOriginalRequestURL().is_valid() ?
+ entry.GetOriginalRequestURL().spec() : std::string());
+
return new SessionCommand(command_id, pickle);
}
@@ -266,6 +269,12 @@ bool BaseSessionService::RestoreUpdateTabNavigationCommand(
navigation->referrer_ = content::Referrer(
referrer_spec.empty() ? GURL() : GURL(referrer_spec),
policy);
+
+ // If the original URL can't be found, leave it empty.
+ std::string url_spec;
+ if (!pickle->ReadString(&iterator, &url_spec))
+ url_spec = std::string();
+ navigation->set_original_request_url(GURL(url_spec));
}
navigation->virtual_url_ = GURL(url_spec);
« no previous file with comments | « no previous file | chrome/browser/sessions/session_service_test_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698