Bug 35997

Summary:

Webkit doesn't build with workers on and database off

Product:

WebKit

Reporter:

Noam Rosenthal <noam>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, kent.hansen

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Proposed patch	none

Description Noam Rosenthal 2010-03-10 17:26:14 PST

This was tested on the Qt version, but might be applicable elsewhere
When building webkit with --no-database, but without --no-workers, the build fails in WorkerThread.cpp line 188

Comment 1 Kent Hansen 2010-03-11 09:35:56 PST

I could make it compile by adding a few #if ENABLE(DATABASE). It looks like those were missed in r53595, since the patch does have DATABASE guards in other places.
Anyways, this code is not Qt-specific.

Comment 2 Kent Hansen 2010-03-11 10:00:21 PST

Created attachment 50513 [details]
Proposed patch

Comment 3 WebKit Commit Bot 2010-03-12 06:53:15 PST

Comment on attachment 50513 [details]
Proposed patch

Clearing flags on attachment: 50513

Committed r55896: <http://trac.webkit.org/changeset/55896>

Comment 4 WebKit Commit Bot 2010-03-12 06:53:25 PST

All reviewed patches have been landed.  Closing bug.