Bug 72432

Summary: [v8] Use throwError instead of compiling and running script in handleMaxRecursionDepthExceeded
Product: WebKit Reporter: Adam Klein <adamk>
Component: New BugsAssignee: Adam Klein <adamk>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, arv, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Fixed test expectations none

Adam Klein
Reported 2011-11-15 15:49:13 PST
[v8] Use throwError instead of compiling and running script in handleMaxRecursionDepthExceeded
Attachments
Patch (1.45 KB, patch)
2011-11-15 15:49 PST, Adam Klein 		no flags
DetailsFormatted DiffDiff
Fixed test expectations (5.61 KB, patch)
2011-11-15 17:05 PST, Adam Klein 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Klein
Comment 1 2011-11-15 15:49:27 PST
Created attachment 115267 [details] Patch
Nate Chapin
Comment 2 2011-11-15 15:51:19 PST
Comment on attachment 115267 [details] Patch /me is embarrassed for missing that first time around :(
WebKit Review Bot
Comment 3 2011-11-15 16:47:54 PST
Comment on attachment 115267 [details] Patch Rejecting attachment 115267 [details] from commit-queue. New failing tests: fast/xmlhttprequest/xmlhttprequest-recursive-sync-event.html Full output: http://queues.webkit.org/results/10361542
Adam Klein
Comment 4 2011-11-15 17:05:25 PST
Created attachment 115284 [details] Fixed test expectations
WebKit Review Bot
Comment 5 2011-11-15 17:28:09 PST
Comment on attachment 115284 [details] Fixed test expectations Clearing flags on attachment: 115284 Committed r100376: <http://trac.webkit.org/changeset/100376>
WebKit Review Bot
Comment 6 2011-11-15 17:28:14 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.