RESOLVED FIXED 77693
Add the "JS" prefix to JSC specific IDL attributes 
https://bugs.webkit.org/show_bug.cgi?id=77693
Summary Add the "JS" prefix to JSC specific IDL attributes
Kentaro Hara
Reported 2012-02-02 17:27:01 PST
Some JSC specific IDLs do not have "JS" prefix, e.g. [CustomIsReachable]. It might be OK since JSC is the main JavaScript engine in WebKit, but distinguishing IDLs widely used in WebKit and IDLs used in JSC only would help people understand the role of IDLs. In this bug, we rename the following JSC specific IDLs: CustomFinalize => JSCustomFinalize CustomIsReachable => JSCustomIsReachable CustomMarkFunction => JSCustomMarkFunction CustomToJS => JSCustomToJS CustomNativeConverter => JSCustomToNativeObject (Note: For naming consistency with [JSCustomToJS]) GenerateIsReachable => JSGenerateIsReachable GenerateToJS => JSGenerateToJS NoStaticTables => JSNoStaticTables WindowEventListener => JSWindowEventListener InlineGetOwnPropertySlot => JSInlineGetOwnPropertySlot DelegatingPrototypePutFunction => JSCustomPrototypePutDelegate
Attachments
Patch (117.99 KB, patch)
2012-02-02 17:51 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
rebased patch for commit (118.38 KB, patch)
2012-02-03 16:41 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-02-02 17:51:03 PST
Created attachment 125219 [details] Patch
Kentaro Hara
Comment 2 2012-02-03 01:17:39 PST
Darin: Would you please review this patch? ("JS" prefixing is acceptable or not)
Kentaro Hara
Comment 3 2012-02-03 16:41:32 PST
Reopening to attach new patch.
Kentaro Hara
Comment 4 2012-02-03 16:41:36 PST
Created attachment 125446 [details] rebased patch for commit
WebKit Review Bot
Comment 5 2012-02-04 02:13:08 PST
Comment on attachment 125446 [details] rebased patch for commit Clearing flags on attachment: 125446 Committed r106737: <http://trac.webkit.org/changeset/106737>
WebKit Review Bot
Comment 6 2012-02-04 02:13:13 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.