Bug 53815

Summary:

NotificationCenter shouldn't keep its own ScriptExecutionContext*

Product:

WebKit

Reporter:

Nate Chapin <japhet>

Component:

WebCore Misc.

Assignee:

Nate Chapin <japhet>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, commit-queue, eric

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
patch	none

Nate Chapin

Reported 2011-02-04 14:54:31 PST

....it should use the one it inherits from ActiveDOMObject. No need to duplicate, especially since only ActiveDOMObject's gets cleared when the ScriptExecutionContext goes away.

Attachments
patch (5.83 KB, patch) 2011-02-04 15:07 PST, Nate Chapin	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Nate Chapin

Comment 1 2011-02-04 15:07:04 PST

Created attachment 81298 [details] patch

WebKit Commit Bot

Comment 2 2011-02-05 02:35:48 PST

The commit-queue encountered the following flaky tests while processing attachment 81298 [details]: animations/suspend-resume-animation.html bug 48161 (author: cmarrin@apple.com) The commit-queue is continuing to process your patch.

WebKit Commit Bot

Comment 3 2011-02-05 02:37:30 PST

Comment on attachment 81298 [details] patch Clearing flags on attachment: 81298 Committed r77738: <http://trac.webkit.org/changeset/77738>

WebKit Commit Bot

Comment 4 2011-02-05 02:37:34 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.