Commit f62950bf authored by Gang Wu's avatar Gang Wu Committed by Commit Bot

Clipboard Text Suggestion should never use favicon

Bug: 1020609
Change-Id: I84fb6c13ca3ab0d3002853b17772775a7ff3eb54
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895968Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
Commit-Queue: Gang Wu <gangwu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#711885}
parent 561360a6
......@@ -214,7 +214,9 @@ public class BasicSuggestionProcessor implements SuggestionProcessor {
model.set(SuggestionViewProperties.TEXT_LINE_2_MAX_LINES, 1);
// Include site favicon if we are presenting URL and have favicon available.
if (mLargeIconBridge != null && suggestion.getUrl() != null) {
// TODO(gangwu): Create a saparate processor for clipboard suggestions.
if (mLargeIconBridge != null && suggestion.getUrl() != null
&& suggestion.getType() != OmniboxSuggestionType.CLIPBOARD_TEXT) {
mLargeIconBridge.getLargeIconForUrl(suggestion.getUrl(), mDesiredFaviconWidthPx,
(Bitmap icon, int fallbackColor, boolean isFallbackColorDefault,
int iconType) -> {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment