Bug 29743 - Add a sqlite3_io_methods finder to Chromium's POSIX VFS
Summary: Add a sqlite3_io_methods finder to Chromium's POSIX VFS
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-25 11:29 PDT by Dumitru Daniliuc
Modified: 2009-09-28 16:33 PDT (History)
7 users (show)

See Also:


Attachments
patch (1.59 KB, patch)
2009-09-25 11:37 PDT, Dumitru Daniliuc
dglazkov: review+
dglazkov: commit-queue-
Details | Formatted Diff | Diff
patch (2.00 KB, patch)
2009-09-25 18:17 PDT, Dumitru Daniliuc
dglazkov: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff
patch (2.00 KB, patch)
2009-09-27 18:20 PDT, Dumitru Daniliuc
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dumitru Daniliuc 2009-09-25 11:29:08 PDT
The code that registers Chromium's POSIX VFS has a bug: it doesn't specify a sqlite3_io_methods finder. This can result in a crash.
Comment 1 Dumitru Daniliuc 2009-09-25 11:37:52 PDT
Created attachment 40128 [details]
patch

Please do NOT commit-queue+ this patch.
Comment 2 Dimitri Glazkov (Google) 2009-09-25 12:08:22 PDT
Comment on attachment 40128 [details]
patch

r=me.
Comment 3 Dumitru Daniliuc 2009-09-25 18:17:58 PDT
Created attachment 40157 [details]
patch

Same patch, but without requiring a change to the SQLite code. Please DO commit-queue+ this patch when you r+ it.
Comment 4 Dimitri Glazkov (Google) 2009-09-25 20:33:04 PDT
Comment on attachment 40157 [details]
patch

even better! r=me.
Comment 5 WebKit Commit Bot 2009-09-25 22:10:24 PDT
Comment on attachment 40157 [details]
patch

Rejecting patch 40157 from commit-queue.

Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1
Running build-dumprendertree
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 11332 test cases.
http/tests/xmlhttprequest/cross-origin-authorization.html -> failed

Exiting early after 1 failures. 8966 tests run.
247.85s total testing time

8965 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
5 test cases (<1%) had stderr output
Comment 6 Eric Seidel (no email) 2009-09-26 01:26:19 PDT
Comment on attachment 40157 [details]
patch

Flakey test.  Bug 29322.
Comment 7 WebKit Commit Bot 2009-09-26 07:52:36 PDT
Comment on attachment 40157 [details]
patch

Rejecting patch 40157 from commit-queue.

Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1
Running build-dumprendertree
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 11332 test cases.
http/tests/xmlhttprequest/cross-origin-authorization.html -> failed

Exiting early after 1 failures. 8966 tests run.
251.00s total testing time

8965 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
5 test cases (<1%) had stderr output
Comment 8 Dimitri Glazkov (Google) 2009-09-26 07:56:13 PDT
Comment on attachment 40157 [details]
patch

Let's spin again.
Comment 9 Eric Seidel (no email) 2009-09-26 11:03:02 PDT
That's bug 29322.  I have a patch up for review to skip the test due to these types of failures. :(
Comment 10 Eric Seidel (no email) 2009-09-26 11:38:38 PDT
The commit-queue got stuck on this patch because of a programming error (calling error() instead of using ScriptError to return the failure).  It will shortly be rejecting this patch due to an extra character before the date line in the WebCore ChangeLog.  The extra '[' will need to be removed for someone to land this.  (Or an updated patch can be posted and the commit-queue can land it.)
Comment 11 WebKit Commit Bot 2009-09-26 11:47:30 PDT
Comment on attachment 40157 [details]
patch

Rejecting patch 40157 from commit-queue.

Failed to parse ChangeLog: /Users/eseidel/Projects/CommitQueue/WebCore/ChangeLog
Comment 12 Dumitru Daniliuc 2009-09-27 18:20:24 PDT
Created attachment 40209 [details]
patch

Remove the '[' in WebCore/ChangeLog. Sorry about that typo...
Comment 13 Dumitru Daniliuc 2009-09-28 12:52:53 PDT
Comment on attachment 40209 [details]
patch

Testing the ability to post "reviews" for a patch. Please ignore.
Comment 14 Dimitri Glazkov (Google) 2009-09-28 15:19:29 PDT
Comment on attachment 40209 [details]
patch

one more time.
Comment 15 WebKit Commit Bot 2009-09-28 16:18:22 PDT
Comment on attachment 40209 [details]
patch

Rejecting patch 40209 from commit-queue.

Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1
Running build-dumprendertree
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 11339 test cases.
fast/workers/dedicated-worker-lifecycle.html -> failed

Exiting early after 1 failures. 8221 tests run.
172.19s total testing time

8220 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
4 test cases (<1%) had stderr output
Comment 16 Eric Seidel (no email) 2009-09-28 16:19:51 PDT
Comment on attachment 40209 [details]
patch

We're having no luck here.  Now you hit bug 29344!  Damn flakey tests!
Comment 17 WebKit Commit Bot 2009-09-28 16:33:32 PDT
Comment on attachment 40209 [details]
patch

Clearing flags on attachment: 40209

Committed r48845: <http://trac.webkit.org/changeset/48845>
Comment 18 WebKit Commit Bot 2009-09-28 16:33:37 PDT
All reviewed patches have been landed.  Closing bug.