Bug 122168

Summary:

Consolidate multiple OSRExit calls into one.

Product:

WebKit

Reporter:

Nadav Rotem <nrotem>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, fpizlo, ggaren

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Nadav Rotem 2013-10-01 11:55:50 PDT

Consolidate multiple OSRExit calls into one.

Comment 1 Nadav Rotem 2013-10-01 11:56:29 PDT

Created attachment 213105 [details]
Patch

Comment 2 Filip Pizlo 2013-10-01 12:06:42 PDT

Comment on attachment 213105 [details]
Patch

r=me.

Comment 3 Nadav Rotem 2013-10-01 13:52:18 PDT

should I cc commit-queue ?

Comment 4 Filip Pizlo 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.

Comment 5 WebKit Commit Bot 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>

Comment 6 WebKit Commit Bot 2013-10-01 14:25:16 PDT

All reviewed patches have been landed.  Closing bug.