IDBObserver: Lifetime Management: Adding Observer

content/child/indexed_db/indexed_db_dispatcher.cc

 // Copyright 2013 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.
 
 #include "content/child/indexed_db/indexed_db_dispatcher.h"
 
 #include <utility>
 
 #include "base/format_macros.h"
 #include "base/lazy_instance.h"
@@ skipping 12 matching lines @@
 
 using blink::WebBlobInfo;
 using blink::WebData;
 using blink::WebIDBCallbacks;
 using blink::WebIDBCursor;
 using blink::WebIDBDatabase;
 using blink::WebIDBDatabaseCallbacks;
 using blink::WebIDBDatabaseError;
 using blink::WebIDBKey;
 using blink::WebIDBMetadata;
+using blink::WebIDBObserver;
 using blink::WebIDBValue;
 using blink::WebString;
 using blink::WebVector;
 using base::ThreadLocalPointer;
 
 namespace content {
 static base::LazyInstance<ThreadLocalPointer<IndexedDBDispatcher> >::Leaky
     g_idb_dispatcher_tls = LAZY_INSTANCE_INITIALIZER;
 
 namespace {
@@ skipping 117 matching lines @@
   // If a message gets here, IndexedDBMessageFilter already determined that it
   // is an IndexedDB message.
   DCHECK(handled) << "Didn't handle a message defined at line "
                   << IPC_MESSAGE_ID_LINE(msg.type());
 }
 
 bool IndexedDBDispatcher::Send(IPC::Message* msg) {
   return thread_safe_sender_->Send(msg);
 }
 
+int32_t IndexedDBDispatcher::AddIDBObserver(int32_t ipc_database_id,
+                                            int64_t transaction_id,
+                                            WebIDBObserver* observer) {
+  // TODO(palakj): Do I wrap WebIDBObserver with unique_ptr as in

[dmurph, 2016/06/22 01:09:49]

unique_ptr

+  // WebIDBCallbacks.
+  int32_t observer_id = observers_.Add(observer);
+  Send(new IndexedDBHostMsg_DatabaseObserve(ipc_database_id, transaction_id,
+                                            observer_id));
+  return observer_id;
+}
+
+std::vector<int32_t> IndexedDBDispatcher::RemoveIDBObservers(
+    int32_t ipc_database_id,
+    WebIDBObserver* observer,
+    std::vector<int32_t> database_observers_id) {
+  std::vector<int32_t> observersToRemove;

[dmurph, 2016/06/22 01:09:49]

no camelCase

+  for (uint32_t i = 0; i < database_observers_id.size(); i++) {
+    WebIDBObserver* obs = observers_.Lookup(database_observers_id[i]);
+    if (obs == observer) {
+      observers_.Remove(database_observers_id[i]);
+      observersToRemove.push_back(database_observers_id[i]);
+    }
+  }
+
+  Send(new IndexedDBHostMsg_DatabaseUnobserve(ipc_database_id,
+                                              observersToRemove));
+  return observersToRemove;
+}
+
+void IndexedDBDispatcher::ClearIDBObserver(

[dmurph, 2016/06/22 01:09:49]

I don't think this method needs to unobserve on the backend, as the close logic
should handle that, right? Also, ClearIDBObservers (add s).

+    std::vector<int32_t> observersToRemove) {
+  for (uint32_t i = 0; i < observersToRemove.size(); i++) {
+    observers_.Remove(observersToRemove[i]);
+  }
+}
+
 void IndexedDBDispatcher::RequestIDBCursorAdvance(
     unsigned long count,
     WebIDBCallbacks* callbacks_ptr,
     int32_t ipc_cursor_id,
     int64_t transaction_id) {
   // Reset all cursor prefetch caches except for this cursor.
   ResetCursorPrefetchCaches(transaction_id, ipc_cursor_id);
 
   std::unique_ptr<WebIDBCallbacks> callbacks(callbacks_ptr);
 
@@ skipping 620 matching lines @@
   typedef std::map<int32_t, WebIDBCursorImpl*>::iterator Iterator;
   for (Iterator i = cursors_.begin(); i != cursors_.end(); ++i) {
     if (i->first == ipc_exception_cursor_id ||
         i->second->transaction_id() != transaction_id)
       continue;
     i->second->ResetPrefetchCache();
   }
 }
 
 }  // namespace content