Catch various SCMErrors in the ValidateDiffResource
Catch various SCMErrors in the ValidateDiffResource

The diff validation was catching a variety of errors, but the scmtools errors
weren't part of that. This wasn't an issue when I first wrote it but subsequent
clean-up has changed that. I've added handling for the ShortSHA1Error specifically
(to make it match what the front-end code expects), and a generic catch-all for
SCMError.

Testing done: Ran unit tests.
Reviewed at https://reviews.reviewboard.org/r/4741/

brennie/reviewboard,custode/reviewboard,custode/reviewboard,chipx86/reviewboard,custode/reviewboard,beol/reviewboard,chipx86/reviewboard,1tush/reviewboard,reviewboard/reviewboard,1tush/reviewboard,KnowNo/reviewboard,brennie/reviewboard,beol/reviewboard,1tush/reviewboard,bkochendorfer/reviewboard,KnowNo/reviewboard,bkochendorfer/reviewboard,KnowNo/reviewboard,1tush/reviewboard,1tush/reviewboard,sgallagher/reviewboard,custode/reviewboard,1tush/reviewboard,brennie/reviewboard,davidt/reviewboard,reviewboard/reviewboard,sgallagher/reviewboard,bkochendorfer/reviewboard,1tush/reviewboard,davidt/reviewboard,reviewboard/reviewboard,chipx86/reviewboard,davidt/reviewboard,reviewboard/reviewboard,beol/reviewboard,1tush/reviewboard,1tush/reviewboard,beol/reviewboard,davidt/reviewboard,brennie/reviewboard,KnowNo/reviewboard,sgallagher/reviewboard,sgallagher/reviewboard,bkochendorfer/reviewboard,chipx86/reviewboard