[Webkit-unassigned] [Bug 27444] New: Wrong FrameLoader::activeDocumentLoader when loading a invalid URL.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27444

           Summary: Wrong FrameLoader::activeDocumentLoader when loading a
                    invalid URL.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Page Loading
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: tonikitoo at gmail.com



Antonio Gomes (tonikitoo) <tonikitoo at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33081|                            |review?
               Flag|                            |


Created an attachment (id=33081)
 --> (https://bugs.webkit.org/attachment.cgi?id=33081)
set the provisional documentLoader as the activeDocument in case of load
failures (v0.1)

Steps to reproduce:

1) load a valid url (e.g. http://google.com). Calling:

FrameLoader::activeDocumentLoader::originalURL() -> http://google.com
FrameLoader::activeDocumentLoader::url() -> http://www.google.com
(resolved/redirected)

--> FROM GDB we see that the proper DocumentLoader is set to the FrameLoader
after the load succeeds (see below):

gdb_$ b FrameLoader.cpp:setDocumentLoader
gdb_$ bt
FrameLoader::finishedLoading:3092
DocumentLoader::finishedLoading:353
main frame - willCloseFrame
FrameLoader::stopLoading:566
FrameLoader::transitionToCommitted:2869
*******FrameLoader::setDocumentLoader:2691 'loader=0x886bdb0'

2) after that, load an invalid url (e.g. http://abcdef.abcdef). 

--> Since it fails to load, FrameLoader::setDocumentLoader is *never* called,
and so calling Frameloader::activeDocumentLoader() returns a reference to
"0x886bdb0" (documentLoader of the previous loaded URL - see above). It results
in wrong originalURL values:

FrameLoader::activeDocumentLoader::originalURL() -> http://google.com     
<---- WRONG
FrameLoader::activeDocumentLoader::url() -> http://abcdef.abcdef

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.