Bug 97368

Summary: IndexedDB: Modernize some esoteric tests
Product: WebKit Reporter: Joshua Bell <jsbell>
Component: New BugsAssignee: Joshua Bell <jsbell>
Status: RESOLVED FIXED    
Severity: Normal CC: alecflett, dgrogan, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 97051    
Bug Blocks: 97482    
Attachments:
Description Flags
Patch none

Joshua Bell
Reported 2012-09-21 15:03:44 PDT
IndexedDB: Modernize some esoteric tests
Attachments
Patch (50.57 KB, patch)
2012-09-21 15:05 PDT, Joshua Bell 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joshua Bell
Comment 1 2012-09-21 15:05:23 PDT
Created attachment 165202 [details] Patch
Joshua Bell
Comment 2 2012-09-21 15:09:30 PDT
A follow-on to http://webkit.org/b/97051 - this ditches all the debug() calls in favor of evalAndLog() and preamble(). No changes to how the tests operate. I was thinking of landing this, then splitting the tests up into separate files, but I could do both steps at once.
Alec Flett
Comment 3 2012-09-21 16:14:13 PDT
lgtm
Joshua Bell
Comment 4 2012-09-21 16:17:31 PDT
tony@ - r? (low priority)
WebKit Review Bot
Comment 5 2012-09-24 09:19:36 PDT
Comment on attachment 165202 [details] Patch Clearing flags on attachment: 165202 Committed r129378: <http://trac.webkit.org/changeset/129378>
WebKit Review Bot
Comment 6 2012-09-24 09:19:39 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.