WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
101244
Back out controversial changes from
Bug 98665
https://bugs.webkit.org/show_bug.cgi?id=101244
Summary
Back out controversial changes from Bug 98665
Dima Gorbik
Reported
2012-11-05 12:13:42 PST
Back out controversial changes from
Bug 98665
(
https://bugs.webkit.org/show_bug.cgi?id=98665
) until further discussions take place on rules for including Platform.h in Assertions.h
Attachments
Proposed fix 0.1
(3.52 KB, patch)
2012-11-05 13:37 PST
,
Dima Gorbik
ddkilzer
: review-
Details
Formatted Diff
Diff
Proposed fix 0.2
(3.52 KB, patch)
2012-11-05 14:12 PST
,
Dima Gorbik
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Dima Gorbik
Comment 1
2012-11-05 13:37:09 PST
Created
attachment 172392
[details]
Proposed fix 0.1
WebKit Review Bot
Comment 2
2012-11-05 13:43:53 PST
Attachment 172392
[details]
did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/tests/minidom.c'..." exit_code: 1 Source/WTF/wtf/Assertions.h:46: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
David Kilzer (:ddkilzer)
Comment 3
2012-11-05 14:10:21 PST
Comment on
attachment 172392
[details]
Proposed fix 0.1 View in context:
https://bugs.webkit.org/attachment.cgi?id=172392&action=review
> Tools/DumpRenderTree/mac/MockGeolocationProvider.mm:26 > +#include "config.h"
Usually we use #import here instead of #include.
Dima Gorbik
Comment 4
2012-11-05 14:12:06 PST
Created
attachment 172402
[details]
Proposed fix 0.2
WebKit Review Bot
Comment 5
2012-11-05 14:19:51 PST
Attachment 172402
[details]
did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/tests/minidom.c'..." exit_code: 1 Source/WTF/wtf/Assertions.h:46: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
David Kilzer (:ddkilzer)
Comment 6
2012-11-05 14:40:29 PST
Comment on
attachment 172402
[details]
Proposed fix 0.2 r=me
WebKit Review Bot
Comment 7
2012-11-05 15:10:32 PST
Comment on
attachment 172402
[details]
Proposed fix 0.2 Clearing flags on attachment: 172402 Committed
r133533
: <
http://trac.webkit.org/changeset/133533
>
WebKit Review Bot
Comment 8
2012-11-05 15:10:36 PST
All reviewed patches have been landed. Closing bug.
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