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

Unified Diff: chrome/browser/ui/browser_navigator.cc

Issue 7701015: Making BrowserNavigator open file browser singleton tab respecting URLs ref value (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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
Index: chrome/browser/ui/browser_navigator.cc
diff --git a/chrome/browser/ui/browser_navigator.cc b/chrome/browser/ui/browser_navigator.cc
index ff2645e3f1a666ec6ffef17ca971ea4931f86727..42f08252155c84e67cfa73421edf76fb368e11e5 100644
--- a/chrome/browser/ui/browser_navigator.cc
+++ b/chrome/browser/ui/browser_navigator.cc
@@ -293,6 +293,7 @@ NavigateParams::NavigateParams(
window_action(NO_ACTION),
user_gesture(true),
path_behavior(RESPECT),
+ ref_behavior(IGNORE_REF),
browser(a_browser),
profile(NULL) {
}
@@ -308,6 +309,7 @@ NavigateParams::NavigateParams(Browser* a_browser,
window_action(NO_ACTION),
user_gesture(true),
path_behavior(RESPECT),
+ ref_behavior(IGNORE_REF),
browser(a_browser),
profile(NULL) {
}
@@ -501,7 +503,8 @@ int GetIndexOfSingletonTab(browser::NavigateParams* params) {
params->browser->GetTabContentsWrapperAt(tab_index);
url_canon::Replacements<char> replacements;
- replacements.ClearRef();
+ if (params->ref_behavior == browser::NavigateParams::IGNORE_REF)
Greg Billock 2011/08/23 23:38:48 Can you add a browser_navigator_browsertest for th
tbarzic 2011/08/24 00:02:58 will do
+ replacements.ClearRef();
if (params->path_behavior == browser::NavigateParams::IGNORE_AND_NAVIGATE ||
params->path_behavior == browser::NavigateParams::IGNORE_AND_STAY_PUT) {
replacements.ClearPath();
« chrome/browser/ui/browser_navigator.h ('K') | « chrome/browser/ui/browser_navigator.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698