Do not show the downloads page without the user explicitly requesting it.

Fixes http://b/issue?id=1671150

diff --git a/src/com/android/browser/BrowserActivity.java b/src/com/android/browser/BrowserActivity.java
index 0167a94..92e1968 100644 (file)
--- a/src/com/android/browser/BrowserActivity.java
+++ b/src/com/android/browser/BrowserActivity.java
@@ -2955,7 +2955,6 @@ public class BrowserActivity extends Activity
         } else {
             final Uri contentUri =
                     getContentResolver().insert(Downloads.Impl.CONTENT_URI, values);
-            viewDownloads(contentUri);
         }
 
     }
@@ -3529,10 +3528,10 @@ public class BrowserActivity extends Activity
 
     /*
      * This method is called as a result of the user selecting the options
-     * menu to see the download window, or when a download changes state. It
-     * shows the download window ontop of the current window.
+     * menu to see the download window. It shows the download window on top of
+     * the current window.
      */
-    /* package */ void viewDownloads(Uri downloadRecord) {
+    private void viewDownloads(Uri downloadRecord) {
         Intent intent = new Intent(this,
                 BrowserDownloadPage.class);
         intent.setData(downloadRecord);

diff --git a/src/com/android/browser/FetchUrlMimeType.java b/src/com/android/browser/FetchUrlMimeType.java
index 51f9873..9e34736 100644 (file)
--- a/src/com/android/browser/FetchUrlMimeType.java
+++ b/src/com/android/browser/FetchUrlMimeType.java
@@ -128,7 +128,6 @@ class FetchUrlMimeType extends AsyncTask<ContentValues, String, String> {
        // Start the download
        final Uri contentUri =
            mActivity.getContentResolver().insert(Downloads.Impl.CONTENT_URI, mValues);
-       mActivity.viewDownloads(contentUri);
     }
 
 }