Bug 79090

Summary: Enable the IDL attribute checker in all build systems
Product: WebKit Reporter: Kentaro Hara <haraken>
Component: WebCore JavaScriptAssignee: Kentaro Hara <haraken>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, darin, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 79081    
Attachments:
Description Flags
WIP patch to see if build succeeds
none
Patch
none
Patch none

Kentaro Hara
Reported 2012-02-21 01:25:39 PST
We enabled the IDL attribute checker in Chromium in bug 79082. In this bug, we enable it in all other build systems.
Attachments
WIP patch to see if build succeeds (11.47 KB, patch)
2012-02-21 01:30 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
Patch (12.08 KB, patch)
2012-02-21 02:00 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
Patch (12.08 KB, patch)
2012-02-21 03:20 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-02-21 01:30:29 PST
Created attachment 127935 [details] WIP patch to see if build succeeds
Kentaro Hara
Comment 2 2012-02-21 02:00:15 PST
Created attachment 127943 [details] Patch
Early Warning System Bot
Comment 3 2012-02-21 02:32:22 PST
Comment on attachment 127943 [details] Patch Attachment 127943 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/11520095
Kentaro Hara
Comment 4 2012-02-21 03:20:25 PST
Created attachment 127950 [details] Patch
Adam Barth
Comment 5 2012-02-21 10:53:07 PST
Comment on attachment 127950 [details] Patch Isn't there some code to remove in the Perl now that this parameter is mandatory?
WebKit Review Bot
Comment 6 2012-02-21 14:51:18 PST
Comment on attachment 127950 [details] Patch Clearing flags on attachment: 127950 Committed r108404: <http://trac.webkit.org/changeset/108404>
WebKit Review Bot
Comment 7 2012-02-21 14:51:23 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.