Summary: | Add the blob URL member to FormData | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jian Li <jianli> | ||||||
Component: | WebCore JavaScript | Assignee: | Jian Li <jianli> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dimich, fishd, kinuko, levin, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Jian Li
2010-08-21 18:44:42 PDT
Created attachment 65040 [details]
Proposed Patch
Attachment 65040 [details] did not pass style-queue:
Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebCore/platform/network/FormData.h:79: An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4]
Total errors found: 1 in 6 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 65044 [details]
Proposed Patch
Fix style error.
Comment on attachment 65044 [details]
Proposed Patch
WebKit/chromium/src/WebHTTPBody.cpp:80
+ result.fileStart = 0;
nit: these fields exist regardless of whether or not ENABLE(BLOB) is defined,
so they probably should be initialized even when ENABLE(BLOB) is not defined.
otherwise, LG
R=me
Committed as http://trac.webkit.org/changeset/65786. |