Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
| index cd508545aa7a47b1b2f4e3f3a30fdb72ef67c7fa..c2ddf51624140d22a10c738bf8c34b0192576852 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java |
| @@ -121,6 +121,9 @@ public class ContextMenuHelper implements OnCreateContextMenuListener { |
| shareImageDirectly(ShareHelper.getLastShareComponentName()); |
| } |
| }); |
| + ((TabularContextMenuUi) menuUi) |
| + .setContentViewCore( |
| + ContentViewCore.fromWebContents(contentViewCore.getWebContents())); |
|
Theresa
2017/05/23 01:58:07
ContentViewCore is going away, per the warning at
Daniel Park
2017/05/24 16:44:31
Done.
|
| menuUi.displayMenu(mActivity, mCurrentContextMenuParams, items, mCallback, mOnMenuShown, |
| mOnMenuClosed); |
| if (mCurrentContextMenuParams.isImage()) { |
| @@ -134,7 +137,7 @@ public class ContextMenuHelper implements OnCreateContextMenuListener { |
| return; |
| } |
| - // The Platform Context Menu requires the listener within this hepler since this helper and |
| + // The Platform Context Menu requires the listener within this helper since this helper and |
| // provides context menu for us to show. |
| view.setOnCreateContextMenuListener(this); |
| if (view.showContextMenu()) { |