Bug 24145

Summary:

Remove "#if ENABLE(WORKERS)" wrap from CrossThreadCopier files.

Product:

WebKit

Reporter:

Jian Li <jianli>

Component:

WebCore Misc.

Assignee:

David Levin <levin>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, dimich, jianli, levin

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Proposed Patch	ap: review+

Jian Li

Reported 2009-02-24 18:03:00 PST

Need to remove "#if ENABLE(WORKERS)" wrap from CrossThreadCopier files.

Attachments
Proposed Patch (1.51 KB, patch) 2009-02-24 18:07 PST, Jian Li	ap: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Jian Li

Comment 1 2009-02-24 18:07:54 PST

Created attachment 27946 [details] Proposed Patch

Alexey Proskuryakov

Comment 2 2009-02-25 02:26:58 PST

Comment on attachment 27946 [details] Proposed Patch r=me Please mention the rationale for changes like this (at least in bugzilla), for those who follow along at home. CrossThreadCopier can be used when communicating with non-worker threads, which is why it doesn't need the ENABLE() guards, and lives in platform/.

David Levin

Comment 3 2009-02-25 09:11:57 PST

Assigned to levin@chromium.org for landing.

David Levin

Comment 4 2009-02-25 10:07:58 PST

Committed in r41217.

Note You need to log in before you can comment on or make changes to this bug.