Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5008)

Unified Diff: chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc

Issue 2471993002: Remove calls to User::email() from chrome/browser/ui/webui/* (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc
diff --git a/chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc b/chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc
index cdf1595be754e2faf8468526e197ccba9e74638a..f83a8b2a6e546a2f56d090642e279be67872d2f2 100644
--- a/chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc
+++ b/chrome/browser/ui/webui/settings/chromeos/change_picture_handler.cc
@@ -215,7 +215,7 @@ void ChangePictureHandler::HandlePageInitialized(const base::ListValue* args) {
void ChangePictureHandler::SendSelectedImage() {
const user_manager::User* user = GetUser();
- DCHECK(!user->email().empty());
+ DCHECK(user->GetAccountId().is_valid());
previous_image_index_ = user->image_index();
switch (previous_image_index_) {

Powered by Google App Engine
This is Rietveld 408576698