[IndexedDB] Propogating changes to observers : Renderer

content/browser/indexed_db/indexed_db_observer.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_OBSERVER_H_
 #define CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_OBSERVER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
+#include <set>
+#include <vector>
+
 #include "base/macros.h"
+#include "base/stl_util.h"
 #include "content/common/content_export.h"
 
 namespace content {
 
 class CONTENT_EXPORT IndexedDBObserver {
  public:
-  IndexedDBObserver(int32_t observer_id);
+  IndexedDBObserver(int32_t observer_id, std::set<int64_t> object_store_ids);
   ~IndexedDBObserver();
 
-  int32_t id() const { return observer_id_; }
+  int32_t id() const { return id_; }
+  const std::set<int64_t>& GetObjectStoreIds() const {
+    return object_store_ids_;
+  }
+  bool IsRecordingType(int32_t type_index) const {
+    return operation_type_[type_index];

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

When is operation_type_ set?

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

Sorry, this is still a wip. I should have mentioned that.

+  }
+  bool IsRecordingObjectStore(int32_t object_store_id) const {
+    return ContainsValue(object_store_ids_, object_store_id);
+  }
 
  private:
-  int32_t observer_id_;
+  int32_t id_;
+  std::set<int64_t> object_store_ids_;
+  // operation_type_ indicates whether each operation type { ADD, PUT, DELETE,
+  // CLEAR } is to be recorded or not.
+  bool operation_type_[4] = {false};

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

using a int32_t (above in IsRecordingType) and [4] is not type safe, and
fragile. Consider something like:

  enum OperationType {
    ADD = 0,
    PUT,
    DELETE,
    CLEAR,
    NUM_TYPES,  // must be last enumeration
  };

and then:

  bool operation_type_[IndexedDBObservation::OperationType::NUM_TYPES] =
{false};

Then you can have:

  bool IsRecordingType(IndexedDBObservation::OperationType type) const {
    DCHECK_NE(type, IndexedDBObservation::OperationType::NUM_TYPES);
    return operation_type_[type];
  }

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

Thanks. Done.

 
   DISALLOW_COPY_AND_ASSIGN(IndexedDBObserver);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_INDEXED_DB_INDEXED_DB_OBSERVER_H_