Fixes regression when opening tabs in background with pinned tabs

chrome/browser/ui/tabs/tab_strip_model.cc

 // Copyright (c) 2012 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/tabs/tab_strip_model.h"
 
 #include <algorithm>
 #include <map>
 #include <set>
 #include <string>
@@ skipping 588 matching lines @@
   DCHECK(opener);
   DCHECK(ContainsIndex(start_index));
 
   std::set<const WebContents*> opener_and_descendants;
   opener_and_descendants.insert(opener);
   int last_index = kNoTab;
 
   for (int i = start_index + 1; i < count(); ++i) {
     // Test opened by transitively, i.e. include tabs opened by tabs opened by
     // opener, etc. Stop when we find the first non-descendant.
-    if (!opener_and_descendants.count(contents_data_[i]->opener()))
+    if (!opener_and_descendants.count(contents_data_[i]->opener())) {
+      // Skip over pinned tabs as new tabs are added after pinned tabs.
+      if (contents_data_[i]->pinned())
+        continue;
       break;
+    }
     opener_and_descendants.insert(contents_data_[i]->web_contents());
     last_index = i;
   }
   return last_index;
 }
 
 void TabStripModel::TabNavigating(WebContents* contents,
                                   ui::PageTransition transition) {
   if (ShouldForgetOpenersForTransition(transition)) {
     // Don't forget the openers if this tab is a New Tab page opened at the
@@ skipping 817 matching lines @@
 
 void TabStripModel::FixOpenersAndGroupsReferencing(int index) {
   WebContents* old_contents = GetWebContentsAtImpl(index);
   for (WebContentsData* data : contents_data_) {
     if (data->group() == old_contents)
       data->set_group(contents_data_[index]->group());
     if (data->opener() == old_contents)
       data->set_opener(contents_data_[index]->opener());
   }
 }