[IndexedDB] Propogating changes to observers : Renderer

content/browser/indexed_db/indexed_db_connection.h

Index: content/browser/indexed_db/indexed_db_connection.h
diff --git a/content/browser/indexed_db/indexed_db_connection.h b/content/browser/indexed_db/indexed_db_connection.h
index 3c305f39d057147c8a8feb12654a0f826404b09a..66ab65a4d8f8e2772d8382a5718280baf0865849 100644
--- a/content/browser/indexed_db/indexed_db_connection.h
+++ b/content/browser/indexed_db/indexed_db_connection.h
@@ -35,6 +35,9 @@ class CONTENT_EXPORT IndexedDBConnection {
   // connection.
   virtual void RemoveObservers(const std::vector<int32_t>& remove_observer_ids);
 
+  void set_id(int32_t id);
+  int32_t id() const { return id_; }
+
   IndexedDBDatabase* database() const { return database_.get(); }
   IndexedDBDatabaseCallbacks* callbacks() const { return callbacks_.get(); }
   const std::vector<std::unique_ptr<IndexedDBObserver>>& active_observers()
@@ -46,9 +49,12 @@ class CONTENT_EXPORT IndexedDBConnection {
   }
 
  private:
+  enum { kInvalidId = -1 };
+  // id_ is ipc_database_id
+  int32_t id_ = kInvalidId;

[palmer, 2016/07/16 00:03:14]

Style nit (ignore me if other people suggested to do it this way): I think we
normally set default values for fields in the constructor(s), rather than doing
it here.

I don't think it matters too much though.

[palakj1, 2016/07/18 22:02:29]

It was suggested by jsbell that I do it here, since it's constant always.

+
   // NULL in some unit tests, and after the connection is closed.
   scoped_refptr<IndexedDBDatabase> database_;
-
   // The callbacks_ member is cleared when the connection is closed.
   // May be NULL in unit tests.
   scoped_refptr<IndexedDBDatabaseCallbacks> callbacks_;