[Offline Page]Making mhtml file recognized as openable by Chrome

chrome/android/java/AndroidManifest.xml

Index: chrome/android/java/AndroidManifest.xml
diff --git a/chrome/android/java/AndroidManifest.xml b/chrome/android/java/AndroidManifest.xml
index a0b5f74b5099ea94726c39cb37af198b11cd97c1..4c5b7b65cd32ac8104f1714cd96160038689f5ef 100644
--- a/chrome/android/java/AndroidManifest.xml
+++ b/chrome/android/java/AndroidManifest.xml
@@ -188,7 +188,30 @@ by a child template that "extends" this file.
             <intent-filter>
                 <action android:name="android.intent.action.VIEW" />
                 <category android:name="android.intent.category.DEFAULT" />

[Ted C, 2016/07/26 17:01:47]

seems odd that this is the one VIEW handler that lacks BROWSABLE...any idea why
that is?

[Vivian, 2016/07/26 17:12:59]

Not really sure about this...seems like this piece of code was there from the
very beginning.

-                <data android:scheme="file" android:mimeType="multipart/related"/>
+                <data android:mimeType="multipart/related"/>

[Ted C, 2016/07/26 17:01:47]

why did this drop the file restriction?  I see in the bug comment that this
handles certain cases, but what are the schemes that are used in those?

[Vivian, 2016/07/26 17:12:59]

Google Drive, Gmail and Android's Download use content scheme.

[Ted C, 2016/07/26 17:17:09]

Should we specify file || content then like the below filters?

[Vivian, 2016/07/26 17:21:37]

In Android's developer guide
(https://developer.android.com/guide/topics/manifest/data-element.html) it says:
"If the filter has a data type set (the mimeType attribute) but no scheme, the
content: and file: schemes are assumed."
But I guess specifying the schemes would make it less confusing.

[Ted C, 2016/07/26 17:24:02]

Ah, didn't know that.  Not sure whether it is worth specifying then.  It is nice
for clarity, but that is only because I'm not intimately familiar with
intent-filters.

[Vivian, 2016/07/26 17:41:31]

Uploaded a new patch, making the schemes explicit.

+            </intent-filter>
+            <intent-filter>
+                <action android:name="android.intent.action.VIEW" />
+                <category android:name="android.intent.category.DEFAULT" />
+                <category android:name="android.intent.category.BROWSABLE" />
+                <data android:scheme="file"/>
+                <data android:scheme="content"/>
+                <data android:host="*" />
+                <data android:pathPattern=".*\\.mhtml"/>
+                <data android:pathPattern=".*\\.mht"/>
+                <data android:pathPattern=".*\\.eml"/> 
+            </intent-filter>
+            <intent-filter>
+                <action android:name="android.intent.action.VIEW" />
+                <category android:name="android.intent.category.DEFAULT" />
+                <category android:name="android.intent.category.BROWSABLE" />
+                <data android:scheme="file"/>
+                <data android:scheme="content"/>
+                <data android:host="*" />
+                <data android:mimeType="*/*"/>
+                <data android:pathPattern=".*\\.mhtml"/>
+                <data android:pathPattern=".*\\.mht"/>
+                <data android:pathPattern=".*\\.eml"/> 
             </intent-filter>
             <intent-filter>
                 <action android:name="android.intent.action.MEDIA_SEARCH" />