Follow-up to bug 102685 (r113862)

BUG=102685


Review URL: http://codereview.chromium.org/8921008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114171 0039d316-1c4b-4281-b951-d872f2087c98
parent ad8a4396
...@@ -61,6 +61,13 @@ SkBitmap GetGAIAPictureForNTP(const gfx::Image& image) { ...@@ -61,6 +61,13 @@ SkBitmap GetGAIAPictureForNTP(const gfx::Image& image) {
return canvas.ExtractBitmap(); return canvas.ExtractBitmap();
} }
// Puts the |content| into a span with the given CSS class.
string16 CreateSpanWithClass(const string16& content,
const std::string& css_class) {
return ASCIIToUTF16("<span class='" + css_class + "'>") +
net::EscapeForHTML(content) + ASCIIToUTF16("</span>");
}
} // namespace } // namespace
NTPLoginHandler::NTPLoginHandler() { NTPLoginHandler::NTPLoginHandler() {
...@@ -186,27 +193,21 @@ void NTPLoginHandler::UpdateLogin() { ...@@ -186,27 +193,21 @@ void NTPLoginHandler::UpdateLogin() {
// title bar and the full name can be ambiguous. // title bar and the full name can be ambiguous.
if (cache.GetNumberOfProfiles() == 1) { if (cache.GetNumberOfProfiles() == 1) {
string16 name = cache.GetGAIANameOfProfileAtIndex(profile_index); string16 name = cache.GetGAIANameOfProfileAtIndex(profile_index);
header = ASCIIToUTF16("<span class='profile-name'>") + header = CreateSpanWithClass(name, "profile-name");
net::EscapeForHTML(name) +
ASCIIToUTF16("</span>");
const gfx::Image* image = const gfx::Image* image =
cache.GetGAIAPictureOfProfileAtIndex(profile_index); cache.GetGAIAPictureOfProfileAtIndex(profile_index);
if (image) if (image)
icon_url = web_ui_util::GetImageDataUrl(GetGAIAPictureForNTP(*image)); icon_url = web_ui_util::GetImageDataUrl(GetGAIAPictureForNTP(*image));
} }
if (header.empty()) { if (header.empty())
header = UTF8ToUTF16("<span class='profile-name'>" + header = CreateSpanWithClass(UTF8ToUTF16(username), "profile-name");
net::EscapeForHTML(username) + "</span>");
}
} }
} else if (SyncPromoUI::ShouldShowSyncPromo(profile) && } else if (SyncPromoUI::ShouldShowSyncPromo(profile) &&
(SyncPromoUI::UserHasSeenSyncPromoAtStartup(profile) || (SyncPromoUI::UserHasSeenSyncPromoAtStartup(profile) ||
PromoResourceService::CanShowNTPSignInPromo(profile))) { PromoResourceService::CanShowNTPSignInPromo(profile))) {
string16 signed_in_link = l10n_util::GetStringUTF16( string16 signed_in_link = l10n_util::GetStringUTF16(
IDS_SYNC_PROMO_NOT_SIGNED_IN_STATUS_LINK); IDS_SYNC_PROMO_NOT_SIGNED_IN_STATUS_LINK);
signed_in_link = ASCIIToUTF16("<span class='link-span'>") + signed_in_link = CreateSpanWithClass(signed_in_link, "link-span");
net::EscapeForHTML(signed_in_link) +
ASCIIToUTF16("</span>");
header = l10n_util::GetStringFUTF16( header = l10n_util::GetStringFUTF16(
IDS_SYNC_PROMO_NOT_SIGNED_IN_STATUS_HEADER, IDS_SYNC_PROMO_NOT_SIGNED_IN_STATUS_HEADER,
l10n_util::GetStringUTF16(IDS_SHORT_PRODUCT_NAME)); l10n_util::GetStringUTF16(IDS_SHORT_PRODUCT_NAME));
......
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