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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/PlatformContextMenuUi.java

Issue 2778613002: Display LinkText/TitleText (bold) + LinkUrl/SrcUrl in ContextMenu title
Patch Set: Created 3 years, 9 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 | « chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuTitleView.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/PlatformContextMenuUi.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/PlatformContextMenuUi.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/PlatformContextMenuUi.java
index cfbaa87973dbca75852187beb8505ce7cd87f3cb..552cc8086a126d18640faa91d6101e6cb712164d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/PlatformContextMenuUi.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/PlatformContextMenuUi.java
@@ -6,6 +6,7 @@ package org.chromium.chrome.browser.contextmenu;
import android.app.Activity;
import android.content.Context;
+import android.text.SpannableStringBuilder;
import android.text.TextUtils;
import android.util.Pair;
import android.view.ContextMenu;
@@ -30,9 +31,8 @@ public class PlatformContextMenuUi implements ContextMenuUi {
public void displayMenu(Activity activity, ContextMenuParams params,
List<Pair<Integer, List<ContextMenuItem>>> itemGroups,
final Callback<Integer> listener) {
-
- String headerText = ChromeContextMenuPopulator.createHeaderText(params);
- if (TextUtils.isEmpty(headerText)) {
+ SpannableStringBuilder headerText = ChromeContextMenuPopulator.createHeaderText(params);
+ if (!TextUtils.isEmpty(headerText)) {
setHeaderText(activity, mMenu, headerText);
}
@@ -53,7 +53,7 @@ public class PlatformContextMenuUi implements ContextMenuUi {
}
}
- private void setHeaderText(Context context, ContextMenu menu, String text) {
+ private void setHeaderText(Context context, ContextMenu menu, SpannableStringBuilder text) {
ContextMenuTitleView title = new ContextMenuTitleView(context, text);
menu.setHeaderView(title);
}
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuTitleView.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698