Gozala/addon-sdk — comment on commit 303029030b9601df5a35867991f7cb0c3325c161
File:  line 0
By warner:

A few concerns:
* this changes the 'reason' sent into upload.js/send (and distributed to observers) from a number to a string. That's probably more likely to be useful to everyone, but it's a change, so I wanted to make sure it's intentional
* could this get stuck in a loop, whereby application shutdown calls bootstrap.js's `shutdown()`, which calls `loader.unload()`, which calls system.js `exit()`, which calls `appStartup.quit(E_FORCE)`, which tells the browser to shutdown, which calls `shutdown()`, etc?
* will sandboxed Loaders (as created by some unit tests) cause the whole browser to quit when they're unloaded? I think testing for `resultFile in options` might protect against it, but only as long as we remember to not copy the top-level `options` into the sub-loader.