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

Issue 11470013: IndexedDB: Tests for unknown serialized data versions in backing stores. (Closed)

Created:
8 years ago by jsbell
Modified:
8 years ago
Reviewers:
alecflett, dgrogan
CC:
chromium-reviews, joi+watch-content_chromium.org, jam, darin-cc_chromium.org
Visibility:
Public.

Description

IndexedDB: Tests for unknown serialized data versions in backing stores. If the serialized data format (SerializedScriptValue) is advanced, a data store is created, then the user downgrades to an older browser version but does not delete the profile directory we treat this the same as an unknown schema version and blow away the backing store. This is not a supported scenario but it happens frequently enough that we don't want to wedge such users with silently corrupted data. This is just a test. See http://webkit.org/b/102243 for the patch. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=172141

Patch Set 1 #

Patch Set 2 : Landed test data separately #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -0 lines) Patch
M content/browser/in_process_webkit/indexed_db_browsertest.cc View 1 chunk +16 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jsbell
dgrogan@, alecflett@ - please take a look? Can't land this until wkbug.com/102243 lands, and I'll ...
8 years ago (2012-12-06 23:54:20 UTC) #1
dgrogan
lgtm
8 years ago (2012-12-07 00:09:31 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jsbell@chromium.org/11470013/4001
8 years ago (2012-12-10 18:52:44 UTC) #3
commit-bot: I haz the power
8 years ago (2012-12-10 22:04:02 UTC) #4
Message was sent while issue was closed.
Change committed as 172141

Powered by Google App Engine
This is Rietveld 408576698