Fix showing some foreground tabs.

chrome/browser/browser_navigator.cc

 // Copyright (c) 2010 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/browser_navigator.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/browser_url_handler.h"
@@ skipping 167 matching lines @@
   if (params->browser->profile()->IsOffTheRecord() &&
       params->disposition == OFF_THE_RECORD) {
     params->disposition = NEW_FOREGROUND_TAB;
   }
 
   // Disposition trumps add types. ADD_SELECTED is a default, so we need to
   // remove it if disposition implies the tab is going to open in the
   // background.
   if (params->disposition == NEW_BACKGROUND_TAB)
     params->tabstrip_add_types &= ~TabStripModel::ADD_SELECTED;
+  else if (params->disposition == NEW_FOREGROUND_TAB)
+    params->tabstrip_add_types |= TabStripModel::ADD_SELECTED;
 
   // Code that wants to open a new window typically expects it to be shown
   // automatically.
   if (params->disposition == NEW_WINDOW || params->disposition == NEW_POPUP) {
     params->show_window = true;
     params->tabstrip_add_types |= TabStripModel::ADD_SELECTED;
   }
 }
 
 // This class makes sure the Browser object held in |params| is made visible
@@ skipping 175 matching lines @@
           params->transition,
           params->tabstrip_add_types);
       // Now that the |params->target_contents| is safely owned by the target
       // Browser's TabStripModel, we can release ownership.
       target_contents_owner.ReleaseOwnership();
     }
   }
 }
 
 }  // namespace browser