RESOLVED INVALID Bug 38586
Web Inspector: crash in InspectorController::identifierForInitialRequest when we have no active document loader. 
https://bugs.webkit.org/show_bug.cgi?id=38586
Summary Web Inspector: crash in InspectorController::identifierForInitialRequest when...
Ilya Tikhonovsky
Reported 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&)
Attachments
[patch] Initial version. (1.34 KB, patch)
2010-05-05 08:11 PDT, Ilya Tikhonovsky 		eric: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ilya Tikhonovsky
Comment 1 2010-05-05 08:11:17 PDT
Created attachment 55125 [details] [patch] Initial version.
Pavel Feldman
Comment 2 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).
Eric Seidel (no email)
Comment 3 2010-06-12 21:03:58 PDT
Comment on attachment 55125 [details] [patch] Initial version. Actxually, r- based on pfeldman's above comment.
Note You need to log in before you can comment on or make changes to this bug.