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

Issue 8242009: Merge 96007 - IndexedDB: Null key path gets stored as empty string key path (Closed)

Created:
9 years, 2 months ago by hans
Modified:
9 years, 2 months ago
Reviewers:
hans
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/874/
Visibility:
Public.

Description

Merge 96007 - IndexedDB: Null key path gets stored as empty string key path https://bugs.webkit.org/show_bug.cgi?id=68726 Patch by Joshua Bell <jsbell@chromium.org>; on 2011-09-26 Reviewed by Tony Chang. Store additional flag to indicate if object store key path is null vs. empty. Added additional runtime tests for integrity of object store metadata. * storage/IDBLevelDBBackingStore.cpp: (WebCore::checkObjectStoreAndMetaDataType): (WebCore::IDBLevelDBBackingStore::getObjectStores): (WebCore::IDBLevelDBBackingStore::createObjectStore): * storage/IDBLevelDBCoding.cpp: TBR=commit-queue@webkit.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97255

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+55 lines, -9 lines) Patch
M Source/WebCore/storage/IDBLevelDBBackingStore.cpp View 4 chunks +48 lines, -9 lines 0 comments Download
M Source/WebCore/storage/IDBLevelDBCoding.h View 1 chunk +1 line, -0 lines 0 comments Download
M Source/WebCore/storage/IDBLevelDBCoding.cpp View 2 chunks +6 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
hans
9 years, 2 months ago (2011-10-12 12:49:30 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698