Closed
Bug 769545
Opened 12 years ago
Closed 12 years ago
Reinstalling an app previously installed that preloads the appcache uninstalls the app
Categories
(Firefox Graveyard :: Web Apps, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 16
People
(Reporter: jsmith, Assigned: Felipe)
References
Details
(Keywords: dataloss, Whiteboard: [qa!])
Attachments
(1 file)
865 bytes,
patch
|
marco
:
review+
|
Details | Diff | Splinter Review |
Steps: 1. Install the app with appcache here - http://people.mozilla.com/~fdesre/openwebapps/test.html 2. Cut your internet connection 3. Reinstall the app with appcache here - http://people.mozilla.com/~fdesre/openwebapps/test.html Expected: Without a network connection, I'd expect app installation to fail. Actual: The app ends up being uninstalled from the user's machine.
Reporter | ||
Comment 1•12 years ago
|
||
Note - this does not happen with an app that is installed from the same hosted origin that does not use appcache preloading.
Reporter | ||
Comment 2•12 years ago
|
||
Change that - this happens also when an internet connection is present.
Summary: Reinstalling an app with no internet connection previously installed that preloads the appcache uninstalls the app → Reinstalling an app previously installed that preloads the appcache uninstalls the app
Reporter | ||
Updated•12 years ago
|
blocking-kilimanjaro: --- → ?
Reporter | ||
Comment 3•12 years ago
|
||
k9o nomination - We're heavily advocating app cache with web apps, especially with app cache preloading. Having incorrect reinstall behavior isn't a good thing.
Comment 4•12 years ago
|
||
When you say it uninstalls the app, does it remove it from the DOM registry, or just clears the offline cache?
Reporter | ||
Comment 5•12 years ago
|
||
(In reply to Fabrice Desré [:fabrice] from comment #4) > When you say it uninstalls the app, does it remove it from the DOM registry, > or just clears the offline cache? Uninstall app in this case implies: - Uninstalling the app natively from the machine (e.g. equilvalent to removing through add or remove programs) I'm currently digging into if this affects the DOMRegistry as well (e.g. removing it from there on reinstall).
Reporter | ||
Updated•12 years ago
|
Priority: -- → P2
Comment 6•12 years ago
|
||
Jason, does it remove the profile directory?
Reporter | ||
Comment 7•12 years ago
|
||
(In reply to Marco Castelluccio from comment #6) > Jason, does it remove the profile directory? Yup. It removes the entire application folder actually, including the profile folder. Marking for re-triage, as we now know this involves data loss, including the app profile.
Keywords: dataloss
Priority: P2 → --
Comment 8•12 years ago
|
||
In the installer, the only case in which we remove the profile directory is when there is an error during installation. Do you see anything in the error console?
Reporter | ||
Comment 9•12 years ago
|
||
(In reply to Marco Castelluccio from comment #8) > In the installer, the only case in which we remove the profile directory is > when there is an error during installation. Do you see anything in the error > console? Indeed I do! Here's the error I got: Timestamp: 7/3/2012 5:01:56 PM Error: Error installing app: ReferenceError: Cr is not defined Source File: resource:///modules/WebappsInstaller.jsm Line: 36
Assignee | ||
Comment 10•12 years ago
|
||
Thanks Jason and Marco for figuring out the problem
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Attachment #638928 -
Flags: review?(mar.castelluccio)
Assignee | ||
Comment 11•12 years ago
|
||
try build to show up here: http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/felipc@gmail.com-cbf852638a63 (the build also includes the fix for bug 769955)
Reporter | ||
Comment 12•12 years ago
|
||
(In reply to Felipe Gomes (:felipe) from comment #11) > try build to show up here: > http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/felipc@gmail.com- > cbf852638a63 > > (the build also includes the fix for bug 769955) Will this patch fix bug 770321 also? I know this error in the error console also reproduces on Mac as well.
Updated•12 years ago
|
Attachment #638928 -
Flags: review?(mar.castelluccio) → review+
Comment 13•12 years ago
|
||
(In reply to Jason Smith [:jsmith] from comment #12) > Will this patch fix bug 770321 also? I know this error in the error console > also reproduces on Mac as well. Most probably it will fix bug 770321. And it could fix also bug 753660.
Assignee | ||
Comment 14•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/66a21d10cc58
Target Milestone: --- → Firefox 16
Reporter | ||
Updated•12 years ago
|
QA Contact: jsmith
Whiteboard: [qa+]
Comment 16•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/66a21d10cc58
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 17•12 years ago
|
||
Verified on Nightly.
Status: RESOLVED → VERIFIED
blocking-kilimanjaro: ? → ---
Whiteboard: [qa+] → [qa!]
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•