Bug 98958 - WebContext should be a MessageReceiver
Summary: WebContext should be a MessageReceiver
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
Depends on: 98986 99280 99281
Blocks:
  Show dependency tree / graph
 
Reported: 2012-10-10 15:26 PDT by Anders Carlsson
Modified: 2012-10-14 19:11 PDT (History)
2 users (show)

See Also:

Attachments
Patch (15.64 KB, patch)
2012-10-10 15:28 PDT, Anders Carlsson 		sam: review+
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 Anders Carlsson 2012-10-10 15:26:10 PDT 
WebContext should be a MessageReceiver
Comment 1 Anders Carlsson 2012-10-10 15:28:15 PDT 
Created attachment 168078 [details]
Patch
Comment 2 Anders Carlsson 2012-10-10 15:34:32 PDT 
Committed r130972: <http://trac.webkit.org/changeset/130972>
Comment 3 WebKit Review Bot 2012-10-10 20:50:48 PDT 
Re-opened since this is blocked by bug 98986
Comment 4 Sam Weinig 2012-10-14 17:26:01 PDT 
Ah, Anders didn't remove the call to dispatchSyncMessages in WebContext::didReceiveSyncMessage, thus resulting in infinite recursion.  I'll check in the fixed version.
Comment 5 Sam Weinig 2012-10-14 17:28:31 PDT 
Re-landed in r131283.
Comment 6 WebKit Review Bot 2012-10-14 18:48:13 PDT 
Re-opened since this is blocked by bug 99280