Bug 71167 - MessagePort should be a ContextDestructionObserver
Summary: MessagePort should be a ContextDestructionObserver
Status: RESOLVED LATER
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on: 71171
Blocks:
  Show dependency tree / graph
 
Reported: 2011-10-28 17:52 PDT by Adam Barth
Modified: 2011-11-07 09:46 PST (History)
5 users (show)

See Also:

Attachments
Patch (5.04 KB, patch)
2011-10-28 17:54 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2011-10-28 17:52:45 PDT 
MessagePort should be a ContextDestructionObserver
Comment 1 Adam Barth 2011-10-28 17:54:33 PDT 
Created attachment 112951 [details]
Patch
Comment 2 WebKit Review Bot 2011-10-28 22:52:08 PDT 
Comment on attachment 112951 [details]
Patch

Clearing flags on attachment: 112951

Committed r98795: <http://trac.webkit.org/changeset/98795>
Comment 3 WebKit Review Bot 2011-10-28 22:52:12 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 4 Ryosuke Niwa 2011-10-28 23:44:14 PDT 
A bunch of worker tests started to crash after this patch: http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20(Tests)/r98795%20(34286)/results.html
Comment 5 Alexey Proskuryakov 2011-10-30 22:30:08 PDT 
There is no rationale for this change anywhere. Also, wasn't this patch rolled out?
Comment 6 Adam Barth 2011-10-30 22:32:37 PDT 
The patch was in fact rolled out.