Fixes regression when opening tabs in background with pinned tabs

Fixes regression when opening tabs in background with pinned tabs

If the active tab is pinned and you have at least one pinned tab after the
active tab then we would end up opening new link clicks after the
pinned tab regardless of the opener. This is a regression from
https://codereview.chromium.org/836223004 .

BUG=446997, 446998, 476038
TEST=covered by test now.
R=pkasting@chromium.org

Committed: https://crrev.com/cce040b21a2d7ed6ce36197f9c02407bee1c2c68
Cr-Commit-Position: refs/heads/master@{#329419}

[Peter Kasting, 2015-05-11 22:10:32]

LGTM

https://codereview.chromium.org/1131233004/diff/20001/chrome/browser/ui/tabs/...
File chrome/browser/ui/tabs/tab_strip_model_unittest.cc (right):

https://codereview.chromium.org/1131233004/diff/20001/chrome/browser/ui/tabs/...
chrome/browser/ui/tabs/tab_strip_model_unittest.cc:2692: // Verifies ordering of
tabs opened via link from a pinned tab with subsequent
Nit: link -> a link, with -> with a

[sky, 2015-05-11 22:13:57]

https://codereview.chromium.org/1131233004/diff/20001/chrome/browser/ui/tabs/...
File chrome/browser/ui/tabs/tab_strip_model_unittest.cc (right):

https://codereview.chromium.org/1131233004/diff/20001/chrome/browser/ui/tabs/...
chrome/browser/ui/tabs/tab_strip_model_unittest.cc:2692: // Verifies ordering of
tabs opened via link from a pinned tab with subsequent
On 2015/05/11 22:10:32, Peter Kasting wrote:
> Nit: link -> a link, with -> with a

Done.

[sky, 2015-05-11 22:14:26]

The CQ bit was checked by sky@chromium.org

[sky, 2015-05-11 22:14:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org
Link to the patchset: https://codereview.chromium.org/1131233004/#ps40001
(title: "feedback")

[commit-bot: I haz the power, 2015-05-11 22:17:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1131233004/40001

[commit-bot: I haz the power, 2015-05-11 22:27:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-11 22:27:58]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[sky, 2015-05-12 15:55:00]

The CQ bit was checked by sky@chromium.org

[sky, 2015-05-12 15:55:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org
Link to the patchset: https://codereview.chromium.org/1131233004/#ps60001
(title: "change comment so dont trigger copyright check")

[commit-bot: I haz the power, 2015-05-12 15:55:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1131233004/60001

[commit-bot: I haz the power, 2015-05-12 16:43:14]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-12 16:44:03]

Patchset 4 (id:??) landed as
https://crrev.com/cce040b21a2d7ed6ce36197f9c02407bee1c2c68
Cr-Commit-Position: refs/heads/master@{#329419}