Remove TODO in AvatarMenu::GetImageForMenuButton
Change-Id: Ie122cdd92aff2bdadddf1106018fc994fae81fdc
Reviewed-on: https://chromium-review.googlesource.com/c/1347099
Reviewed-by: David Roger <[email protected]>
Commit-Queue: Xi Cheng <[email protected]>
Cr-Commit-Position: refs/heads/master@{#610470}diff --git a/chrome/browser/profiles/avatar_menu_desktop.cc b/chrome/browser/profiles/avatar_menu_desktop.cc
index 40b6a4e3..edaeef0 100644
--- a/chrome/browser/profiles/avatar_menu_desktop.cc
+++ b/chrome/browser/profiles/avatar_menu_desktop.cc
@@ -29,16 +29,15 @@
// If there is a Gaia image available, try to use that.
if (entry->IsUsingGAIAPicture()) {
- // TODO(chengx): The GetGAIAPicture API call will trigger an async image
- // load from disk if it has not been loaded. This is non-obvious and
- // dependency should be avoided. We should come with a better idea to handle
- // this.
+ // The GetGAIAPicture API call will trigger an async image load from disk if
+ // it has not been loaded into memory.
const gfx::Image* gaia_image = entry->GetGAIAPicture();
if (gaia_image) {
*image = *gaia_image;
return ImageLoadStatus::LOADED;
}
+
if (entry->IsGAIAPictureLoaded())
status = ImageLoadStatus::MISSING;
else
