Unify and Improve the Sign-In and Sync Confirmation Screens on Clank.

chrome/browser/android/profiles/profile_downloader_android.cc

 // Copyright 2014 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/android/profiles/profile_downloader_android.h"
 
 #include <stddef.h>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
@@ skipping 175 matching lines @@
                                  const JavaParamRef<jobject>& jprofile,
                                  const JavaParamRef<jstring>& jemail,
                                  jint image_side_pixels,
                                  jboolean is_pre_signin) {
   Profile* profile = ProfileAndroid::FromProfileAndroid(jprofile);
   const std::string email =
       base::android::ConvertJavaStringToUTF8(env, jemail);
   AccountTrackerService* account_tracker_service =
       AccountTrackerServiceFactory::GetForProfile(profile);
 
+  AccountInfo account_info =

[newt (away), 2016/02/09 23:08:12]

Is this supposed to be here? If so, why / what is this fixing? (I'm not familiar
with this class)

[PEConn, 2016/02/11 14:44:56]

This catches the issue where the AccountTrackerService doesn't contain the user
account information of an added Profile and logs an error for it:
crbug.com/585065

+      account_tracker_service->FindAccountInfoByEmail(email);
+
+  if (account_info.account_id.empty()) {
+      LOG(ERROR) << "Could not get valid account information for " << email;
+      return;
+  }
+
   AccountInfoRetriever* retriever = new AccountInfoRetriever(
       profile,
       account_tracker_service->FindAccountInfoByEmail(email).account_id, email,
       image_side_pixels, is_pre_signin);
   retriever->Start();
 }
 
 // static
 bool RegisterProfileDownloader(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }