Merge promise code into trunk

Merge promise code into trunk

This merges the promise code into the trunk and
adds a promise store in the model controller to 
combat a race condition with the multiple dispatch

https://code.launchpad.net/~hatch/juju-gui/promise-conflict/+merge/162414

(do not edit description out of merge proposal)

[jeff.pihach, 2013-05-03 16:21:37]

Please take a look.

[bcsaller, 2013-05-03 16:54:00]

LGTM

Seems to hide the race, still too aggressive with the notifications. Add a card
to put a timer to debounce that or something.

As a matter of process couldn't you have re-proposed the same branch rather than
having this come in as a new MP and losing the previous comments? Makes it
harder to verify.

https://codereview.appspot.com/9178043/diff/1/app/app.js
File app/app.js (right):

https://codereview.appspot.com/9178043/diff/1/app/app.js#newcode610
app/app.js:610: });
In testing with improv and the large sample I still see 3 notifications, but
haven't gotten a loading screen.

https://codereview.appspot.com/9178043/diff/1/app/models/model-controller.js
File app/models/model-controller.js (right):

https://codereview.appspot.com/9178043/diff/1/app/models/model-controller.js#...
app/models/model-controller.js:43: promise.then(cleanUp, cleanUp);
Promise cleanups are nice :)

https://codereview.appspot.com/9178043/diff/1/app/models/model-controller.js#...
app/models/model-controller.js:71: //db.fire('update');
remove comment

[jeff.pihach, 2013-05-03 17:00:23]

Thanks for the review - a card will be created for the notifications

https://codereview.appspot.com/9178043/diff/1/app/models/model-controller.js
File app/models/model-controller.js (right):

https://codereview.appspot.com/9178043/diff/1/app/models/model-controller.js#...
app/models/model-controller.js:71: //db.fire('update');
On 2013/05/03 16:54:00, bcsaller wrote:
> remove comment

Done.

[jeff.pihach, 2013-05-03 17:06:34]

*** Submitted:

Merge promise code into trunk

This merges the promise code into the trunk and
adds a promise store in the model controller to 
combat a race condition with the multiple dispatch

R=bcsaller
CC=
https://codereview.appspot.com/9178043