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 =
+      account_tracker_service->FindAccountInfoByEmail(email);
+
+  if (account_info.account_id.empty()) {
+      LOG(ERROR) << "Could not get valid account information for " << email;

[newt (away), 2016/02/11 16:46:06]

I believe we shouldn't log email addresses. I'd remove the email here, or check
with someone to make sure it's OK.

[PEConn, 2016/02/12 18:41:20]

Done.

+      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);
 }