28175 2009-08-11 06:21:18 -0700 HTML 5 database and LocalStorage is not persistent on the Symbian port of QtWebKit 2009-08-12 09:42:08 -0700 1 1 1 Unclassified WebKit WebKit Qt 528+ (Nightly build) PC Other RESOLVED FIXED P2 Normal --- 0 laszlo.gombos webkit-unassigned eric oldest_to_newest 138946 0 laszlo.gombos 2009-08-11 06:21:18 -0700 The path separators in the path that is being passed to sqlite is not the native OS path; instead it is always the "/" - the Qt universal directory separator. Most OS (even Windows I think) understands the "/" separator - but not Symbian. The problem could be solved by calling QDir::toNativeSeparators() before the path is passed to the OS. The problem and the possible solution was brought to my attention by David Leong. 138947 1 34553 laszlo.gombos 2009-08-11 06:27:44 -0700 Created attachment 34553 proposed patch. WebCore::pathByAppendingComponent() seems to be a good place to call QDir::toNativeSeparators before the path is passed to the OS. All the path construction to sqlite DBs are going trough this function. 139240 2 34553 eric 2009-08-11 22:32:21 -0700 Comment on attachment 34553 proposed patch. Seems OK. I'm not sure this is the right place for this change. But i'm not sure I have a better suggestion. 139363 3 34553 eric 2009-08-12 09:42:05 -0700 Comment on attachment 34553 proposed patch. Clearing flags on attachment: 34553 Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/platform/qt/FileSystemQt.cpp Committed r47114 M WebCore/ChangeLog M WebCore/platform/qt/FileSystemQt.cpp r47114 = 7f2ff9356922227e477417a6efb09c9d854e485e (trunk) No changes between current HEAD and refs/remotes/trunk Resetting to the latest refs/remotes/trunk http://trac.webkit.org/changeset/47114 139364 4 eric 2009-08-12 09:42:08 -0700 All reviewed patches have been landed. Closing bug. 34553 2009-08-11 06:27:44 -0700 2009-08-12 09:42:05 -0700 proposed patch. patch_28175.txt text/plain 1264 laszlo.gombos SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0NzAzMSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTcgQEAKKzIwMDktMDgtMTEgIExhc3psbyBHb21ib3MgIDxsYXN6bG8uMS5nb21i b3NAbm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg ICAgICAgIEhUTUwgNSBkYXRhYmFzZSBhbmQgTG9jYWxTdG9yYWdlIGlzIG5vdCBwZXJzaXN0ZW50 IG9uIHRoZSBTeW1iaWFuIHBvcnQgb2YgUXRXZWJLaXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI4MTc1CisKKyAgICAgICAgQ2FsbCBRRGlyOjp0b05h dGl2ZVNlcGFyYXRvcnMoKSBiZWZvcmUgdGhlIHBhdGggaXMgcGFzc2VkIHRvIHRoZSBPUy4KKwor ICAgICAgICBCYXNlZCBvbiB3b3JrIGJ5IERhdmlkIExlb25nLgorICAgICAgICAKKyAgICAgICAg KiBwbGF0Zm9ybS9xdC9GaWxlU3lzdGVtUXQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6cGF0aEJ5 QXBwZW5kaW5nQ29tcG9uZW50KToKKwogMjAwOS0wOC0xMSAgU2ltb24gSGF1c21hbm4gIDxzaW1v bi5oYXVzbWFubkBub2tpYS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChidWls ZCBmaXgpLgpJbmRleDogV2ViQ29yZS9wbGF0Zm9ybS9xdC9GaWxlU3lzdGVtUXQuY3BwCj09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT0KLS0tIFdlYkNvcmUvcGxhdGZvcm0vcXQvRmlsZVN5c3RlbVF0LmNwcAkocmV2aXNpb24g NDY5NjkpCisrKyBXZWJDb3JlL3BsYXRmb3JtL3F0L0ZpbGVTeXN0ZW1RdC5jcHAJKHdvcmtpbmcg Y29weSkKQEAgLTgxLDcgKzgxLDcgQEAgYm9vbCBtYWtlQWxsRGlyZWN0b3JpZXMoY29uc3QgU3Ry aW5nJiBwYQogCiBTdHJpbmcgcGF0aEJ5QXBwZW5kaW5nQ29tcG9uZW50KGNvbnN0IFN0cmluZyYg cGF0aCwgY29uc3QgU3RyaW5nJiBjb21wb25lbnQpCiB7Ci0gICAgcmV0dXJuIFFEaXIocGF0aCku ZmlsZVBhdGgoY29tcG9uZW50KTsKKyAgICByZXR1cm4gUURpcjo6dG9OYXRpdmVTZXBhcmF0b3Jz KFFEaXIocGF0aCkuZmlsZVBhdGgoY29tcG9uZW50KSk7CiB9CiAKIFN0cmluZyBob21lRGlyZWN0 b3J5UGF0aCgpCg==