85293 2012-05-01 09:39:29 -0700 IndexedDB: Refactor *::loadCurrentRow() to share more code 2013-04-08 11:02:50 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED WONTFIX P2 Normal --- 1 alecflett alecflett oldest_to_newest 613473 0 alecflett 2012-05-01 09:39:29 -0700 There is a lot of duplicated code, specifically a lot of tricky version-checking logic that is duplicated between: * ObjectStoreCursorImpl::loadCurrentRow() * IndexCursorImpl::loadCurrentRow() * IndexKeyCursorImpl::loadCurrentRow() Most of the common/version-checking code could go into CursorImplCommon As an aside, there are also a lot of places where these methods return 'false' because the data found is either the wrong version or somehow corrupt. We should think about if we should, at least in the index cases, remove the data so that corrupt data doesn't build up inside the leveldb and slow down iteration. (i.e. imagine something goes haywire and some large chunk of data is corrupt - we'll have to keep skipping over that data every time we iterate) See bug 85224 for an example of where this was causing a real performance issue. That case has been cleaned up but we might want to handle more cases. 613475 1 alecflett 2012-05-01 09:39:54 -0700 filed bug 85293 about the refactoring