[Fixes #490, #488] restore run-time cycle support to the loader.
[Fixes #490, #488] restore run-time cycle support to the loader.

This does so, while continue to support good error handling semantics.
If the module failed to load, previously it would not fail to load the second time, merely become the empty SEEN object. This results in very crappy debugging experiences.

Also, wrapping the experience in a try/catch would cause different but equally as terrible experiences.

So this solution utilizes a try/finally to correctly set and track state of modules.
Please note, that if an error occurs while loading a module in IE8, it will fail due to no "catch" being present in the try/finally block. But this trade-off seems fair.

grese/miner-app-OLD,ghempton/ember-libs-example,lucaspottersky/ember-lab-eak,achambers/appkit-hanging-test-example,mixonic/ember-app-kit,mtian/ember-app-kit-azure,mtian/ember-app-kit-azure,kiwiupover/ember-weather,tomclose/minimal_eak_test_stub_problem,tomclose/test_loading_problem,shin1ohno/want-js,tobobo/vice-frontend,shin1ohno/EmberAppKit,digitalplaywright/eak-simple-auth-blog-client,tobobo/thyme-frontend,lucaspottersky/ember-lab-eak,michaelorionmcmanus/eak-bootstrap,stefanpenner/ember-app-kit,stefanpenner/ember-app-kit,sajt/ember-weather,Anshdesire/ember-app-kit,sajt/ember-weather,tobobo/thyme-frontend,digitalplaywright/eak-simple-auth,kiwiupover/ember-weather,aboveproperty/ember-app-kit-karma,grese/miner-app-OLD,Anshdesire/ember-app-kit