Add a confirmation when discarding a pending review from the review dialog.

When a user has a draft review, and opens up the "Edit Review" box to finalize
their comments and publish, the "Discard Review" command did not confirm at
all. Now it gives a prompt to discard or cancel.

Testing Done:
Tested visually, by publishing a review; both pressing and not pressing 'ok'
under the comment.

Case 1: Clicking Discard Review, then clicking cancel. It takes away the prompt.
Case 2: Clicking Discard Review, then clicking discard. Discards the review and
goes back to the review request.

Bugs closed: 3300

Reviewed at https://reviews.reviewboard.org/r/6341/
Fix discarding of comments in the diff viewer when double-clicking flags.

When double-clicking on a line number to add a comment in the diff view, this
is registered as two seperate clicks. The comment dialog opens on the first
click, and then on the second click the dialog is closed, the comment is
deleted, and then the dialog is reopened (but no new comment is created). So,
clicking save would not save the comment.

To solve this:
Before the comment dialog is shown, first check to see if there is already a
comment dialog active. If there is, and it is a comment dialog for the same
block as the one that is about to be shown, then don't show a new comment
dialog (keep the old one as it displays the same comment). A double-click is
registered as two clicks for the same comment block, and so the first click
will show the dialog, and the second will do nothing.

Testing Done:

Tried all of the following (in Firefox, Chrome):
 - Adding comment by double-clicking on a line number and comment flag
 - Adding comment by single-clicking on a line number and comment flag
 - Adding comment by selecting a range of lines

All JavaScript unit tests passed.

Bugs closed: 3437
Reviewed at https://reviews.reviewboard.org/r/6340/
Added instructions for running RBTools unittests to Getting Started guide.

Although this is a ReviewBoard documentation, it would be helpful to have the
instructions for running RBTools tests in the Getting Started guide.

Reviewed at https://reviews.reviewboard.org/r/6347/
Merge branch 'release-2.0.x'
Refactor revision selectors for future file attachment support.

This change refactors the DiffRevisionSelectorView control to abstract out
common functionality into a new RevisionSelectorView abstract base class. This
is then inherited by the existing DiffRevisionSelectorView and a new
FileAttachmentRevisionSelectorView.

Testing done:
Tested the various behavior in the DiffRevisionSelectorView.
FileAttachmentRevisionSelectorView is currently un-tested.

Reviewed at https://reviews.reviewboard.org/r/6342/