MB-45459: Remove unused param from VBucket::accountItem()

With the previous change for this MB the parameter is no longer
required.

Change-Id: I7fc85a4d764bede47007d54ec16844a70a2146c2
Reviewed-on: http://review.couchbase.org/c/kv_engine/+/151034
Tested-by: Build Bot <[email protected]>
Reviewed-by: Trond Norbye <[email protected]>

Author: BenHuddleston

engines/ep/src/vbucket.cc

@@ -696,18 +696,13 @@ void VBucket::doStatsForQueueing(const Item& qi, size_t itemBytes)
     dirtyQueuePendingWrites.fetch_add(itemBytes);
 }
 
-void VBucket::AggregatedFlushStats::accountItem(
-        const Item& item,
-        std::optional<std::chrono::steady_clock::time_point> queueTime) {
+void VBucket::AggregatedFlushStats::accountItem(const Item& item) {
     Expects(item.getQueuedTime().time_since_epoch().count() != 0);
-    if (!queueTime) {
-        queueTime = item.getQueuedTime();
-    }
 
     ++numItems;
     totalBytes += item.size();
     totalAgeInMicro += std::chrono::duration_cast<std::chrono::microseconds>(
-                               queueTime->time_since_epoch())
+                               item.getQueuedTime().time_since_epoch())
                                .count();
 }
 

engines/ep/src/vbucket.h

@@ -477,12 +477,8 @@ class VBucket : public std::enable_shared_from_this<VBucket> {
         /**
          * Add the stored values from the Item
          * @param Item being accounted
-         * @param queueTime optional queueTime. If not supplied, takes the
-         *                  queueTime of the Item.
          */
-        void accountItem(const Item& item,
-                         std::optional<std::chrono::steady_clock::time_point>
-                                 queueTime = {});
+        void accountItem(const Item& item);
 
         size_t getNumItems() const {
             return numItems;