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

Unified Diff: ios/chrome/browser/ui/favicon/favicon_attributes_provider.mm

Issue 2890193003: Open Most Visited items (Closed)
Patch Set: Fix showcase Created 3 years, 6 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: ios/chrome/browser/ui/favicon/favicon_attributes_provider.mm
diff --git a/ios/chrome/browser/ui/favicon/favicon_attributes_provider.mm b/ios/chrome/browser/ui/favicon/favicon_attributes_provider.mm
index 4ae86430c5849c166bace028ae6e14ea1a1de609..348611f6d939498d003c58b0dcd18450b46c16eb 100644
--- a/ios/chrome/browser/ui/favicon/favicon_attributes_provider.mm
+++ b/ios/chrome/browser/ui/favicon/favicon_attributes_provider.mm
@@ -67,10 +67,12 @@
skia::UIColorFromSkColor(result.fallback_icon_style->text_color);
NSString* monogram =
base::SysUTF16ToNSString(favicon::GetFallbackIconText(blockURL));
- attributes =
- [FaviconAttributes attributesWithMonogram:monogram
- textColor:textColor
- backgroundColor:backgroundColor];
+ attributes = [FaviconAttributes
+ attributesWithMonogram:monogram
+ textColor:textColor
+ backgroundColor:backgroundColor
+ defaultBackgroundColor:result.fallback_icon_style->
+ is_default_background_color];
}
DCHECK(attributes);
completion(attributes);

Powered by Google App Engine
This is Rietveld 408576698