Closed
Bug 793276
Opened 8 years ago
Closed 8 years ago
new android updater downloads on cell data connection without asking
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox17+ fixed, firefox18+ verified)
VERIFIED
FIXED
Firefox 18
People
(Reporter: cdlrobinson, Assigned: snorp)
References
Details
(Keywords: dogfood)
Attachments
(1 file)
1.53 KB,
patch
|
mfinkle
:
review+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:15.0) Gecko/20100101 Firefox/15.0.1 Build ID: 20120905151427 Steps to reproduce: I dismissed the notification that a download was available since I was not on WiFi and have a limited data plan. Actual results: later when I checked my phone there was a notification to install the downloaded update. Expected results: The download should not happen over a cell data connection unless I select to download it. I would prefer a setting to not even offer the download unless the phone is on WiFi.
Updated•8 years ago
|
Updated•8 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 1•8 years ago
|
||
Since there's apparently no way to disable the new update service, I had to uninstall Nightly :-(
Severity: normal → blocker
Keywords: dogfood
Updated•8 years ago
|
tracking-firefox17:
--- → ?
tracking-firefox18:
--- → ?
Comment 2•8 years ago
|
||
Had this happen to me this morning too as soon as I turned my device on and switched to cellular, the download initiated.
Comment 3•8 years ago
|
||
Tracking for release, but we'll untrack for versions as the move to beta (which doesn't use the updater).
Comment 5•8 years ago
|
||
From bug 795705: "I have 300 MB monthly data limit. Right now, in the "middle" of the month I used 220 MB out of 300, and Firefox used 136 MB alone, that is more than all other apps combined."
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #666547 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 7•8 years ago
|
||
The "force download" flag that we include in the intent that is used when you accept an update over mobile data was accidentally kept later on when we did a scheduled update check. Attached patch fixes that. Stupid API.
Comment 8•8 years ago
|
||
Comment on attachment 666547 [details] [diff] [review] Make sure we do not force an automatic download for Android updates Yeah, not the most straight forward API
Attachment #666547 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 9•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8c035b44ef28
Comment 10•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8c035b44ef28 Should this have a test?
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox18:
--- → fixed
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
Assignee | ||
Comment 11•8 years ago
|
||
(In reply to Ryan VanderMeulen from comment #10) > https://hg.mozilla.org/mozilla-central/rev/8c035b44ef28 > > Should this have a test? No way currently to have an automated test for this, no. You should be able to reproduce the bug prior to this fix, though, with the following STR: 1) Disable wifi, ensure you have a mobile data connection 2) Go to about:firefox, click 'check for updates' (with an old enough nightly obviously) 3) When prompted, start the update download 4) Swipe to cancel the 'apply update' notification, or cancel the installation after clicking on the notification. 5) Check for updates again, and the download would start automatically.
Comment 12•8 years ago
|
||
Updates are not downloaded automatically on 3G on the latest Nightly. Each time when STR from comment #11 are performed, the update notification is triggered which it's expected. Closing bug as verified fixed on: Firefox 18.0a1 (2012-10-08) Device: Galaxy Nexus OS: Android 4.1.1
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
status-firefox17:
--- → affected
Comment 13•8 years ago
|
||
Looks like we still need a beta uplift nom here if we want this to go out with 17.
Assignee | ||
Comment 14•8 years ago
|
||
Comment on attachment 666547 [details] [diff] [review] Make sure we do not force an automatic download for Android updates [Approval Request Comment] Low-risk, fixes annoying user-facing update bug
Attachment #666547 -
Flags: approval-mozilla-beta?
Comment 15•8 years ago
|
||
Comment on attachment 666547 [details] [diff] [review] Make sure we do not force an automatic download for Android updates Definitely worth uplifting this fix to reduce an annoying user-facing issue.
Attachment #666547 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 16•8 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/518ee633eb9c
Updated•8 years ago
|
Depends on: 808595
Updated•29 days ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•