delegate to underlying impl.

Author: ehsannas

firestore/src/android/document_change_android.cc

@@ -55,5 +55,11 @@ std::size_t DocumentChangeInternal::new_index() const {
   return env.Call(obj_, kNewIndex);
 }
 
+bool operator==(const DocumentChangeInternal& lhs,
+                const DocumentChangeInternal& rhs) {
+  Env env = FirestoreInternal::GetEnv();
+  return Object::Equals(env, lhs.ToJava(), rhs.ToJava());
+}
+
 }  // namespace firestore
 }  // namespace firebase

firestore/src/android/document_change_android.h

@@ -24,6 +24,14 @@ class DocumentChangeInternal : public Wrapper {
   std::size_t new_index() const;
 };
 
+bool operator==(const DocumentChangeInternal& lhs,
+                const DocumentChangeInternal& rhs);
+
+inline bool operator!=(const DocumentChangeInternal& lhs,
+                       const DocumentChangeInternal& rhs) {
+  return !(lhs == rhs);
+}
+
 }  // namespace firestore
 }  // namespace firebase
 

firestore/src/common/document_change.cc

@@ -105,8 +105,11 @@ std::size_t DocumentChange::new_index() const {
 }
 
 bool operator==(const DocumentChange& lhs, const DocumentChange& rhs) {
-  return lhs.type() == rhs.type() && lhs.old_index() == rhs.old_index() &&
-         lhs.new_index() == rhs.new_index() && lhs.document() == rhs.document();
+  if (!lhs.internal_ || !rhs.internal_) {
+    return lhs.internal_ == rhs.internal_;
+  }
+
+  return lhs.internal_ == rhs.internal_ || *lhs.internal_ == *rhs.internal_;
 }
 
 }  // namespace firestore

firestore/src/include/firebase/firestore/document_change.h

@@ -173,6 +173,8 @@ class DocumentChange {
   bool is_valid() const { return internal_ != nullptr; }
 
  private:
+  friend bool operator==(const DocumentChange& lhs, const DocumentChange& rhs);
+
   friend class FirestoreInternal;
   friend class Wrapper;
   friend struct ConverterImpl;

firestore/src/main/document_change_main.cc

@@ -46,5 +46,10 @@ std::size_t DocumentChangeInternal::new_index() const {
   return change_.new_index();
 }
 
+bool operator==(const DocumentChangeInternal& lhs,
+                const DocumentChangeInternal& rhs) {
+  return lhs.change_ == rhs.change_;
+}
+
 }  // namespace firestore
 }  // namespace firebase

firestore/src/main/document_change_main.h

@@ -27,10 +27,18 @@ class DocumentChangeInternal {
   std::size_t old_index() const;
   std::size_t new_index() const;
 
+  friend bool operator==(const DocumentChangeInternal& lhs,
+                         const DocumentChangeInternal& rhs);
+
  private:
   api::DocumentChange change_;
 };
 
+inline bool operator!=(const DocumentChangeInternal& lhs,
+                       const DocumentChangeInternal& rhs) {
+  return !(lhs == rhs);
+}
+
 }  // namespace firestore
 }  // namespace firebase