Bug 38586

Summary: Web Inspector: crash in InspectorController::identifierForInitialRequest when we have no active document loader.
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Severity: Normal CC: pfeldman, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
[patch] Initial version. eric: review-

Description Ilya Tikhonovsky 2010-05-05 08:08:40 PDT
0x03959ee1	 [Google Chrome Framework	 - InspectorController.cpp:867]	WebCore::InspectorController::identifierForInitialRequest(unsigned long, WebCore::DocumentLoader*, WebCore::ResourceRequest const&)
0x035f16ff	 [Google Chrome Framework	 - webplugin_impl.cc:1040]	webkit_glue::WebPluginImpl::InitiateHTTPRequest(unsigned long, webkit_glue::WebPluginResourceClient*, GURL const&, char const*, char const*, int, char const*, webkit_glue::WebPluginImpl::Referrer)
0x035f4454	 [Google Chrome Framework	 - webplugin_impl.cc:986]	webkit_glue::WebPluginImpl::HandleURLRequestInternal(char const*, char const*, char const*, char const*, unsigned int, int, bool, webkit_glue::WebPluginImpl::Referrer)
0x035f463a	 [Google Chrome Framework	 - webplugin_impl.cc:918]	webkit_glue::WebPluginImpl::HandleURLRequest(char const*, char const*, char const*, char const*, unsigned int, int, bool)
0x02f4915e	 [Google Chrome Framework	 - webplugin_delegate_proxy.cc:1328]	WebPluginDelegateProxy::OnHandleURLRequest(PluginHostMsg_URLRequest_Params const&)
0x02f4b3d3	 [Google Chrome Framework	 - ../base/tuple.h:422]	WebPluginDelegateProxy::OnMessageReceived(IPC::Message const&)
0x0294b172	 [Google Chrome Framework	 - message_router.cc:40]	MessageRouter::RouteMessage(IPC::Message const&)
Comment 1 Ilya Tikhonovsky 2010-05-05 08:11:17 PDT
Created attachment 55125 [details]
[patch] Initial version.
Comment 2 Pavel Feldman 2010-05-05 08:20:43 PDT
Comment on attachment 55125 [details]
[patch] Initial version.

I'd suggest that we support nullable loader in InspectorController instead. We should be able to support timing for resources that have no explicit type and/or content (i.e. no live loader object).
Comment 3 Eric Seidel (no email) 2010-06-12 21:03:58 PDT
Comment on attachment 55125 [details]
[patch] Initial version.

Actxually, r- based on pfeldman's above comment.