WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
24145
Remove "#if ENABLE(WORKERS)" wrap from CrossThreadCopier files.
https://bugs.webkit.org/show_bug.cgi?id=24145
Summary
Remove "#if ENABLE(WORKERS)" wrap from CrossThreadCopier files.
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.
Top of Page
Format For Printing
XML
Clone This Bug