Fixes from code review

Author: tom-andersen

firestore/src/android/query_android.h

@@ -71,6 +71,12 @@ class QueryInternal : public Wrapper {
    */
   virtual AggregateQuery Count() const;
 
+  /**
+   * @brief Creates and returns a new Query with the additional filter.
+   *
+   * @param filter The new filter to apply to the existing query.
+   * @return The created Query.
+   */
   Query Where(const Filter& filter) const;
 
   /**

firestore/src/android/wrapper.cc

@@ -60,7 +60,7 @@ Wrapper::Wrapper(Wrapper* rhs) : Wrapper() {
 
 Wrapper::~Wrapper() = default;
 
-jni::Env Wrapper::GetEnv() { return FirestoreInternal::GetEnv(); }
+jni::Env Wrapper::GetEnv() const { return firestore_->GetEnv(); }
 
 Local<Object> Wrapper::ToJava() const {
   Env env = GetEnv();

firestore/src/android/wrapper.h

@@ -55,7 +55,7 @@ class Wrapper {
   // Similar to a copy constructor, but can handle the case where `rhs` is null.
   explicit Wrapper(Wrapper* rhs);
 
-  static jni::Env GetEnv();
+  jni::Env GetEnv() const;
 
   FirestoreInternal* firestore_ = nullptr;  // not owning
   jni::ArenaRef obj_;