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

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

Issue 2855743002: [Android] Fix context menu direct share bugs (Closed)
Patch Set: Update based on Ted's new comments. Created 3 years, 7 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 | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/share/ShareHelper.java » ('j') | 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/TabularContextMenuListAdapter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuListAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuListAdapter.java
index 9b6a9947d84b37e32766e2b366e4c8f45bb78ed9..cccbaaec6499d0f4f16e4088d4f647a326f0fbc6 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuListAdapter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuListAdapter.java
@@ -5,6 +5,7 @@
package org.chromium.chrome.browser.contextmenu;
import android.app.Activity;
+import android.content.Intent;
import android.graphics.drawable.Drawable;
import android.util.Pair;
import android.view.LayoutInflater;
@@ -81,8 +82,9 @@ class TabularContextMenuListAdapter extends BaseAdapter {
viewHolder.mIcon.setVisibility(icon != null ? View.VISIBLE : View.INVISIBLE);
if (menuItem == ContextMenuItem.SHARE_IMAGE) {
+ Intent shareIntent = ShareHelper.getShareImageIntent(null);
final Pair<Drawable, CharSequence> shareInfo =
- ShareHelper.getShareableIconAndName(mActivity);
+ ShareHelper.getShareableIconAndName(mActivity, shareIntent);
if (shareInfo.first != null) {
viewHolder.mShareIcon.setImageDrawable(shareInfo.first);
viewHolder.mShareIcon.setVisibility(View.VISIBLE);
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/share/ShareHelper.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698