Session restore shouldn't care about profile home pages.

chrome/browser/ui/startup/startup_browser_creator_impl.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/startup/startup_browser_creator_impl.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "apps/app_restore_service.h"
@@ skipping 230 matching lines @@
     const extensions::Extension* extension =
         extensions.GetAppByURL(autolaunch_tabs.at(i).url);
     if (extension) {
       CoreAppLauncherHandler::RecordAppLaunchType(
           extension_misc::APP_LAUNCH_AUTOLAUNCH,
           extension->GetType());
     }
   }
 }
 
-bool IsNewTabURL(Profile* profile, const GURL& url) {
-  GURL ntp_url(chrome::kChromeUINewTabURL);
-  return url == ntp_url ||
-         (url.is_empty() && profile->GetHomePage() == ntp_url);
-}
-
 class WebContentsCloseObserver : public content::NotificationObserver {
  public:
   WebContentsCloseObserver() : contents_(NULL) {}
   virtual ~WebContentsCloseObserver() {}
 
   void SetContents(content::WebContents* contents) {
     DCHECK(!contents_);
     contents_ = contents;
 
     registrar_.Add(this,
@@ skipping 616 matching lines @@
   // TODO(atwilson): Simplify the logic that decides which tabs to open on
   // start-up and make it more consistent. http://crbug.com/248883
 
   // If we have urls specified by the first run master preferences use them
   // and nothing else.
   if (browser_creator_ && startup_urls->empty()) {
     if (!browser_creator_->first_run_tabs_.empty()) {
       std::vector<GURL>::iterator it =
           browser_creator_->first_run_tabs_.begin();
       while (it != browser_creator_->first_run_tabs_.end()) {
-        // Replace magic names for the actual urls.
-        if (it->host() == "new_tab_page") {

[sky, 2014/05/20 17:30:54]

Are you sure we aren't actually using these names in some places? Say installers
or perhaps managed/custom installs? I'm adding cpu as he added these.

[cpu_(ooo_6.6-7.5), 2014/05/20 18:35:38]

They where at some point at least. They will be in the master preferences files
that omaha carries then they become the preferences entries in tab restore or
something like that.

marga@ add grt@ to the review or at least ping him, if no answer try
robertshield@

[robertshield, 2014/05/20 19:36:26]

What Carlos said: these magic values are used by distribution partners,
enterprise admins, etc. who wish to customize Chrome at first run using
master_preferences files. 

Is there a reason for deleting them from here?

-          startup_urls->push_back(GURL(chrome::kChromeUINewTabURL));
-        } else if (it->host() == "welcome_page") {
-          startup_urls->push_back(internals::GetWelcomePageURL());
-        } else {
-          startup_urls->push_back(*it);
-        }
+        startup_urls->push_back(*it);
         ++it;
       }
       browser_creator_->first_run_tabs_.clear();
     }
   }
 
   // Otherwise open at least the new tab page (and the welcome page, if this
   // is the first time the browser is being started), or the set of URLs
   // specified on the command line.
   if (startup_urls->empty()) {
@@ skipping 15 matching lines @@
       if (*it == sync_promo_url) {
         add_promo = false;
         break;
       }
     }
 
     if (add_promo) {
       // If the first URL is the NTP, replace it with the sync promo. This
       // behavior is desired because completing or skipping the sync promo
       // causes a redirect to the NTP.
-      if (!startup_urls->empty() && IsNewTabURL(profile_, startup_urls->at(0)))
+      if (!startup_urls->empty() &&
+          startup_urls->at(0) == GURL(chrome::kChromeUINewTabURL))
         startup_urls->at(0) = sync_promo_url;
       else
         startup_urls->insert(startup_urls->begin(), sync_promo_url);
     }
   }
 }