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

Side by Side Diff: chrome/browser/themes/theme_properties.h

Issue 2667753004: Themes: Remove unused IDs from ThemeProperties (Closed)
Patch Set: Remove {} and update/add comments Created 3 years, 10 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 #ifndef CHROME_BROWSER_THEMES_THEME_PROPERTIES_H_ 5 #ifndef CHROME_BROWSER_THEMES_THEME_PROPERTIES_H_
6 #define CHROME_BROWSER_THEMES_THEME_PROPERTIES_H_ 6 #define CHROME_BROWSER_THEMES_THEME_PROPERTIES_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 22 matching lines...) Expand all
33 // GetDefaultColor). 33 // GetDefaultColor).
34 COLOR_FRAME_INCOGNITO, 34 COLOR_FRAME_INCOGNITO,
35 COLOR_FRAME_INCOGNITO_INACTIVE, 35 COLOR_FRAME_INCOGNITO_INACTIVE,
36 COLOR_TOOLBAR, 36 COLOR_TOOLBAR,
37 COLOR_TAB_TEXT, 37 COLOR_TAB_TEXT,
38 COLOR_BACKGROUND_TAB_TEXT, 38 COLOR_BACKGROUND_TAB_TEXT,
39 COLOR_BOOKMARK_TEXT, 39 COLOR_BOOKMARK_TEXT,
40 COLOR_NTP_BACKGROUND, 40 COLOR_NTP_BACKGROUND,
41 COLOR_NTP_TEXT, 41 COLOR_NTP_TEXT,
42 COLOR_NTP_LINK, 42 COLOR_NTP_LINK,
43 COLOR_NTP_LINK_UNDERLINE,
44 COLOR_NTP_HEADER, 43 COLOR_NTP_HEADER,
45 COLOR_NTP_SECTION,
46 COLOR_NTP_SECTION_TEXT,
47 COLOR_NTP_SECTION_LINK,
48 COLOR_NTP_SECTION_LINK_UNDERLINE,
49 COLOR_BUTTON_BACKGROUND, 44 COLOR_BUTTON_BACKGROUND,
50 45
51 TINT_BUTTONS, 46 TINT_BUTTONS,
52 TINT_FRAME, 47 TINT_FRAME,
53 TINT_FRAME_INACTIVE, 48 TINT_FRAME_INACTIVE,
54 TINT_FRAME_INCOGNITO, 49 TINT_FRAME_INCOGNITO,
55 TINT_FRAME_INCOGNITO_INACTIVE, 50 TINT_FRAME_INCOGNITO_INACTIVE,
56 TINT_BACKGROUND_TAB, 51 TINT_BACKGROUND_TAB,
57 52
58 NTP_BACKGROUND_ALIGNMENT, 53 NTP_BACKGROUND_ALIGNMENT,
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 112
118 // The throbber colors for tabs or anything on a toolbar (currently, only 113 // The throbber colors for tabs or anything on a toolbar (currently, only
119 // the download shelf). If you're adding a throbber elsewhere, such as in 114 // the download shelf). If you're adding a throbber elsewhere, such as in
120 // a dialog or bubble, you likely want 115 // a dialog or bubble, you likely want
121 // NativeTheme::kColorId_ThrobberSpinningColor. 116 // NativeTheme::kColorId_ThrobberSpinningColor.
122 COLOR_TAB_THROBBER_SPINNING, 117 COLOR_TAB_THROBBER_SPINNING,
123 COLOR_TAB_THROBBER_WAITING, 118 COLOR_TAB_THROBBER_WAITING,
124 119
125 // These colors don't have constant default values. They are derived from 120 // These colors don't have constant default values. They are derived from
126 // the runtime value of other colors. 121 // the runtime value of other colors.
127 COLOR_NTP_SECTION_HEADER_TEXT,
128 COLOR_NTP_SECTION_HEADER_TEXT_HOVER,
129 COLOR_NTP_SECTION_HEADER_RULE,
130 COLOR_NTP_SECTION_HEADER_RULE_LIGHT,
131 COLOR_NTP_TEXT_LIGHT, 122 COLOR_NTP_TEXT_LIGHT,
132 #if BUILDFLAG(ENABLE_SUPERVISED_USERS) 123 #if BUILDFLAG(ENABLE_SUPERVISED_USERS)
133 COLOR_SUPERVISED_USER_LABEL, 124 COLOR_SUPERVISED_USER_LABEL,
134 COLOR_SUPERVISED_USER_LABEL_BACKGROUND, 125 COLOR_SUPERVISED_USER_LABEL_BACKGROUND,
135 COLOR_SUPERVISED_USER_LABEL_BORDER, 126 COLOR_SUPERVISED_USER_LABEL_BORDER,
136 #endif 127 #endif
137 128
138 #if defined(OS_MACOSX) 129 #if defined(OS_MACOSX)
139 COLOR_FRAME_VIBRANCY_OVERLAY, 130 COLOR_FRAME_VIBRANCY_OVERLAY,
140 COLOR_TOOLBAR_INACTIVE, 131 COLOR_TOOLBAR_INACTIVE,
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 174
184 // Returns the default color for the given color |id| COLOR_* enum value. 175 // Returns the default color for the given color |id| COLOR_* enum value.
185 // Returns gfx::kPlaceholderColor if |id| is invalid. 176 // Returns gfx::kPlaceholderColor if |id| is invalid.
186 static SkColor GetDefaultColor(int id, bool otr); 177 static SkColor GetDefaultColor(int id, bool otr);
187 178
188 private: 179 private:
189 DISALLOW_IMPLICIT_CONSTRUCTORS(ThemeProperties); 180 DISALLOW_IMPLICIT_CONSTRUCTORS(ThemeProperties);
190 }; 181 };
191 182
192 #endif // CHROME_BROWSER_THEMES_THEME_PROPERTIES_H_ 183 #endif // CHROME_BROWSER_THEMES_THEME_PROPERTIES_H_
OLDNEW
« no previous file with comments | « chrome/browser/themes/browser_theme_pack_unittest.cc ('k') | chrome/browser/themes/theme_properties.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698