Expect two tabs in StartupSupervisedUserProfile test on Windows 10.

Expect two tabs in StartupSupervisedUserProfile test on Windows 10.

Follow-on CL from https://codereview.chromium.org/1226643002

BUG=511859
TEST=browser_tests --gtest_filter=SupervisedUserBrowserCreatorTest.StartupSupervisedUserProfile

Committed: https://crrev.com/629687e959bee49c9e17f64195ceec3dd06e94f3
Cr-Commit-Position: refs/heads/master@{#339791}

[Will Harris, 2015-07-21 22:41:43]

wfh@chromium.org changed reviewers:
+ msw@chromium.org

[Will Harris, 2015-07-21 22:44:27]

PTAL

[msw, 2015-07-21 23:14:35]

lgtm with optional nit

https://codereview.chromium.org/1247063007/diff/20001/chrome/browser/ui/start...
File chrome/browser/ui/startup/startup_browser_creator_browsertest.cc (right):

https://codereview.chromium.org/1247063007/diff/20001/chrome/browser/ui/start...
chrome/browser/ui/startup/startup_browser_creator_browsertest.cc:1191: if
(IsWindows10OrNewer())
nit: consider:
  const size_t tab_count = IsWindows10OrNewer() ? 2 : 1;
  EXPECT_EQ(tab_count, tab_strip->count());

[Will Harris, 2015-07-21 23:21:08]

I had a small cry at the static_cast in tab_strip_model.h

https://codereview.chromium.org/1247063007/diff/20001/chrome/browser/ui/start...
File chrome/browser/ui/startup/startup_browser_creator_browsertest.cc (right):

https://codereview.chromium.org/1247063007/diff/20001/chrome/browser/ui/start...
chrome/browser/ui/startup/startup_browser_creator_browsertest.cc:1191: if
(IsWindows10OrNewer())
On 2015/07/21 23:14:35, msw wrote:
> nit: consider:
>   const size_t tab_count = IsWindows10OrNewer() ? 2 : 1;
>   EXPECT_EQ(tab_count, tab_strip->count());

Done. TabStripModel::count() returns a signed int though :(

[Will Harris, 2015-07-21 23:21:22]

The CQ bit was checked by wfh@chromium.org

[Will Harris, 2015-07-21 23:21:22]

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

[commit-bot: I haz the power, 2015-07-21 23:22:06]

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

[commit-bot: I haz the power, 2015-07-22 00:09:25]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-07-22 00:10:28]

Patchset 3 (id:??) landed as
https://crrev.com/629687e959bee49c9e17f64195ceec3dd06e94f3
Cr-Commit-Position: refs/heads/master@{#339791}