Enforce READ_ONLY transactions in IndexedDB
Created attachment 78565 [details] Patch
Andrei/hans, please take a look. Nate, can you please do the official review?
Comment on attachment 78565 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=78565&action=review The tests have huge diffs for no visible reason, so I'm assuming the line endings changed. Are they right before or after this patch? > Source/WebCore/storage/IDBCursorBackendImpl.cpp:169 > > - // FIXME: Check that the transaction is READ_WRITE > - // if (m_transaction->mode() == IDBTransaction::READ_ONLY) { > - // FIXME: We must return READ_ONLY_ERR here. Fix this when we update IDBDatabaseException to match the spec. > - // ec = IDBDatabaseException::NOT_ALLOWED_ERR; > - // return; > - // } > + Looks like two lines of whitespace here.
(In reply to comment #2) > Andrei/hans, please take a look. Looks good.
Committed r75596: <http://trac.webkit.org/changeset/75596>