[Webkit-unassigned] [Bug 36001] New: SQLiteFileSystem::ensureDatabaseFileExists() doesn't do what it supposed to do.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=36001

           Summary: SQLiteFileSystem::ensureDatabaseFileExists() doesn't
                    do what it supposed to do.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Platform
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: lyon.chen at torchmobile.com
                CC: yong.li.webkit at gmail.com


For SQLiteFileSystem::ensureDatabaseFileExists(const String& fileName, bool
checkPathOnly), the second parameter is checkPathOnly and it means check path
only when it's true, but the only place it's called is
DatabaseTracker::openTrackerDatabase(), which calls ensureDatabaseFileExists()
with parameter createIfDoesNotExist and it expect the file will be created if
it doesn't existed already.

I think SQLiteFileSystem::ensureDatabaseFileExists() should be changed to do as
its function name indicates, create the file if it's not existed already.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.