GNOME Bugzilla – Bug 671134
Extension installation via website doesn't work
Last modified: 2012-03-16 20:07:30 UTC
See patches...
Created attachment 208789 [details] [review] ExtensionSystem: fix installing from website loadExtension() fails if the extension object is already created, but the installation dialog was creating a dummy object in the downloading state. Since nothing requires that (and the object is not in the correct format anyway), just kill it.
Created attachment 208790 [details] [review] ExtensionSystem: fix uninstalling from website uninstall function was still referring to the old state and error objects, which were removed by the refactoring.
Review of attachment 208790 [details] [review]: I swore I removed these...
Review of attachment 208789 [details] [review]: The reason that we need the dummy object is so that we can set the state back to UNINSTALLED if the user clicks "No" to the install dialog, if I remember correctly
Review of attachment 208789 [details] [review]: Wait, no, that's another code path. I'm not sure what this is here for, then. Kill it.
Attachment 208789 [details] pushed as 772638c - ExtensionSystem: fix installing from website Attachment 208790 [details] pushed as 64ce622 - ExtensionSystem: fix uninstalling from website
OK, I'm going to come out and say it right now: we really need unit tests. This never would have happened if we had them, so I'm probably going to make it a project sometime soon to just create a bunch of unit tests that try and test every code path.
*** Bug 672230 has been marked as a duplicate of this bug. ***