filter.h/query.h: use const vector instead of non-const vector with ThreadSafeMemoizer

Author: dconeybe

Firestore/core/src/core/filter.h

@@ -164,7 +164,7 @@ class Filter {
      * Memoized list of all field filters that can be found by
      * traversing the tree of filters contained in this composite filter.
      */
-    mutable util::ThreadSafeMemoizer<std::vector<FieldFilter>>
+    mutable util::ThreadSafeMemoizer<const std::vector<FieldFilter>>
         memoized_flattened_filters_;
   };
 

Firestore/core/src/core/query.cc

@@ -92,7 +92,7 @@ absl::optional<Operator> Query::FindOpInsideFilters(
   return absl::nullopt;
 }
 
-std::shared_ptr<std::vector<OrderBy>> Query::CalculateNormalizedOrderBys()
+std::shared_ptr<const std::vector<OrderBy>> Query::CalculateNormalizedOrderBys()
     const {
   // Any explicit order by fields should be added as is.
   auto result = std::make_shared<std::vector<OrderBy>>(explicit_order_bys_);

Firestore/core/src/core/query.h

@@ -299,14 +299,10 @@ class Query {
 
   Target ToTarget(const std::vector<OrderBy>& order_bys) const;
 
-  // For properties below, use a `std::shared_ptr<ThreadSafeMemoizer>` rather
-  // than using `ThreadSafeMemoizer` directly so that this class is copyable
-  // (`ThreadSafeMemoizer` is not copyable because of its `std::once_flag`
-  // member variable, which is not copyable).
-
   // The memoized list of sort orders.
-  std::shared_ptr<std::vector<OrderBy>> CalculateNormalizedOrderBys() const;
-  mutable util::ThreadSafeMemoizer<std::vector<OrderBy>>
+  std::shared_ptr<const std::vector<OrderBy>> CalculateNormalizedOrderBys()
+      const;
+  mutable util::ThreadSafeMemoizer<const std::vector<OrderBy>>
       memoized_normalized_order_bys_;
 
   // The corresponding Target of this Query instance.