Bug 122168

Summary: Consolidate multiple OSRExit calls into one.
Product: WebKit Reporter: Nadav Rotem <nrotem>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, fpizlo, ggaren
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Nadav Rotem
Reported 2013-10-01 11:55:50 PDT
Consolidate multiple OSRExit calls into one.
Attachments
Patch (3.88 KB, patch)
2013-10-01 11:56 PDT, Nadav Rotem 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Nadav Rotem
Comment 1 2013-10-01 11:56:29 PDT
Created attachment 213105 [details] Patch
Filip Pizlo
Comment 2 2013-10-01 12:06:42 PDT
Comment on attachment 213105 [details] Patch r=me.
Nadav Rotem
Comment 3 2013-10-01 13:52:18 PDT
should I cc commit-queue ?
Filip Pizlo
Comment 4 2013-10-01 14:02:12 PDT
(In reply to comment #3) > should I cc commit-queue ? You can set 'cq' to '?', and then a committer can set cq+. I presume you want to commit so I will set cq+ for you. The convention is that if a patch is sitting without cq? then it shouldn't be landed, under the assumption that the patch author is still doing some additional tests. But if the patch author sets cq?, then any committer can set cq+ to auto-land it, or land it themselves.
WebKit Commit Bot
Comment 5 2013-10-01 14:25:13 PDT
Comment on attachment 213105 [details] Patch Clearing flags on attachment: 213105 Committed r156734: <http://trac.webkit.org/changeset/156734>
WebKit Commit Bot
Comment 6 2013-10-01 14:25:16 PDT
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.