Bug 1164114 - Open Intent URIs without a specific package set. r?margaret
authorMichael Comella <michael.l.comella@gmail.com>
Mon, 08 Jun 2015 16:17:35 -0700
changeset 248086 25f2050820ffe5ccd19828451a11f7d011ef584d
parent 248085 77a1a1eaad91560b18856ebe7da3ca5ae1dba134
child 248087 0e92a104e35142d42068043edc984d345ccd8c3f
push id60888
push userkwierso@gmail.com
push dateThu, 11 Jun 2015 01:38:38 +0000
treeherdermozilla-inbound@39e638ed06bf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmargaret
bugs1164114
milestone41.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1164114 - Open Intent URIs without a specific package set. r?margaret The Android Activity chooser dialog will appear.
mobile/android/base/GeckoAppShell.java
--- a/mobile/android/base/GeckoAppShell.java
+++ b/mobile/android/base/GeckoAppShell.java
@@ -1202,21 +1202,16 @@ public class GeckoAppShell
 
             // Only handle applications which can accept arbitrary data from a browser.
             intent.addCategory(Intent.CATEGORY_BROWSABLE);
 
             // Prevent site from explicitly opening our internal activities, which can leak data.
             intent.setComponent(null);
             nullIntentSelector(intent);
 
-            // We only handle explicit Intents at the moment (see bug 851693 comment 20).
-            if (intent.getPackage() == null) {
-                return null;
-            }
-
             return intent;
         }
 
         // Compute our most likely intent, then check to see if there are any
         // custom handlers that would apply.
         // Start with the original URI. If we end up modifying it, we'll
         // overwrite it.
         final String extension = MimeTypeMap.getFileExtensionFromUrl(targetURI);