Regression (r102866): Navigating away from or closing a page with a plugin crashes

• Component: WebKit2
• Status: RESOLVED FIXED

[Brady Eidson]

Regression (r102866): Navigating away from a cachable page with a plugin crashes

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.WebKit2             	0x000000010f7d694f CoreIPC::Connection::dispatchConnectionDidClose() + 63 (Connection.cpp:606)
1   com.apple.WebKit2             	0x000000010f7ddc10 WTF::FunctionWrapper<void (CoreIPC::Connection::*)()>::operator()(CoreIPC::Connection*) + 112 (Functional.h:135)
2   com.apple.WebKit2             	0x000000010f7ddb95 WTF::BoundFunctionImpl<WTF::FunctionWrapper<void (CoreIPC::Connection::*)()>, void ()(CoreIPC::Connection*)>::operator()() + 37 (Functional.h:221)
3   com.apple.WebKit2             	0x000000010f83e978 WTF::Function<void ()()>::operator()() + 136 (Functional.h:299)
4   com.apple.WebKit2             	0x000000010f83e8dc FunctionWorkItem::execute() + 28 (WorkItem.h:230)
5   com.apple.WebKit2             	0x000000010f83d82f RunLoop::performWork() + 207 (RunLoop.cpp:64)
6   com.apple.WebKit2             	0x000000010f83ebb0 RunLoop::performWork(void*) + 96 (RunLoopMac.mm:38)

The crashing line is:
    client->didClose(this);

[Brady Eidson]

<rdar://problem/10590024>

[Brady Eidson]

You can actually trigger this just by closing a tab with a page with a plug-in - I noticed closing a tab with Yelp.

[Anders Carlsson]

Created attachment 119530 [details]
Patch

[WebKit Review Bot]

Attachment 119530 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1

Tools/TestWebKitAPI/Tests/WTF/Functional.cpp:156:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Tools/TestWebKitAPI/Tests/WTF/Functional.cpp:166:  More than one command on the same line  [whitespace/newline] [4]
Tools/TestWebKitAPI/Tests/WTF/Functional.cpp:169:  More than one command on the same line  [whitespace/newline] [4]
Total errors found: 3 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Anders Carlsson]

Committed r103008: <http://trac.webkit.org/changeset/103008>

[Alexey Proskuryakov]

*** Bug 74684 has been marked as a duplicate of this bug. ***