This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: Ignored Files: reviewboard/static/rb/js/views/tests/screenshotThumbnailViewTests.js reviewboard/static/rb/js/views/tests/fileAttachmentThumbnailViewTests.js reviewboard/static/rb/js/models/baseResourceModel.js
Add "destroying" and "destroyed" signals to BaseResourceModel.
Review Request #4115 — Created May 8, 2013 and submitted
Add "destroying" and "destroyed" signals to BaseResourceModel. "destroying" is triggered as soon as the destroy() call is called. "destroyed" is triggered after a successful destroy. Note that Backbone.js provides a "destroy" signal, but this doesn't follow the naming conventions we're otherwise using.
Unit tests pass. Used this in another change.