RESOLVED FIXED 57568
WebKitTestRunner needs layoutTestController.setDatabaseQuota 
https://bugs.webkit.org/show_bug.cgi?id=57568
Summary WebKitTestRunner needs layoutTestController.setDatabaseQuota
Chang Shu
Reported 2011-03-31 13:15:41 PDT
Failed layout tests include at least: storage/open-database-creation-callback.html
Attachments
fix patch (8.18 KB, patch)
2011-04-01 08:25 PDT, Chang Shu 		no flags
DetailsFormatted DiffDiff
fix patch 2: fix build break (8.32 KB, patch)
2011-04-01 09:04 PDT, Chang Shu 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chang Shu
Comment 1 2011-04-01 08:25:05 PDT
Created attachment 87859 [details] fix patch
Early Warning System Bot
Comment 2 2011-04-01 08:44:52 PDT
Attachment 87859 [details] did not build on qt: Build output: http://queues.webkit.org/results/8319193
Chang Shu
Comment 3 2011-04-01 09:04:06 PDT
Created attachment 87866 [details] fix patch 2: fix build break
Build Bot
Comment 4 2011-04-01 09:28:39 PDT
Attachment 87859 [details] did not build on win: Build output: http://queues.webkit.org/results/8307198
WebKit Commit Bot
Comment 5 2011-04-01 12:57:06 PDT
The commit-queue encountered the following flaky tests while processing attachment 87866 [details]: http/tests/loading/load-javascript-after-many-xhrs.html bug 57652 (author: hayato@chromium.org) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 6 2011-04-01 12:59:44 PDT
Comment on attachment 87866 [details] fix patch 2: fix build break Clearing flags on attachment: 87866 Committed r82710: <http://trac.webkit.org/changeset/82710>
WebKit Commit Bot
Comment 7 2011-04-01 12:59:48 PDT
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 8 2011-04-01 13:35:47 PDT
http://trac.webkit.org/changeset/82710 might have broken Windows 7 Release (Tests)
Chris Dumez
Comment 9 2012-09-14 06:20:28 PDT
Comment on attachment 87866 [details] fix patch 2: fix build break View in context: https://bugs.webkit.org/attachment.cgi?id=87866&action=review > Source/WebKit2/WebProcess/InjectedBundle/InjectedBundle.cpp:130 > + WebDatabaseManager::shared().setQuotaForOrigin("file:///", quota); WebDatabaseManager::setQuotaForOrigin() actually expects a databaseIdentifier so "file:///" does not work here (I believe "file__0" is what we use as databaseIdentifier for local files). I'm addressing the issue in Bug 57570. I noticed the problem after adding the database callbacks dumping.
Note You need to log in before you can comment on or make changes to this bug.