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

Side by Side Diff: chrome/browser/ui/views/harmony/chrome_typography.cc

Issue 2904823002: Inactive toast ux changes (Closed)
Patch Set: remove the need for a font_list Created 3 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/harmony/chrome_typography.h" 5 #include "chrome/browser/ui/views/harmony/chrome_typography.h"
6 6
7 #include "ui/base/default_style.h" 7 #include "ui/base/default_style.h"
8 #include "ui/base/resource/resource_bundle.h" 8 #include "ui/base/resource/resource_bundle.h"
9 9
10 const gfx::FontList& LegacyTypographyProvider::GetFont(int context, 10 const gfx::FontList& LegacyTypographyProvider::GetFont(int context,
11 int style) const { 11 int style) const {
12 constexpr int kHeadlineDelta = 8; 12 constexpr int kHeadlineDelta = 8;
13 constexpr int kDialogMessageDelta = 1; 13 constexpr int kDialogMessageDelta = 1;
14 14
15 int size_delta; 15 int size_delta;
16 gfx::Font::Weight font_weight; 16 gfx::Font::Weight font_weight;
17 GetDefaultFont(context, style, &size_delta, &font_weight); 17 GetDefaultFont(context, style, &size_delta, &font_weight);
18 18
19 #if defined(USE_ASH) 19 #if defined(USE_ASH)
20 ash::ApplyAshFontStyles(context, style, &size_delta, &font_weight); 20 ash::ApplyAshFontStyles(context, style, &size_delta, &font_weight);
21 #endif 21 #endif
22 22
23 ApplyCommonFontStyles(context, style, &size_delta, &font_weight);
24
23 switch (context) { 25 switch (context) {
24 case CONTEXT_HEADLINE: 26 case CONTEXT_HEADLINE:
25 size_delta = kHeadlineDelta; 27 size_delta = kHeadlineDelta;
26 break; 28 break;
27 case CONTEXT_BODY_TEXT_LARGE: 29 case CONTEXT_BODY_TEXT_LARGE:
28 // Note: Not using ui::kMessageFontSizeDelta, so 13pt in most cases. 30 // Note: Not using ui::kMessageFontSizeDelta, so 13pt in most cases.
29 size_delta = kDialogMessageDelta; 31 size_delta = kDialogMessageDelta;
30 break; 32 break;
31 case CONTEXT_BODY_TEXT_SMALL: 33 case CONTEXT_BODY_TEXT_SMALL:
32 size_delta = ui::kLabelFontSizeDelta; 34 size_delta = ui::kLabelFontSizeDelta;
(...skipping 15 matching lines...) Expand all
48 50
49 SkColor LegacyTypographyProvider::GetColor(int context, 51 SkColor LegacyTypographyProvider::GetColor(int context,
50 int style, 52 int style,
51 const ui::NativeTheme& theme) const { 53 const ui::NativeTheme& theme) const {
52 // Use "disabled grey" for HINT and SECONDARY when Harmony is disabled. 54 // Use "disabled grey" for HINT and SECONDARY when Harmony is disabled.
53 if (style == STYLE_HINT || style == STYLE_SECONDARY) 55 if (style == STYLE_HINT || style == STYLE_SECONDARY)
54 style = views::style::STYLE_DISABLED; 56 style = views::style::STYLE_DISABLED;
55 57
56 return DefaultTypographyProvider::GetColor(context, style, theme); 58 return DefaultTypographyProvider::GetColor(context, style, theme);
57 } 59 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698