Rust cargo doc for all features (#1995)

* run rustfix rustdoc in grandpa pallet benchmarks

* enable rustdoc for all features

* fix rialto parachain runtime compilation

bridges/bin/rialto-parachain/runtime/Cargo.toml

@@ -83,10 +83,12 @@ runtime-benchmarks = [
 	'sp-runtime/runtime-benchmarks',
 	'frame-benchmarking',
 	'frame-support/runtime-benchmarks',
-	'frame-system-benchmarking',
+	'frame-system-benchmarking/runtime-benchmarks',
 	'frame-system/runtime-benchmarks',
 	'pallet-balances/runtime-benchmarks',
 	'pallet-timestamp/runtime-benchmarks',
+	'pallet-xcm/runtime-benchmarks',
+	'xcm-builder/runtime-benchmarks',
 ]
 std = [
 	"bp-messages/std",

bridges/bin/rialto/node/Cargo.toml

@@ -45,5 +45,6 @@ frame-benchmarking-cli = { git = "https://github.com/paritytech/substrate", bran
 [features]
 default = []
 runtime-benchmarks = [
+	"polkadot-service/runtime-benchmarks",
 	"rialto-runtime/runtime-benchmarks",
 ]

bridges/modules/grandpa/src/benchmarking.rs

@@ -30,8 +30,8 @@
 //!
 //! Consider the following:
 //!
-//!   / [B'] <- [C']
-//! [A] <- [B] <- [C]
+//!   / B <- C'
+//! A <- B <- C
 //!
 //! The common ancestor of both forks is block A, so this is what GRANDPA will finalize. In order to
 //! verify this we will have vote ancestries of `[B, C, B', C']` and pre-commits `[C, C']`.