Summary: | [Curl] Download does not replace existing file. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | peavo | ||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bfulgham, commit-queue, galpeter | ||||
Priority: | P2 | Keywords: | Curl | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
peavo
2013-07-03 06:09:49 PDT
Created attachment 205997 [details]
Patch
Well I'm not a reviewer but this looks good to me. I've just got one (two) question(s) if the MoveFileEx fails then the old file will be still there? and if so won't that cause problems? (In reply to comment #2) > Well I'm not a reviewer but this looks good to me. > > I've just got one (two) question(s) if the MoveFileEx fails then the old file will be still there? and if so won't that cause problems? Thanks for commenting :) I think this scenario should be ok, as the old file is a temporary file in a Windows temp folder. The next download will generate a new, unique, temporary file name, to download to, and not have any references to the previous temporary file. Comment on attachment 205997 [details]
Patch
r=me
Comment on attachment 205997 [details] Patch Clearing flags on attachment: 205997 Committed r152649: <http://trac.webkit.org/changeset/152649> All reviewed patches have been landed. Closing bug. (In reply to comment #4) > (From update of attachment 205997 [details]) > r=me Thanks for reviewing! |