commit 9a293b0b3489673564e2a665fbd8e65c7531135a
author Axesh  R. Ajmera <aajmera@codeaurora.org> Tue Jun 10 14:03:55 2014 -0700
committer WebTech Code Review <code-review@localhost> Thu Jul 24 13:06:44 2014 -0700
tree 4201aad15a7c2136c13b5e28e21c25b171b00076
parent 3aae10191c95a41d285779672e309fe227305e72

Fix bug for content disposition

 - fixes the regex for parsing the content disposition header
   which helps in dowloading the appropiate filename

Change-Id: Idaed529ba7092b31ca2f35b39a42d76bed36d774

src/com/android/browser/DownloadHandler.java

diff --git a/src/com/android/browser/DownloadHandler.java b/src/com/android/browser/DownloadHandler.java
index ab24b76..0e41701 100644
--- a/src/com/android/browser/DownloadHandler.java
+++ b/src/com/android/browser/DownloadHandler.java
@@ -330,7 +330,7 @@
 
     static String trimContentDisposition(String contentDisposition) {
         final Pattern CONTENT_DISPOSITION_PATTERN =
-            Pattern.compile("attachment;\\s*filename\\s*=\\s*(\"?)([^\"]*)\\1\\s*;",
+            Pattern.compile("filename\\s*=\\s*(\"?)([^\"]*)\\1\\s*;?",
                 Pattern.CASE_INSENSITIVE);
 
         if (contentDisposition != null) {
@@ -338,10 +338,10 @@
             try {
                 Matcher m = CONTENT_DISPOSITION_PATTERN.matcher(contentDisposition);
                 if (m.find()) {
-                    return m.group();
-                } else {
-                    return contentDisposition;
+                    contentDisposition = "attachment; filename="+m.group(2);
                 }
+
+                return contentDisposition;
             } catch (IllegalStateException ex) {
                 // This function is defined as returning null when it can't parse the header
             }