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

Unified Diff: chrome/browser/ui/views/first_run_bubble.cc

Issue 2734113006: "Bootstrap" a toolkit-views Typography spec. (Closed)
Patch Set: Rebase (conflict in layout_delegate.h due to r457774) Created 3 years, 9 months 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/views/first_run_bubble.cc
diff --git a/chrome/browser/ui/views/first_run_bubble.cc b/chrome/browser/ui/views/first_run_bubble.cc
index 67177528b78559998ce30b9e6681065ba508cfe0..2a71452cc5ea43feb78938da3658a7468cc5b208 100644
--- a/chrome/browser/ui/views/first_run_bubble.cc
+++ b/chrome/browser/ui/views/first_run_bubble.cc
@@ -46,18 +46,20 @@ void FirstRunBubble::Init() {
GetDefaultSearchEngineName(
TemplateURLServiceFactory::GetForProfile(browser_->profile())) :
base::string16();
+
+ // TODO(tapted): Update these when there are mocks. http://crbug.com/699338.
views::Label* title = new views::Label(
l10n_util::GetStringFUTF16(IDS_FR_BUBBLE_TITLE, search_engine_name),
- original_font_list.Derive(2, gfx::Font::NORMAL, gfx::Font::Weight::BOLD));
+ {original_font_list.Derive(2, gfx::Font::NORMAL,
+ gfx::Font::Weight::BOLD)});
views::Link* change =
new views::Link(l10n_util::GetStringUTF16(IDS_FR_BUBBLE_CHANGE));
change->SetFontList(original_font_list);
change->set_listener(this);
- views::Label* subtext =
- new views::Label(l10n_util::GetStringUTF16(IDS_FR_BUBBLE_SUBTEXT),
- original_font_list);
+ views::Label* subtext = new views::Label(
+ l10n_util::GetStringUTF16(IDS_FR_BUBBLE_SUBTEXT), {original_font_list});
views::GridLayout* layout = views::GridLayout::CreatePanel(this);
SetLayoutManager(layout);

Powered by Google App Engine
This is Rietveld 408576698