Bug 52278

Summary: A more suitable place for webkitgtk's database location on Windows
Product: WebKit Reporter: Bakhtiar Hasmanan <mr.tiar>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: mrobinson
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   

Description Bakhtiar Hasmanan 2011-01-11 23:36:52 PST
Just a suggestion,
WebkitGTK by default using g_get_user_data_dir() for storing sqlite database file. In Windows this path mapped into My Documents folder not Application Data folder. It would be nice to use APPDATA environment variable using g_getenv() instead for Windows.
Comment 1 Martin Robinson 2011-01-12 12:53:05 PST
According to the documentation for g_get_user_data_dir, GLib uses CSIDL_LOCAL_APPDATA on Windows. Documentation for the CSIDL_Foo constants is here http://msdn.microsoft.com/en-us/library/bb762494(v=vs.85).aspx. If this location isn't consistent with what a Windows user expects, wouldn't it make more sense to file this bug against GLib?
Comment 2 Bakhtiar Hasmanan 2011-01-12 15:57:16 PST
(In reply to comment #1)
> According to the documentation for g_get_user_data_dir, GLib uses CSIDL_LOCAL_APPDATA on Windows. Documentation for the CSIDL_Foo constants is here http://msdn.microsoft.com/en-us/library/bb762494(v=vs.85).aspx. If this location isn't consistent with what a Windows user expects, wouldn't it make more sense to file this bug against GLib?

You're right, that's strange it should be correct then. 
I'll dig this more and report to glib.. I think this report can be closed
Comment 3 Martin Robinson 2011-01-12 16:12:48 PST
(In reply to comment #2)
> (In reply to comment #1)
> > According to the documentation for g_get_user_data_dir, GLib uses CSIDL_LOCAL_APPDATA on Windows. Documentation for the CSIDL_Foo constants is here http://msdn.microsoft.com/en-us/library/bb762494(v=vs.85).aspx. If this location isn't consistent with what a Windows user expects, wouldn't it make more sense to file this bug against GLib?
> 
> You're right, that's strange it should be correct then. 
> I'll dig this more and report to glib.. I think this report can be closed

Thank you for reporting the issue! If you find that this is indeed a problem with WebKitGTK+ feel free to re-open this bug.