Bug 23554

Summary: dispatchDecidePolicyForNavigationAction invoked for invalid URL of embed tag
Product: WebKit Reporter: Rahul Kuchhal <kuchhal>
Component: WebCore Misc.Assignee: Dimitri Glazkov (Google) <dglazkov>
Status: ASSIGNED    
Severity: Normal CC: cmarcelo, dglazkov
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Test html none

Rahul Kuchhal
Reported 2009-01-26 14:58:15 PST
If I load a html file that only has "<html><embed src="" type="text/html"></html>" contents, the method WebFrame::dispatchDecidePolicyForNavigationAction() gets called even though the request URL is invalid. Is this the right behavior? Below is the stack trace for Safari on Windows and html file is attached. WebKit.dll!WebFrame::dispatchDecidePolicyForNavigationAction WebKit.dll!WebCore::FrameLoader::checkNavigationPolicy WebKit.dll!WebCore::FrameLoader::loadWithDocumentLoader WebKit.dll!WebCore::FrameLoader::loadWithNavigationAction WebKit.dll!WebCore::FrameLoader::loadURL WebKit.dll!WebCore::FrameLoader::loadURL WebKit.dll!WebFrameLoaderClient::loadURLIntoChild WebKit.dll!WebFrameLoaderClient::createFrame WebKit.dll!WebFrameLoaderClient::createFrame WebKit.dll!WebCore::FrameLoader::loadSubframe WebKit.dll!WebCore::FrameLoader::requestObject WebKit.dll!WebCore::RenderPartObject::updateWidget WebKit.dll!WebCore::HTMLEmbedElement::updateWidget WebKit.dll!WebCore::HTMLPlugInElement::updateWidgetCallback WebKit.dll!WebCore::ContainerNode::dispatchPostAttachCallbacks WebKit.dll!WebCore::ContainerNode::attach WebKit.dll!WebCore::Element::attach() Line 648 C++ WebKit.dll!WebCore::HTMLEmbedElement::attach() Line 161 C++ WebKit.dll!WebCore::HTMLParser::insertNode WebKit.dll!WebCore::HTMLParser::handleError WebKit.dll!WebCore::HTMLParser::insertNode WebKit.dll!WebCore::HTMLParser::handleError WebKit.dll!WebCore::HTMLParser::insertNode WebKit.dll!WebCore::HTMLParser::parseToken WebKit.dll!WebCore::HTMLTokenizer::processToken WebKit.dll!WebCore::HTMLTokenizer::parseTag WebKit.dll!WebCore::HTMLTokenizer::write WebKit.dll!WebCore::FrameLoader::write WebKit.dll!WebCore::FrameLoader::endIfNotLoadingMainResource WebKit.dll!WebCore::FrameLoader::end WebKit.dll!WebCore::DocumentLoader::finishedLoading WebKit.dll!WebCore::FrameLoader::finishedLoading WebKit.dll!WebCore::MainResourceLoader::didFinishLoading WebKit.dll!WebCore::ResourceLoader::didFinishLoading WebKit.dll!WebCore::didFinishLoading
Attachments
Test html (44 bytes, text/html)
2009-01-26 14:58 PST, Rahul Kuchhal 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Rahul Kuchhal
Comment 1 2009-01-26 14:58:47 PST
Created attachment 27050 [details] Test html
Dimitri Glazkov (Google)
Comment 2 2009-08-06 10:47:08 PDT
That is a good question. Will use this bug for investigating. For reference, the test case has already made it into a repository as a layout test: http://trac.webkit.org/changeset/40792 https://bugs.webkit.org/show_bug.cgi?id=23806
Note You need to log in before you can comment on or make changes to this bug.