Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1533)

Unified Diff: content/child/indexed_db/webidbdatabase_impl.cc

Issue 2062203004: IDBObserver: Lifetime Management: Adding Observer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Unobserve functionality Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/child/indexed_db/webidbdatabase_impl.cc
diff --git a/content/child/indexed_db/webidbdatabase_impl.cc b/content/child/indexed_db/webidbdatabase_impl.cc
index 28408b7c32facfd6d381f350e2f39ef57d4cb5a9..c9e7414677b4a3407ca7c2ca89d2695e212f6adb 100644
--- a/content/child/indexed_db/webidbdatabase_impl.cc
+++ b/content/child/indexed_db/webidbdatabase_impl.cc
@@ -106,7 +106,17 @@ void WebIDBDatabaseImpl::observe(blink::IDBObserver* observer,
long long transaction_id) {
IndexedDBDispatcher* dispatcher =
IndexedDBDispatcher::ThreadSpecificInstance(thread_safe_sender_.get());
- dispatcher->AddIDBObserver(ipc_database_id_, transaction_id, observer);
+ int32_t observer_id =
+ dispatcher->AddIDBObserver(ipc_database_id_, transaction_id, observer);
+ observers_id_.push_back(observer_id);
+}
+
+void WebIDBDatabaseImpl::unobserve(blink::IDBObserver* observer) {
+ IndexedDBDispatcher* dispatcher =
+ IndexedDBDispatcher::ThreadSpecificInstance(thread_safe_sender_.get());
+ std::vector<int32_t> remove_observers =
+ dispatcher->RemoveIDBObserver(ipc_database_id_, observer);
+ // TODO(palakj): Remove id from observes_id_
}
void WebIDBDatabaseImpl::get(long long transaction_id,

Powered by Google App Engine
This is Rietveld 408576698