Commit 6adbc9ea authored by Xi Cheng's avatar Xi Cheng Committed by Commit Bot

Remove TODO in AvatarToolbarButton::GetIconImageFromProfile

Change-Id: I39c796801d9cfc70465633a9c682d66ba7e41cd6
Reviewed-on: https://chromium-review.googlesource.com/c/1345535
Commit-Queue: Xi Cheng <chengx@chromium.org>
Reviewed-by: default avatarPeter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610191}
parent 0da17cd2
...@@ -331,10 +331,8 @@ gfx::Image AvatarToolbarButton::GetIconImageFromProfile() const { ...@@ -331,10 +331,8 @@ gfx::Image AvatarToolbarButton::GetIconImageFromProfile() const {
// If there is a GAIA image available, try to use that. // If there is a GAIA image available, try to use that.
if (entry->IsUsingGAIAPicture()) { if (entry->IsUsingGAIAPicture()) {
// TODO(chengx): The GetGAIAPicture API call will trigger an async image // The GetGAIAPicture API call will trigger an async image load from disk if
// load from disk if it has not been loaded. This is non-obvious and // it has not been loaded.
// dependency should be avoided. We should come with a better idea to handle
// this.
const gfx::Image* gaia_image = entry->GetGAIAPicture(); const gfx::Image* gaia_image = entry->GetGAIAPicture();
if (gaia_image) if (gaia_image)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment