Fix StartupBrowserCreatorTriggeredResetTest.TestTriggeredResetDoesNotShowWithFirstRunURLs

Fix StartupBrowserCreatorTriggeredResetTest.TestTriggeredResetDoesNotShowWithFirstRunURLs
in official builds.

Expect the first tab to be the signin page during browser tests run for an official build.

BUG=536730
TEST=browser_Tests.exe --gtest_filter=*StartupBrowserCreatorTriggeredResetTest*
TBR=msw

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

[robertshield, 2015-09-28 15:29:13]

Patchset #1 (id:1) has been deleted

[robertshield, 2015-09-28 15:38:41]

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

[robertshield, 2015-09-28 15:38:42]

This has broken tests on the official builder, so I'll land it now, CQ willing.

Mike, please let me know if you'd like this to change and I'll update it in
another CL.

TBR=msw

[robertshield, 2015-09-28 15:38:53]

The CQ bit was checked by robertshield@chromium.org

[commit-bot: I haz the power, 2015-09-28 15:39:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1369323002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1369323002/20001

[commit-bot: I haz the power, 2015-09-28 15:39:29]

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

[commit-bot: I haz the power, 2015-09-28 15:39:30]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[robertshield, 2015-09-28 15:59:06]

The CQ bit was checked by robertshield@chromium.org

[commit-bot: I haz the power, 2015-09-28 15:59:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1369323002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1369323002/20001

[commit-bot: I haz the power, 2015-09-28 16:36:29]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-09-28 16:37:50]

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

[msw, 2015-09-28 18:19:43]

lgtm with very optional nit

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

https://codereview.chromium.org/1369323002/diff/20001/chrome/browser/ui/start...
chrome/browser/ui/startup/startup_browser_creator_triggered_reset_browsertest_win.cc:159:
EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
optional nit for optional follow up refactoring :)
  GURL expected_url = signin::ShouldShowPromoAtStartup(browser()->profile(),
true) ? signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false) :
GURL(chrome::kChromeUINewTabURL);
  EXPECT_EQ(expected_url, tab_strip->GetWebContentsAt(0)->GetURL());