Bug 70845 – ASSERTION FAILED: checkURLReceivedFromWebProcess(originalURL) loading a local file, if the WebProcess crashed

• Component: WebKit2
• Status: RESOLVED FIXED

[Simon Fraser (smfr)]

Launch Safari, drag a local file into the window, and you assert:


ASSERTION FAILED: checkURLReceivedFromWebProcess(originalURL)
/Volumes/SSData/Development/OSX/webkit/OpenSource/Source/WebKit2/UIProcess/WebProcessProxy.cpp(252) : void WebKit::WebProcessProxy::addBackForwardItem(uint64_t, const WTF::String &, const WTF::String &, const WTF::String &, const CoreIPC::DataReference &)
1   0x102bf7d9a WebKit::WebProcessProxy::addBackForwardItem(unsigned long long, WTF::String const&, WTF::String const&, WTF::String const&, CoreIPC::DataReference const&)
2   0x102cf8827 void CoreIPC::callMemberFunction<WebKit::WebProcessProxy, void (WebKit::WebProcessProxy::*)(unsigned long long, WTF::String const&, WTF::String const&, WTF::String const&, CoreIPC::DataReference const&), unsigned long long, WTF::String, WTF::String, WTF::String, CoreIPC::DataReference>(CoreIPC::Arguments5<unsigned long long, WTF::String, WTF::String, WTF::String, CoreIPC::DataReference> const&, WebKit::WebProcessProxy*, void (WebKit::WebProcessProxy::*)(unsigned long long, WTF::String const&, WTF::String const&, WTF::String const&, CoreIPC::DataReference const&))
3   0x102cf6dad void CoreIPC::handleMessage<Messages::WebProcessProxy::AddBackForwardItem, WebKit::WebProcessProxy, void (WebKit::WebProcessProxy::*)(unsigned long long, WTF::String const&, WTF::String const&, WTF::String const&, CoreIPC::DataReference const&)>(CoreIPC::ArgumentDecoder*, WebKit::WebProcessProxy*, void (WebKit::WebProcessProxy::*)(unsigned long long, WTF::String const&, WTF::String const&, WTF::String const&, CoreIPC::DataReference const&))
4   0x102cf6834 WebKit::WebProcessProxy::didReceiveWebProcessProxyMessage(CoreIPC::Connection*, CoreIPC::MessageID, CoreIPC::ArgumentDecoder*)
5   0x102bf8192 WebKit::WebProcessProxy::didReceiveMessage(CoreIPC::Connection*, CoreIPC::MessageID, CoreIPC::ArgumentDecoder*)
6   0x102aa308c CoreIPC::Connection::dispatchMessage(CoreIPC::Connection::Message<CoreIPC::ArgumentDecoder>&)
7   0x102aa58a3 CoreIPC::Connection::dispatchMessages()
8   0x102aac6bb MemberFunctionWorkItem0<CoreIPC::Connection>::execute()
9   0x102b0b4cf RunLoop::performWork()
10  0x102b0c400 RunLoop::performWork(void*)
11  0x7fff88d97b51 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__
12  0x7fff88d973bd __CFRunLoopDoSources0
13  0x7fff88dbe1a9 __CFRunLoopRun
14  0x7fff88dbdae6 CFRunLoopRunSpecific
15  0x7fff8ea543d3 RunCurrentEventLoopInMode
16  0x7fff8ea5b63d ReceiveNextEventCommon
17  0x7fff8ea5b4ca BlockUntilNextEventMatchingListInMode
18  0x7fff909083f1 _DPSNextEvent
19  0x7fff90907cf5 -[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:]
20  0x1000db2f3 -[BrowserApplication nextEventMatchingMask:untilDate:inMode:dequeue:]
21  0x7fff9090462d -[NSApplication run]
22  0x7fff90b8380c NSApplicationMain
23  0x1005398c2 SafariMain
24  0x100000f32 main
25  0x100000f04 start

[Simon Fraser (smfr)]

My WebProcess was hitting an unrelated assertion just before this assertion.

[Alexey Proskuryakov]

<rdar://problem/10269985>

[Alexey Proskuryakov]

Created attachment 126191 [details]
proposed fix

[Philippe Normand]

Comment on attachment 126191 [details]
proposed fix

Attachment 126191 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/11461655

[Alexey Proskuryakov]

Created attachment 126197 [details]
with a typo fixed

[Anders Carlsson]

Comment on attachment 126197 [details]
with a typo fixed

View in context: https://bugs.webkit.org/attachment.cgi?id=126197&action=review

> Source/WebKit2/UIProcess/WebProcessProxy.cpp:250
> +    for (WebBackForwardListItemMap::iterator iter = m_backForwardListItemMap.begin(); iter != m_backForwardListItemMap.end(); ++iter) {

The WebKit style is to put end in a local variable. Something like

WebBackForwardListItemMap::iterator iter = m_backForwardListItemMap.begin(), end = ...

[Alexey Proskuryakov]

Created attachment 126367 [details]
patch for landing

[WebKit Review Bot]

Comment on attachment 126367 [details]
patch for landing

Clearing flags on attachment: 126367

Committed r107356: <http://trac.webkit.org/changeset/107356>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.