Bug 1239503 - Don't include URL when sharing text. r=sebastian draft
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Mon, 18 Jan 2016 09:38:11 -0500
changeset 322388 cdfe0ec507dc4aca776461b5ef06f3c4c0193208
parent 322387 888e32f5c427e5840ca943dbb7ac6344aff59b38
child 513096 0761d3859dd976bf75280a0a08ef9ee66d566f9d
push id9599
push usermleibovic@mozilla.com
push dateMon, 18 Jan 2016 14:38:31 +0000
reviewerssebastian
bugs1239503
milestone46.0a1
Bug 1239503 - Don't include URL when sharing text. r=sebastian
mobile/android/base/java/org/mozilla/gecko/GeckoApp.java
--- a/mobile/android/base/java/org/mozilla/gecko/GeckoApp.java
+++ b/mobile/android/base/java/org/mozilla/gecko/GeckoApp.java
@@ -627,23 +627,21 @@ public abstract class GeckoApp
 
         } else if ("PrivateBrowsing:Data".equals(event)) {
             mPrivateBrowsingSession = message.optString("session", null);
 
         } else if ("Session:StatePurged".equals(event)) {
             onStatePurged();
 
         } else if ("Share:Text".equals(event)) {
-            String text = message.getString("text");
+            final String text = message.getString("text");
             final Tab tab = Tabs.getInstance().getSelectedTab();
             String title = "";
             if (tab != null) {
                 title = tab.getDisplayTitle();
-                final String url = ReaderModeUtils.stripAboutReaderUrl(tab.getURL());
-                text += "\n\n" + url;
             }
             GeckoAppShell.openUriExternal(text, "text/plain", "", "", Intent.ACTION_SEND, title, false);
 
             // Context: Sharing via chrome list (no explicit session is active)
             Telemetry.sendUIEvent(TelemetryContract.Event.SHARE, TelemetryContract.Method.LIST, "text");
 
         } else if ("Snackbar:Show".equals(event)) {
             SnackbarHelper.showSnackbar(this, message, callback);