From 0894e9d54e92aa5419f1020f8f2f76e625be5c36 Mon Sep 17 00:00:00 2001 From: Michael Comella Date: Mon, 13 Jul 2015 17:14:41 -0700 Subject: [PATCH] Bug 1182328 - Backed out changeset 2e02ac7f87ab. r=me Did not mean to land this WIP patch. --HG-- extra : commitid : 5eVOarG5Ath --- mobile/android/base/GeckoAppShell.java | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/mobile/android/base/GeckoAppShell.java b/mobile/android/base/GeckoAppShell.java index d1501af32711..1b03b551c110 100644 --- a/mobile/android/base/GeckoAppShell.java +++ b/mobile/android/base/GeckoAppShell.java @@ -1197,15 +1197,9 @@ public class GeckoAppShell final Intent intent = getOpenURIIntentInner(context, targetURI, mimeType, action, title); if (intent != null) { - // Ideally, we add Browsable on links from web content (bug 1182140) but we can't. Instead, - // we add Browsable to any Intent that has a scheme. This prevents us from sharing harmless - // content (e.g. plain text) and protects any potentially registered custom uri schemes. - // - // However, we also push file schemes through here and most applications that open files - // aren't expecting file schemes from web content so in order to be compatible, we drop - // Browsable from them (bug 1100100). - if (!TextUtils.isEmpty(intent.getScheme()) && - !targetURI.startsWith("file:")) { + // Setting category on file:// URIs breaks about:downloads (Bug 1176018) + if (!targetURI.startsWith("file:")) { + // Only handle applications which can accept arbitrary data from a browser. intent.addCategory(Intent.CATEGORY_BROWSABLE); }