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

Unified Diff: chrome/browser/themes/theme_properties.cc

Issue 1885183002: Allow constexpr, and use it in a few places. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add guidance Created 4 years, 8 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
« no previous file with comments | « no previous file | styleguide/c++/c++11.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/themes/theme_properties.cc
diff --git a/chrome/browser/themes/theme_properties.cc b/chrome/browser/themes/theme_properties.cc
index d4fb13af7afdac4d0c279ecba45ae999eafdff9d..8d5a8effd608e60914f88a6863f36b45775dd9bb 100644
--- a/chrome/browser/themes/theme_properties.cc
+++ b/chrome/browser/themes/theme_properties.cc
@@ -71,14 +71,15 @@ const SkColor kDefaultDetachedBookmarkBarBackground[] = {
const SkColor kDefaultDetachedBookmarkBarBackgroundIncognito[] = {
SkColorSetRGB(0xF1, 0xF1, 0xF1), SkColorSetRGB(0x32, 0x32, 0x32)};
-const SkColor kDefaultColorTabText = SK_ColorBLACK;
-const SkColor kDefaultColorTabTextIncognito[] = {SK_ColorBLACK, SK_ColorWHITE};
+constexpr SkColor kDefaultColorTabText = SK_ColorBLACK;
+constexpr SkColor kDefaultColorTabTextIncognito[] = {kDefaultColorTabText,
+ SK_ColorWHITE};
#if defined(OS_MACOSX)
-const SkColor kDefaultColorBackgroundTabText[] = {
- SK_ColorBLACK, SK_ColorBLACK };
-const SkColor kDefaultColorBackgroundTabTextIncognito[] = {
- SK_ColorBLACK, SK_ColorWHITE };
+constexpr SkColor kDefaultColorBackgroundTabText[] = {SK_ColorBLACK,
+ SK_ColorBLACK};
+constexpr SkColor kDefaultColorBackgroundTabTextIncognito[] = {
+ kDefaultColorBackgroundTabText[0], SK_ColorWHITE};
#else
const SkColor kDefaultColorBackgroundTabText[] = {
SkColorSetRGB(64, 64, 64), SK_ColorBLACK };
@@ -86,9 +87,9 @@ const SkColor kDefaultColorBackgroundTabTextIncognito[] = {
SkColorSetRGB(64, 64, 64), SK_ColorWHITE };
#endif // OS_MACOSX
-const SkColor kDefaultColorBookmarkText = SK_ColorBLACK;
-const SkColor kDefaultColorBookmarkTextIncognito[] = {SK_ColorBLACK,
- SK_ColorWHITE};
+constexpr SkColor kDefaultColorBookmarkText = SK_ColorBLACK;
+constexpr SkColor kDefaultColorBookmarkTextIncognito[] = {
+ kDefaultColorBookmarkText, SK_ColorWHITE};
#if defined(OS_WIN)
const SkColor kDefaultColorNTPBackground =
@@ -99,32 +100,31 @@ const SkColor kDefaultColorNTPLink =
color_utils::GetSysSkColor(COLOR_HOTLIGHT);
#else
// TODO(beng): source from theme provider.
-const SkColor kDefaultColorNTPBackground = SK_ColorWHITE;
-const SkColor kDefaultColorNTPText = SK_ColorBLACK;
+constexpr SkColor kDefaultColorNTPBackground = SK_ColorWHITE;
+constexpr SkColor kDefaultColorNTPText = SK_ColorBLACK;
const SkColor kDefaultColorNTPLink = SkColorSetRGB(6, 55, 116);
#endif // OS_WIN
const SkColor kDefaultColorNTPHeader = SkColorSetRGB(150, 150, 150);
const SkColor kDefaultColorNTPSection = SkColorSetRGB(229, 229, 229);
-const SkColor kDefaultColorNTPSectionText = SK_ColorBLACK;
+constexpr SkColor kDefaultColorNTPSectionText = SK_ColorBLACK;
const SkColor kDefaultColorNTPSectionLink = SkColorSetRGB(6, 55, 116);
-const SkColor kDefaultColorButtonBackground = SkColorSetARGB(0, 0, 0, 0);
+constexpr SkColor kDefaultColorButtonBackground = SK_ColorTRANSPARENT;
// Default tints.
-const color_utils::HSL kDefaultTintButtons = {-1, -1, -1};
-// In pre-md, reuse the normal tint for incognito.
-const color_utils::HSL kDefaultTintButtonsIncognito[] = {{-1, -1, -1},
- {-1, -1, 0.85}};
-const color_utils::HSL kDefaultTintFrame = {-1, -1, -1};
-const color_utils::HSL kDefaultTintFrameInactive = {-1, -1, 0.75};
-const color_utils::HSL kDefaultTintFrameIncognito = {-1, 0.2, 0.35};
-const color_utils::HSL kDefaultTintFrameIncognitoInactive = {-1, 0.3, 0.6};
-const color_utils::HSL kDefaultTintBackgroundTab = {-1, -1, 0.75};
+constexpr color_utils::HSL kDefaultTintButtons = {-1, -1, -1};
+constexpr color_utils::HSL kDefaultTintButtonsIncognito[] = {
+ kDefaultTintButtons, {-1, -1, 0.85}};
+constexpr color_utils::HSL kDefaultTintFrame = {-1, -1, -1};
+constexpr color_utils::HSL kDefaultTintFrameInactive = {-1, -1, 0.75};
+constexpr color_utils::HSL kDefaultTintFrameIncognito = {-1, 0.2, 0.35};
+constexpr color_utils::HSL kDefaultTintFrameIncognitoInactive = {-1, 0.3, 0.6};
+constexpr color_utils::HSL kDefaultTintBackgroundTab = {-1, -1, 0.75};
// ----------------------------------------------------------------------------
// Defaults for properties which are not stored in the browser theme pack.
-const SkColor kDefaultColorControlBackground = SK_ColorWHITE;
+constexpr SkColor kDefaultColorControlBackground = SK_ColorWHITE;
const SkColor kDefaultDetachedBookmarkBarSeparator[] = {
SkColorSetRGB(170, 170, 171), SkColorSetRGB(182, 180, 182)};
const SkColor kDefaultDetachedBookmarkBarSeparatorIncognito[] = {
@@ -159,17 +159,17 @@ const SkColor kDefaultColorToolbarStrokeThemeInactive =
// ----------------------------------------------------------------------------
// Strings used in alignment properties.
-const char kAlignmentCenter[] = "center";
-const char kAlignmentTop[] = "top";
-const char kAlignmentBottom[] = "bottom";
-const char kAlignmentLeft[] = "left";
-const char kAlignmentRight[] = "right";
+constexpr char kAlignmentCenter[] = "center";
+constexpr char kAlignmentTop[] = "top";
+constexpr char kAlignmentBottom[] = "bottom";
+constexpr char kAlignmentLeft[] = "left";
+constexpr char kAlignmentRight[] = "right";
// Strings used in background tiling repetition properties.
-const char kTilingNoRepeat[] = "no-repeat";
-const char kTilingRepeatX[] = "repeat-x";
-const char kTilingRepeatY[] = "repeat-y";
-const char kTilingRepeat[] = "repeat";
+constexpr char kTilingNoRepeat[] = "no-repeat";
+constexpr char kTilingRepeatX[] = "repeat-x";
+constexpr char kTilingRepeatY[] = "repeat-y";
+constexpr char kTilingRepeat[] = "repeat";
// The image resources that will be tinted by the 'button' tint value.
// If you change this list, you must increment the version number in
@@ -179,17 +179,35 @@ const char kTilingRepeat[] = "repeat";
//
// TODO(erg): The cocoa port is the last user of the IDR_*_[HP] variants. These
// should be removed once the cocoa port no longer uses them.
-const int kToolbarButtonIDs[] = {
- IDR_BACK, IDR_BACK_D, IDR_BACK_H, IDR_BACK_P,
- IDR_FORWARD, IDR_FORWARD_D, IDR_FORWARD_H, IDR_FORWARD_P,
- IDR_HOME, IDR_HOME_H, IDR_HOME_P,
- IDR_RELOAD, IDR_RELOAD_H, IDR_RELOAD_P,
- IDR_STOP, IDR_STOP_D, IDR_STOP_H, IDR_STOP_P,
- IDR_BROWSER_ACTIONS_OVERFLOW, IDR_BROWSER_ACTIONS_OVERFLOW_H,
- IDR_BROWSER_ACTIONS_OVERFLOW_P,
- IDR_TOOLS, IDR_TOOLS_H, IDR_TOOLS_P,
- IDR_MENU_DROPARROW,
- IDR_TOOLBAR_BEZEL_HOVER, IDR_TOOLBAR_BEZEL_PRESSED, IDR_TOOLS_BAR,
+constexpr int kToolbarButtonIDs[] = {
+ IDR_BACK,
+ IDR_BACK_D,
+ IDR_BACK_H,
+ IDR_BACK_P,
+ IDR_FORWARD,
+ IDR_FORWARD_D,
+ IDR_FORWARD_H,
+ IDR_FORWARD_P,
+ IDR_HOME,
+ IDR_HOME_H,
+ IDR_HOME_P,
+ IDR_RELOAD,
+ IDR_RELOAD_H,
+ IDR_RELOAD_P,
+ IDR_STOP,
+ IDR_STOP_D,
+ IDR_STOP_H,
+ IDR_STOP_P,
+ IDR_BROWSER_ACTIONS_OVERFLOW,
+ IDR_BROWSER_ACTIONS_OVERFLOW_H,
+ IDR_BROWSER_ACTIONS_OVERFLOW_P,
+ IDR_TOOLS,
+ IDR_TOOLS_H,
+ IDR_TOOLS_P,
+ IDR_MENU_DROPARROW,
+ IDR_TOOLBAR_BEZEL_HOVER,
+ IDR_TOOLBAR_BEZEL_PRESSED,
+ IDR_TOOLS_BAR,
};
SkColor TintForUnderline(SkColor input) {
« no previous file with comments | « no previous file | styleguide/c++/c++11.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698