Last modified: 2014-11-18 19:37:33 UTC
From comments at https://gerrit.wikimedia.org/r/#/c/173296/1/src/ui/filedrophandlers/ve.ui.DSVFileDropHandler.js : * Right now every child class calls createProgress( insertableDataDeferred.promise() ); in its process() method. This seems like something that should be in the base class instead. Would you ever want to not do that, or pass a different promise? * Child classes should not have to override abort() just to add this.reader.abort();, because this.reader is added by the base class.