Commit 99929f72 authored by Romain Courteaud's avatar Romain Courteaud

Protect the changeState calls with the 'changestate' mutex.

parent 0b307c54
...@@ -698,54 +698,37 @@ ...@@ -698,54 +698,37 @@
return this.element; return this.element;
}) })
.declareMethod('changeState', function changeState(state_dict) { .declareMethod('changeState', function changeState(state_dict) {
var next_onStateChange = new RSVP.Queue(), var context = this,
previous_onStateCHange, key,
context = this; modified = false,
if (context.hasOwnProperty('__previous_onStateChange')) { previous_cancelled = context.hasOwnProperty('__modification_dict'),
previous_onStateCHange = context.__previous_onStateChange; modification_dict;
next_onStateChange if (previous_cancelled) {
.push(function waitForPreviousStateChange() { modification_dict = context.__modification_dict;
return previous_onStateCHange; modified = true;
} else {
modification_dict = {};
}
for (key in state_dict) {
if (state_dict.hasOwnProperty(key) &&
(state_dict[key] !== context.state[key])) {
context.state[key] = state_dict[key];
modification_dict[key] = state_dict[key];
modified = true;
}
}
if (modified && context.__state_change_callback !== undefined) {
context.__modification_dict = modification_dict;
return new RSVP.Queue()
.push(function waitForStateChangeCallback() {
return context.__state_change_callback(modification_dict);
}) })
.push(undefined, function handlePreviousStateChangeError() { .push(function handleStateChangeSuccess(result) {
// Run callback even if previous failed delete context.__modification_dict;
return; return result;
}); });
} }
context.__previous_onStateChange = next_onStateChange; }, {mutex: 'changestate'});
return next_onStateChange
.push(function checkStateModification() {
var key,
modified = false,
previous_cancelled = context.hasOwnProperty('__modification_dict'),
modification_dict;
if (previous_cancelled) {
modification_dict = context.__modification_dict;
modified = true;
} else {
modification_dict = {};
}
for (key in state_dict) {
if (state_dict.hasOwnProperty(key) &&
(state_dict[key] !== context.state[key])) {
context.state[key] = state_dict[key];
modification_dict[key] = state_dict[key];
modified = true;
}
}
if (modified && context.__state_change_callback !== undefined) {
context.__modification_dict = modification_dict;
return new RSVP.Queue()
.push(function waitForStateChangeCallback() {
return context.__state_change_callback(modification_dict);
})
.push(function handleStateChangeSuccess(result) {
delete context.__modification_dict;
return result;
});
}
});
});
///////////////////////////////////////////////////////////////// /////////////////////////////////////////////////////////////////
// RenderJSGadget.declareAcquiredMethod // RenderJSGadget.declareAcquiredMethod
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment