WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
21347
FormDataStreamMac.mm calls isMainThread without #including <wtf/Threading.h>
https://bugs.webkit.org/show_bug.cgi?id=21347
Summary
FormDataStreamMac.mm calls isMainThread without #including <wtf/Threading.h>
Mark Mentovai
Reported
2008-10-03 14:21:34 PDT
WebCore/platform/network/mac/FormDataStreamMac.mm calls isMainThread without the #include that declares that function. This causes a problem when building Chromium. FormDataStreamMac.mm does include <wtf/MainThread.h>, but that file does not bring in Threading.h or the isMainThread function.
Attachments
Adds the appropriate #import
(910 bytes, patch)
2008-10-03 14:24 PDT
,
Mark Mentovai
sam
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Mark Mentovai
Comment 1
2008-10-03 14:24:02 PDT
Created
attachment 24068
[details]
Adds the appropriate #import
Darin Adler
Comment 2
2008-10-03 14:26:25 PDT
Comment on
attachment 24068
[details]
Adds the appropriate #import Why doesn't this break compilation? Is there something different about your configuration?
Mark Mentovai
Comment 3
2008-10-03 14:29:35 PDT
You might be getting it via prefix header injection, although we're using the same prefix header I assume you are, WebCore/WebCorePrefix.h.
Eric Seidel (no email)
Comment 4
2008-10-06 18:11:52 PDT
Committing to
http://svn.webkit.org/repository/webkit/trunk
... M WebCore/ChangeLog M WebCore/platform/network/mac/FormDataStreamMac.mm Committed
r37361
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