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

Unified Diff: chrome/browser/profiles/profile_info_util_unittest.cc

Issue 8711002: Add GAIA picture to Settings profile overlay (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 9 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/profiles/profile_info_util_unittest.cc
diff --git a/chrome/browser/profiles/profile_info_util_unittest.cc b/chrome/browser/profiles/profile_info_util_unittest.cc
index edb0e45048ac702b70de75a9e2ed382d8d50e0c9..0f291ac60f928b5184b4bc95a78b7e8cb86033f7 100644
--- a/chrome/browser/profiles/profile_info_util_unittest.cc
+++ b/chrome/browser/profiles/profile_info_util_unittest.cc
@@ -27,6 +27,21 @@ TEST(ProfileInfoUtilTest, MenuIcon) {
EXPECT_FALSE(gfx::test::IsEqual(gaia_picture, result2));
}
+TEST(ProfileInfoUtilTest, WebUIIcon) {
+ // Test that an avatar icon isn't changed.
+ const gfx::Image& profile_image(
+ ResourceBundle::GetSharedInstance().GetImageNamed(IDR_PROFILE_AVATAR_0));
+ gfx::Image result = profiles::GetAvatarIconForWebUI(profile_image, false);
+ EXPECT_FALSE(gfx::test::IsEmpty(result));
+ EXPECT_TRUE(gfx::test::IsEqual(profile_image, result));
+
+ // Test that a GAIA picture is changed.
+ gfx::Image gaia_picture(gfx::test::CreateImage());
+ gfx::Image result2 = profiles::GetAvatarIconForWebUI(gaia_picture, true);
+ EXPECT_FALSE(gfx::test::IsEmpty(result2));
+ EXPECT_FALSE(gfx::test::IsEqual(gaia_picture, result2));
+}
+
TEST(ProfileInfoUtilTest, TitleBarIcon) {
// Test that an avatar icon isn't changed.
const gfx::Image& profile_image(
« no previous file with comments | « chrome/browser/profiles/profile_info_util.cc ('k') | chrome/browser/resources/options/manage_profile_overlay.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698