Bug 150929

Summary: generate-js-builtins.js should support @internal annotation
Product: WebKit Reporter: youenn fablet <youennf>
Component: Tools / TestsAssignee: youenn fablet <youennf>
Severity: Normal CC: bburg, benjamin, calvaris, commit-queue, lforschler
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description youenn fablet 2015-11-05 05:24:19 PST
@internals allows to distinguish files containing JS built-in functions implementing public APIs, hence tied to a constructor/prototype from internal JS functions, stored within private slots of JSGlobalObject.
Comment 1 youenn fablet 2015-11-05 06:02:40 PST
Created attachment 264859 [details]
Comment 2 WebKit Commit Bot 2015-11-05 06:04:54 PST
This patch modifies the JS builtins code generator. Please ensure that you have rebaselined any generator test results (i.e., by running `Tools/Scripts/run-builtins-generator-tests --reset-results`)
Comment 3 youenn fablet 2015-11-05 06:08:48 PST
(In reply to comment #1)
> Created attachment 264859 [details]
> Patch

This patch only addresses XXBuiltinsFunctions boilerplate classes.
We should also remove XXBuiltinsWrapper::exportNames for non-@internal built-ins.
This might be better done as a follow-up patch once bug 150482 is resolved.
Comment 4 BJ Burg 2015-11-06 11:01:11 PST
Comment on attachment 264859 [details]

Comment 5 WebKit Commit Bot 2015-11-08 03:40:32 PST
Comment on attachment 264859 [details]

Clearing flags on attachment: 264859

Committed r192142: <http://trac.webkit.org/changeset/192142>
Comment 6 WebKit Commit Bot 2015-11-08 03:40:37 PST
All reviewed patches have been landed.  Closing bug.