Skip profiles in GetLastOpenedProfiles that fail to initialize

chrome/browser/profiles/profile.h

Index: chrome/browser/profiles/profile.h
diff --git a/chrome/browser/profiles/profile.h b/chrome/browser/profiles/profile.h
index f5feb756b325422e5159699680eaff0c9890e264..d304b81ff5c5178bdcf2d64627c77ba4f3fc2a76 100644
--- a/chrome/browser/profiles/profile.h
+++ b/chrome/browser/profiles/profile.h
@@ -129,6 +129,8 @@ class Profile : public content::BrowserContext {
 
   // Create a new profile given a path. If |create_mode| is
   // CREATE_MODE_ASYNCHRONOUS then the profile is initialized asynchronously.
+  // If |create_mode| is CREATE_MODE_SYNCHRONOUS, and the creation of the
+  // profile directory fails, nullptr is returned.

[Peter Kasting, 2016/05/31 19:12:30]

You said there are many other callers that presume this never returns null. 
Will you be fixing them separately?  If not, who will be?  We definitely want to
address that issue now that we're aware of it.

Nit: Avoid passive voice:

  // This can return null if |create_mode| is CREATE_MODE_SYNCHRONOUS and the
  // creation of the profile directory fails.

[WC Leung, 2016/06/01 18:56:55]

I'll continue to fix them, most likely in separate CLs. Without fixing those
related problems, the crashes simply continue in a different form.

Anyway, I need to discuss with Anthony and maybe other people in the profile
team about what to do in the other cases, particularly what to do if the
|last_used_profile| cannot be opened.
Done.

   static Profile* CreateProfile(const base::FilePath& path,
                                 Delegate* delegate,
                                 CreateMode create_mode);