Add InitStep to fix wrong primary key column order

For 2.9 and 2.10 we cannot perform this fix as a database
migration as we cannot increase the schema version. Therefore, do the
primary key fix in an InitStep for 2.9 and 2.10. In the master the
primary key fix will be done as a normal schema migration and the
InitStep introduced here will be deleted.

Change-Id: I4478aabcd63f907a1acfc8e23982bcc6f1c4346f
Don't use deprecated PGPPublicKeyRingCollection constructor

The PGPPublicKeyRingCollection constructor was deprecated in
Bouncycastle v1.51.

Use BcPGPPublicKeyRingCollection instead.

Change-Id: I5bc4fb4927ba37ecbf7d3763809acc2f53d7fd4e
(cherry picked from commit d26e43f22f2336efd18213ac88c285f8b95827cd)
Fix incorrect formatting in json documentation

Change-Id: Idaf47978ddd09c9f7a5708efb7d70afcfb8899f8
Add CSS style name on review comment div

Having style names on the elements makes it easy to apply custom
themes.

The DIV element containing the review comment was the only one
that did not have a name.

Bug: Issue 2998
Change-Id: I028c3bf4c4067ef31b10b364f85a456750876d56
Fix NullPointerException in Reindex

Reindex was loading the AccessControlModule which is loading the
GitUploadPackGroupsProvider and the GitReceivePackGroupsProvider. Those
2 classes need the CurrentUser in order to determine if he can see the
configured groups if those are not visible to all the users. Reindex
binds the CurrentUser to null which was causing the
NullPointerException if upload of receive groups were configured in
gerrit.config.

Instead of binding the CurrentUser to the InternalUser class which
would have fixed the issue, bind the GitUploadPackGroups and
GitReceivePackGroups to empty sets. The reason to use the second option
is because the providers of GitUploadPackGroups and GitReceivePackGroups
will require to load groups information from the GroupBackend which can
take time and is not required by the Reindex.

Issue: 3025
Change-Id: I78d7bb93195aefb82ebec421237147db2f7e4b3d
Merge "Add InitStep to fix wrong primary key column order" into stable-2.9
Merge branch 'stable-2.9' into stable-2.10

* stable-2.9:
  Add CSS style name on review comment div
  Fix incorrect formatting in json documentation
  Don't use deprecated PGPPublicKeyRingCollection constructor
  Add InitStep to fix wrong primary key column order

Change-Id: Id89fc4ec6684396ce13acb57f925fcff53f6b06e
Update 2.10 release notes with information from 2.9.2

Mention that 2.10 includes the fixes that were done in 2.9.2.

Add a warning about database table primary key update during
site init.

Change-Id: I1a517b338d223b922de8f2b45b2e37a0b2891d28