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

Unified Diff: components/favicon_base/fallback_icon_style.h

Issue 2883053003: Delete unused chrome://fallback-icon/ handling (Closed)
Patch Set: Merge branch 'searchbox2' into searchbox3 Created 3 years, 7 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: components/favicon_base/fallback_icon_style.h
diff --git a/components/favicon_base/fallback_icon_style.h b/components/favicon_base/fallback_icon_style.h
index 925773080eb1343bb95a352c89126dea01329a12..cfd62037e7f9b807b1769cdf752dc56de611e6c3 100644
--- a/components/favicon_base/fallback_icon_style.h
+++ b/components/favicon_base/fallback_icon_style.h
@@ -17,30 +17,13 @@ struct FallbackIconStyle {
FallbackIconStyle();
~FallbackIconStyle();
- // If any member changes, also update FallbackIconStyleBuilder.
-
// Icon background fill color.
SkColor background_color;
bool is_default_background_color;
- // Icon text color.
- SkColor text_color;
-
- // Ratio in [0.0, 1.0] of the text font size (pixels) to the icon size.
- double font_size_ratio;
-
- // The roundness of the icon's corners. 0 => square icon, 1 => circle icon.
- double roundness;
-
bool operator==(const FallbackIconStyle& other) const;
};
-// Reassigns |style|'s |text_color| to matches well against |background_color|.
-void MatchFallbackIconTextColorAgainstBackgroundColor(FallbackIconStyle* style);
-
-// Returns whether |style| values are within bounds.
-bool ValidateFallbackIconStyle(const FallbackIconStyle& style);
-
// Set |style|'s background color to the dominant color of |bitmap_data|,
// clamping luminance down to a reasonable maximum value so that light text is
// readable.

Powered by Google App Engine
This is Rietveld 408576698