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
Merge branch 'stable-2.10'

* stable-2.10:
  Update 2.10 release notes with information from 2.9.2
  Fix NullPointerException in Reindex
  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

This reverts commit 12b44cb3904ee982c47a8cfbf3499f0f174c67b6
(Add InitStep to fix wrong primary key column order) which is not
needed on master.

This reworks commit db88d1539931ba28b8a9f7e44274b9efeb2ac51f
(Fix NullPointerException in Reindex) due to injection setup for
the reindex being changed between stable-2.10 and master.

Conflicts:
	gerrit-pgm/src/main/java/com/google/gerrit/pgm/Reindex.java

Change-Id: I2061bd2265a7117f1a978c68ac692fe2f09b1884
Releases: Update website to use Cloud Storage v1 API

Requests to the v1beta2 API stopped working recently, and the v1 API
has been released. Nothing in the migration notes[1] seems to apply to
us, and the page renders with no JS errors.

[1] https://cloud.google.com/storage/docs/json_api/v1/how-tos/migrate

Change-Id: I59357487f20f8abf3398531eb34a06aa765c6be9
Document theme attribute in diff preferences info entity

Ic97c237d7 added support for CodeMirror themes in SBS2, but missed to
document theme attribute in diff preferences info entity.

Change-Id: I7cd5ea519b8b8491138192053bfe7be09ba741d2
Rename __standalone_mode__ to STANDALONE_MODE

The fix for [1] prevents from importing python variables __foo__,
as explained in [2]. Rename standalone mode flag correspondingly.

[1] https://github.com/facebook/buck/issues/182
[2] https://github.com/facebook/buck/issues/221

Change-Id: If8682278e7c91df3fc18b3d623eb43950e839448
Fix broken formatting in rest-api-projects.txt

asciidoctor seems to need table closing lines to be the same
length as the opening lines.

Change-Id: Ib324f0513054041dcca6f8fd5d64f5e77b3014ae
Add missing "JSON Entities" section title in rest-api-access.txt

Change-Id: I3f66431c5003b6f4a44869743e7b4a5300985c96
Don't limit tables to 50% width in REST API documentation

Fixing the width of the tables to 50% makes it difficult to read
entries that have long descriptions.

Change-Id: Ic080c684bc5cf009d47149b2839390c66f3e9af0