[Chromium] IndexedDB: Turn "should only be true in unit tests" comments into ASSERTs
Created attachment 195778 [details] Patch
alecflett@, dgrogan@ - worth it or overkill?
Comment on attachment 195778 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195778&action=review Definitely worth it. > Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.cpp:1359 > + ASSERT(m_factory || WebKit::Platform::current()->unitTestSupport()); Does it make sense to put #else ASSERT(m_factory) Do the other ports have unit tests?
Comment on attachment 195778 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195778&action=review >> Source/WebCore/Modules/indexeddb/IDBDatabaseBackendImpl.cpp:1359 >> + ASSERT(m_factory || WebKit::Platform::current()->unitTestSupport()); > > Does it make sense to put > #else > ASSERT(m_factory) > > Do the other ports have unit tests? Not at this level of intrusion. Waffling on the #else - it's valid for now so probably worth adding.
Created attachment 196014 [details] Patch
abarth@ - could you take a look at this?
Comment on attachment 196014 [details] Patch Clearing flags on attachment: 196014 Committed r147381: <http://trac.webkit.org/changeset/147381>
All reviewed patches have been landed. Closing bug.