Bug 86103

Summary:

Some Spelling Mistakes Result in Compile Errors with ENABLE(FILE_SYSTEM) in LocalFileSystem.cpp

Product:

WebKit

Reporter:

Chris H-C <chutten>

Component:

WebKit Misc.

Assignee:

Kinuko Yasuda <kinuko>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

chutten, kinuko, kpiascik, mlattanzio, rwlbuis, tabbott

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	rwlbuis: review+

Description Chris H-C 2012-05-10 09:18:49 PDT

As per Bug#84135, there are some compilation errors in LocalFileSystem.cpp (FileSystemSynchronousMode doesn't exist, for instance). This bug shall track their resolution.

Comment 1 Kinuko Yasuda 2012-05-15 07:20:28 PDT

Created attachment 141962 [details]
Patch

Comment 2 Kinuko Yasuda 2012-05-15 07:21:19 PDT

I think this should fix build errors with ENABLE(FILE_SYSTEM) on non-chromium ports (though I couldn't test on all platforms).

Comment 3 Chris H-C 2012-05-15 07:26:39 PDT

(In reply to comment #2)
> I think this should fix build errors with ENABLE(FILE_SYSTEM) on non-chromium ports (though I couldn't test on all platforms).

Sorry about that. We're looking to get a BlackBerry buildbot up and running ASAP to make this less painful all around.

Patch lgtm, but that's all I can do :S Adding some people from our port.

Comment 4 Rob Buis 2012-05-17 07:27:45 PDT

Comment on attachment 141962 [details]
Patch

Looks good, thanks for the BB fix too!

Comment 5 Kinuko Yasuda 2012-05-17 22:43:27 PDT

Committed r117544: <http://trac.webkit.org/changeset/117544>