Bug 76841

Summary: [Refactoring] Remove finish() from CodeGeneratorV8.pm
Product: WebKit Reporter: Kentaro Hara <haraken>
Component: WebCore JavaScriptAssignee: Kentaro Hara <haraken>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, darin, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 76836    
Attachments:
Description Flags
Patch
none
rebased patch for review none

Kentaro Hara
Reported 2012-01-23 10:52:10 PST
This is one of steps to stop rebuilding .h/.cpp files generated by unchanged IDLs (bug 76836). As refactoring, we are planning to remove finish() from all CodeGenerators. In this bug, we remove finish() from CodeGeneratorV8.pm.
Attachments
Patch (6.26 KB, patch)
2012-01-23 11:01 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
rebased patch for review (6.24 KB, patch)
2012-01-23 22:06 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-01-23 11:01:55 PST
Created attachment 123576 [details] Patch
WebKit Review Bot
Comment 2 2012-01-23 22:00:42 PST
Comment on attachment 123576 [details] Patch Rejecting attachment 123576 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ion 55 --non-interactive --force --accept theirs-conflict --ignore-externals returned non-zero exit status 1 in /mnt/git/webkit-commit-queue/Source/WebKit/chromium Error: 'depot_tools/gclient sync --force --reset --delete_unversioned_trees' failed 3 tries and returned 256 at Tools/Scripts/update-webkit-chromium line 106. Re-trying 'depot_tools/gclient sync --force --reset --delete_unversioned_trees' No such file or directory at /mnt/git/webkit-commit-queue/Tools/Scripts/webkitdirs.pm line 2302. Full output: http://queues.webkit.org/results/11333241
Kentaro Hara
Comment 3 2012-01-23 22:06:02 PST
Created attachment 123697 [details] rebased patch for review
Kentaro Hara
Comment 4 2012-01-23 22:07:20 PST
adam: Sorry, would you please r+ again? I forgot to add --no-review option.
WebKit Review Bot
Comment 5 2012-01-24 02:33:03 PST
Comment on attachment 123697 [details] rebased patch for review Clearing flags on attachment: 123697 Committed r105714: <http://trac.webkit.org/changeset/105714>
WebKit Review Bot
Comment 6 2012-01-24 02:33:08 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.