Ignore and navigate to url when opening bookmark manager for a node.

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 0c8c5e65f25b9893efbb74b76db810e5d525d2c3..968395dc071005c4fba4410f7f4a95d13951cb50 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1248,6 +1248,7 @@ browser::NavigateParams Browser::GetSingletonTabNavigateParams(
   browser::NavigateParams params(this, url, PageTransition::AUTO_BOOKMARK);
   params.disposition = SINGLETON_TAB;
   params.window_action = browser::NavigateParams::SHOW_WINDOW;
+  params.path_behavior = browser::NavigateParams::IGNORE_AND_NAVIGATE;

[Evan Stade, 2011/08/11 17:43:24]

If we always want this, then why do we even have the enumeration? when is
IGNORE_AND_STAY_PUT ever used? when will RESPECT be used after this change?

[flackr, 2011/08/12 14:44:10]

So after playing with the other sections (history, downloads, etc) I'm not sure
whether we always want this or not. I'm not sure if opening downloads should
take you to all downloads if you already have a downloads tab with a search
open. I'll just special case it for OpenBookmarkManagerForNode for now as is
done with ShowOptionsTab.

On 2011/08/11 17:43:24, Evan Stade wrote:

   params.user_gesture = true;
   return params;
 }