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

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

Issue 2815453002: [Android] Show Browser Actions dialog in Chrome (Closed)
Patch Set: Update based on Ted's comments. Created 3 years, 8 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: 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 ed8b36eb4d3d71a6dd650787974d6f33bada7164..645bc79119ca1be61de42b4807458db46e5d65c3 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
@@ -22,7 +22,7 @@ import java.util.List;
public class PlatformContextMenuUi implements ContextMenuUi {
private ContextMenu mMenu;
- PlatformContextMenuUi(ContextMenu menu) {
+ public PlatformContextMenuUi(ContextMenu menu) {
mMenu = menu;
}
@@ -46,7 +46,7 @@ public class PlatformContextMenuUi implements ContextMenuUi {
List<ContextMenuItem> group = itemGroups.get(groupIndex).second;
for (int itemIndex = 0; itemIndex < group.size(); itemIndex++) {
ContextMenuItem item = group.get(itemIndex);
- MenuItem menuItem = mMenu.add(0, item.menuId, 0, item.getString(activity));
+ MenuItem menuItem = mMenu.add(0, item.getMenuId(), 0, item.getTitle(activity));
menuItem.setOnMenuItemClickListener(menuListener);
}
}

Powered by Google App Engine
This is Rietveld 408576698