WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
115961
Fix several style warnings in generated bindings
https://bugs.webkit.org/show_bug.cgi?id=115961
Summary
Fix several style warnings in generated bindings
Chris Dumez
Reported
2013-05-11 10:41:29 PDT
The generating bindings in Source/WebCore/bindings/scripts/test/ do not always follow the WebKit coding style and the style script is complaining when updating them.
Attachments
Patch
(46.68 KB, patch)
2013-05-11 11:09 PDT
,
Chris Dumez
haraken
: review+
commit-queue
: commit-queue-
Details
Formatted Diff
Diff
Patch for landing
(46.71 KB, patch)
2013-05-11 11:24 PDT
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Chris Dumez
Comment 1
2013-05-11 11:09:52 PDT
Created
attachment 201464
[details]
Patch
WebKit Commit Bot
Comment 2
2013-05-11 11:18:10 PDT
Comment on
attachment 201464
[details]
Patch Rejecting
attachment 201464
[details]
from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'validate-changelog', '--non-interactive', 201464, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit /Volumes/Data/EWS/WebKit/Source/WebCore/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive). Full output:
http://webkit-queues.appspot.com/results/417518
Chris Dumez
Comment 3
2013-05-11 11:24:21 PDT
Created
attachment 201466
[details]
Patch for landing
WebKit Commit Bot
Comment 4
2013-05-11 12:07:14 PDT
Comment on
attachment 201466
[details]
Patch for landing Clearing flags on attachment: 201466 Committed
r149935
: <
http://trac.webkit.org/changeset/149935
>
WebKit Commit Bot
Comment 5
2013-05-11 12:07:19 PDT
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