Bug 45462

Summary: Application cache directory should be checked before sending to the web process
Product: WebKit Reporter: Balazs Kelemen <kbalazs>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, commit-queue, sam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
proposed patch
none
proposed patch
none
proposed patch none

Description Balazs Kelemen 2010-09-09 09:16:03 PDT
If the platform does not implements WebContext::applicationCacheDirectory (returns with null string) then the WebProcess asserts in ApplicationCacheStorage::setCacheDirectory. Since this is an optional feature we should check the value before sending it to the WebProcess.
Comment 1 Balazs Kelemen 2010-09-09 09:25:44 PDT
Created attachment 67044 [details]
proposed patch
Comment 2 Balazs Kelemen 2010-09-09 09:45:14 PDT
Created attachment 67047 [details]
proposed patch

Check emptiness instead of null value since it makes more sense (and includes the null case).
Comment 3 Balazs Kelemen 2010-09-10 09:44:54 PDT
Created attachment 67196 [details]
proposed patch

I am thoughtless. One more typo fix was needed.
Comment 4 Andreas Kling 2010-09-10 10:14:11 PDT
Comment on attachment 67196 [details]
proposed patch

r=me
Comment 5 WebKit Commit Bot 2010-09-10 22:21:46 PDT
Comment on attachment 67196 [details]
proposed patch

Clearing flags on attachment: 67196

Committed r67278: <http://trac.webkit.org/changeset/67278>
Comment 6 WebKit Commit Bot 2010-09-10 22:21:51 PDT
All reviewed patches have been landed.  Closing bug.