Bug 38586 - Web Inspector: crash in InspectorController::identifierForInitialRequest when we have no active document loader.
Summary: Web Inspector: crash in InspectorController::identifierForInitialRequest when...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2010-05-05 08:08 PDT by Ilya Tikhonovsky
Modified: 2010-08-26 12:01 PDT (History)
3 users (show)

See Also:

[patch] Initial version. (1.34 KB, patch)
2010-05-05 08:11 PDT, Ilya Tikhonovsky
eric: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.