docs: Name the field, not the getter, where the value is meant

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: mraszyk

rs/canonical_state/src/encoding/types.rs

@@ -738,7 +738,7 @@ impl
         // by deleted canisters and does not account for non-deleted canisters.
         //
         // Starting with `V29`, the reported total is the stored
-        // `SubnetMetrics::consumed_cycles_total_including_canisters()`, which no
+        // `SubnetMetrics::consumed_cycles_total_including_canisters`, which no
         // longer double counts deleted canisters and does account for the existing
         // ones.
         let consumed_cycles_total = if certification_version >= CertificationVersion::V29 {

rs/replicated_state/src/metadata_state.rs

@@ -587,7 +587,7 @@ impl SubnetMetrics {
     /// This is the current computation, which avoids double counting the cycles
     /// consumed by deleted canisters, as the legacy
     /// [`Self::consumed_cycles_total_v28`] does. It is one of the two summands of
-    /// [`Self::consumed_cycles_total_including_canisters()`], which is what the
+    /// [`Self::consumed_cycles_total_including_canisters`], which is what the
     /// canonical state consumer reports from certification version `V29` on.
     pub fn consumed_cycles_total(&self) -> NominalCycles {
         let mut total = NominalCycles::zero();
@@ -677,7 +677,7 @@ impl SubnetMetrics {
     /// `consumed_cycles_by_use_case` map, and both are summed here. It is kept
     /// unchanged to preserve the certified state for certification versions up
     /// to and including `V28`; from `V29` on the consumer reports
-    /// [`Self::consumed_cycles_total_including_canisters()`], which does not
+    /// [`Self::consumed_cycles_total_including_canisters`], which does not
     /// double count.
     pub fn consumed_cycles_total_v28(&self) -> NominalCycles {
         let mut total = NominalCycles::zero();

rs/replicated_state/src/metadata_state/tests.rs

@@ -2786,7 +2786,7 @@ fn consumed_cycles_total_calculates_the_right_amount() {
 
 /// The `replicated_state_consumed_cycles_since_replica_started` gauge is set in
 /// `ReplicatedStateMetrics::observe` from
-/// [`SubnetMetrics::consumed_cycles_total_including_canisters()`]. This test
+/// [`SubnetMetrics::consumed_cycles_total_including_canisters`]. This test
 /// exercises every subnet-level use case that contributes to the total, so that
 /// omitting any of them (as the `SchnorrOutcalls`/`VetKd`/`DroppedMessages` use
 /// cases once were) would change the reported value and fail the assertion, plus