595737 - Don't show file picker for "Save Page as PDF" on Android

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

We should just save PDFs to the default download folder, at least on Android.  (See also bug 593408, for "Save Image" and "Save Link.")

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: patch

This is mostly working - it hides the file picker on Android, and both code paths work fine on desktop.  The "download" never finishes on Android, though - possibly for some unrelated reason.

Comment 2

[Matt Brubeck (:mbrubeck)]

Comment on attachment 474728 [details] [diff] [review]
patch

This patch is good; filed followup bug 595919 for the new error on Android.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 474728 [details] [diff] [review]
patch

>diff -r cb86fc4a74bf chrome/content/browser-ui.js

>+    let file;
>+#ifdef ANDROID
>+    file = downloadsDir.clone();
>+    file.append(fileName);
>+#else

Let's just keep the "let" with the variable

#ifdef ANDROID
    let file = downloadsDir.clone();
    file.append(fileName);
#else


>+    file = picker.file;
>+#endif

and here:

    let file = picker.file;
#endif

Comment 4

[Matt Brubeck (:mbrubeck)]

(In reply to comment #3)
> Let's just keep the "let" with the variable

Fixed and pushed: http://hg.mozilla.org/mobile-browser/rev/cc98b92ce4e0

Comment 5

[Cristian Nicolae (:xti)]

PDFs are saved to sdcard/download as expected. Also there are no more issues during the downloading process on the latest Nightly build.

Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110913
Firefox/9.0a1 Fennec/9.0a1
Device: Samsung Galaxy S
OS: Android 2.2