RESOLVED FIXED Bug 67862
Removed ENABLE(SINGLE_THREADED) support, since it is always false 
https://bugs.webkit.org/show_bug.cgi?id=67862
Summary Removed ENABLE(SINGLE_THREADED) support, since it is always false
Geoffrey Garen
Reported 2011-09-09 13:29:37 PDT
Created attachment 106911 [details] patch Next step toward making the baseline platform assumption that threads exist.
Attachments
patch (9.96 KB, patch)
2011-09-09 13:29 PDT, Geoffrey Garen 		no flags
DetailsFormatted DiffDiff
patch (10.42 KB, patch)
2011-09-09 14:25 PDT, Geoffrey Garen 		mitz: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
WebKit Review Bot
Comment 1 2011-09-09 13:31:19 PDT
Attachment 106911 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/JavaScriptCore/ChangeLog:3: ChangeLog entry has no bug number [changelog/bugnumber] [5] Source/WebCore/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Total errors found: 2 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Geoffrey Garen
Comment 2 2011-09-09 14:25:06 PDT
Created attachment 106922 [details] patch improved changelog
mitz
Comment 3 2011-09-09 15:52:05 PDT
Comment on attachment 106922 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=106922&action=review r=me if the bots approve > JavaScriptCore/ChangeLog:21 > + * wtf/wtf.pri: Removed now-dead code. You actually removed a reference to ThreadingNone.cpp here. > WebCore/ChangeLog:12 > + * WebCore.pri: > + * features.pri: Removed references to > + ThreadingNone.cpp, which was only compiled in single-threaded mode. You actually removed now-dead code here.
Geoffrey Garen
Comment 4 2011-09-09 16:58:33 PDT
The EWS bots that have a chance of finishing are green, so I think this is a go.
Geoffrey Garen
Comment 5 2011-09-09 17:04:33 PDT
Committed r94890: <http://trac.webkit.org/changeset/94890>
Note You need to log in before you can comment on or make changes to this bug.