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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/Source/modules/indexeddb/WebIDBObserverImpl.cpp

Issue 2125213002: [IndexedDB] Propogating changes to observers : Renderer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lifetime
Patch Set: Post dmurph review Created 4 years, 5 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
« content/child/indexed_db/indexed_db_dispatcher.cc ('K') | « content/content_browser.gypi ('k') | third_party/WebKit/public/platform/modules/indexeddb/WebIDBTypes.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/modules/indexeddb/WebIDBObserverImpl.cpp
diff --git a/third_party/WebKit/Source/modules/indexeddb/WebIDBObserverImpl.cpp b/third_party/WebKit/Source/modules/indexeddb/WebIDBObserverImpl.cpp
index e55179c0147f24cd02fe55fc123a1a4b430958a5..3610e284300e18bc3879f0e27cc4158f5083ac62 100644
--- a/third_party/WebKit/Source/modules/indexeddb/WebIDBObserverImpl.cpp
+++ b/third_party/WebKit/Source/modules/indexeddb/WebIDBObserverImpl.cpp
@@ -24,8 +24,8 @@ WebIDBObserverImpl::WebIDBObserverImpl(IDBObserver* observer)
// Remove observe call id from IDBObserver.
WebIDBObserverImpl::~WebIDBObserverImpl()
{
- DCHECK_NE(kInvalidObserverId, m_id);
- m_observer->removeObserver(m_id);
+ if (m_id != kInvalidObserverId)
+ m_observer->removeObserver(m_id);
}
void WebIDBObserverImpl::setId(int32_t id)
« content/child/indexed_db/indexed_db_dispatcher.cc ('K') | « content/content_browser.gypi ('k') | third_party/WebKit/public/platform/modules/indexeddb/WebIDBTypes.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698