Bug 127301

Summary:

CodeBlockSet::traceMarked doesn't need to visit the ownerExecutable

Product:

WebKit

Reporter:

Mark Hahnenberg <mhahnenberg>

Component:

JavaScriptCore

Assignee:

Mark Hahnenberg <mhahnenberg>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, fpizlo

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Mark Hahnenberg 2014-01-20 12:44:06 PST

We used to just call CodeBlock::visitAggregate, but now we call visitChildren on the ownerExecutable, which is unnecessary. We should go back to the old way of doing things.

Comment 1 Mark Hahnenberg 2014-01-20 13:14:08 PST

Created attachment 221684 [details]
Patch

Comment 2 WebKit Commit Bot 2014-01-20 14:13:02 PST

Comment on attachment 221684 [details]
Patch

Clearing flags on attachment: 221684

Committed r162377: <http://trac.webkit.org/changeset/162377>

Comment 3 WebKit Commit Bot 2014-01-20 14:13:04 PST

All reviewed patches have been landed.  Closing bug.