[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..cdfb2f5b9b6fdd1c01aee439ef78665f9d831ae4 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 setId(int32_t id) { id_ = id; }

[cmumford, 2016/07/07 21:14:21]

Nit: setId/id are a pair, so I'd eliminate whitespace in between.

[palakj1, 2016/07/08 17:37:55]

Done.

+
+  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()
@@ -47,6 +50,7 @@ class CONTENT_EXPORT IndexedDBConnection {
 
  private:
   // NULL in some unit tests, and after the connection is closed.
+  int32_t id_;

[cmumford, 2016/07/07 21:14:21]

id_ got put in between the comment and database_, so something needs to move.

[palakj1, 2016/07/08 17:37:55]

Sorry for that. Changed.

   scoped_refptr<IndexedDBDatabase> database_;
 
   // The callbacks_ member is cleared when the connection is closed.