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

chrome/browser/ui/browser.cc

 // Copyright (c) 2011 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/browser/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 1230 matching lines @@
     return false;
   controller.GoToIndex(index);
   return true;
 }
 
 browser::NavigateParams Browser::GetSingletonTabNavigateParams(
     const GURL& url) {
   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;
 }
 
 void Browser::ShowSingletonTab(const GURL& url) {
   browser::NavigateParams params(GetSingletonTabNavigateParams(url));
   browser::Navigate(&params);
 }
 
 void Browser::ShowSingletonTabOverwritingNTP(
@@ skipping 3579 matching lines @@
 }
 
 void Browser::ShowSyncSetup() {
   ProfileSyncService* service =
       profile()->GetOriginalProfile()->GetProfileSyncService();
   if (service->HasSyncSetupCompleted())
     ShowOptionsTab(chrome::kSyncSetupSubPage);
   else
     service->ShowLoginDialog();
 }