Adding Pinned Tab and Preferences-specified Tab logic to Startup Flow refactor

chrome/browser/ui/startup/startup_tab_provider_unittest.cc

Index: chrome/browser/ui/startup/startup_tab_provider_unittest.cc
diff --git a/chrome/browser/ui/startup/startup_tab_provider_unittest.cc b/chrome/browser/ui/startup/startup_tab_provider_unittest.cc
index f82d50e7edc28123104c755458cd324c110e0211..f92fa3066dc7cf71768fb9130dd4e0f5a113f924 100644
--- a/chrome/browser/ui/startup/startup_tab_provider_unittest.cc
+++ b/chrome/browser/ui/startup/startup_tab_provider_unittest.cc
@@ -67,3 +67,30 @@ TEST(StartupTabProviderTest, CheckResetTriggerTabPolicy_Negative) {
 
   ASSERT_TRUE(output.empty());
 }
+
+TEST(StartupTabProviderTest, CheckPreferencesTabPolicy) {
+  SessionStartupPref pref(SessionStartupPref::Type::URLS);
+  pref.urls = {GURL(base::ASCIIToUTF16("https://www.google.com"))};
+

[Peter Kasting, 2016/10/18 00:53:41]

Nit: No blank here?  (You don't do similar blanks in the next test)

[tmartino, 2016/10/19 22:01:29]

I did use blanks in the above tests, so I've added them below instead.

+  StartupTabs output = StartupTabProviderImpl::CheckPreferencesTabPolicy(pref);
+
+  ASSERT_EQ(1U, output.size());
+  EXPECT_EQ("www.google.com", output[0].url.host());
+}
+
+TEST(StartupTabProviderTest, CheckPreferencesTabPolicy_Negative) {
+  SessionStartupPref pref_default(SessionStartupPref::Type::DEFAULT);
+  pref_default.urls =
+      std::vector<GURL>({GURL(base::ASCIIToUTF16("https://www.google.com"))});

[Peter Kasting, 2016/10/18 00:53:41]

Nit: I think you can omit "std::vector<GURL>()" (2 places)

[tmartino, 2016/10/19 22:01:29]

Done

+  StartupTabs output =
+      StartupTabProviderImpl::CheckPreferencesTabPolicy(pref_default);
+
+  EXPECT_TRUE(output.empty());
+
+  SessionStartupPref pref_last(SessionStartupPref::Type::LAST);
+  pref_last.urls =
+      std::vector<GURL>({GURL(base::ASCIIToUTF16("https://www.google.com"))});
+  output = StartupTabProviderImpl::CheckPreferencesTabPolicy(pref_last);
+
+  EXPECT_TRUE(output.empty());
+}