[Webkit-unassigned] [Bug 24992] crash at http://browserspy.dk/browser.php

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu May 7 01:20:07 PDT 2009


ap at webkit.org changed:

           What    |Removed                     |Added
             Status|UNCONFIRMED                 |NEW
          Component|Frames                      |Page Loading
     Ever Confirmed|0                           |1

------- Comment #11 from ap at webkit.org  2009-05-07 01:20 PDT -------
So, the problem is that a client call (setStatusbarText) is made when there is
still no document in the frame. It happens so that Qt asks for repaint, but
another client could ask for something else.

Instead of adding document null checks to all code paths a client could
potentially trigger, I think that it would be better to ensure that the frame
has a document before any client calls.

An obvious way to achieve this is to move "begin(KURL(), false);" one or two
lines upwards in FrameLoader::init(). I don't know this code well enough to
predict if that will cause any undesired consequences, but it seems worth
experimenting with. If it works, please also add an ASSERT(m_doc) in
Frame::setJSStatusBarText(), with a comment briefly explaining that we want the
frame to be in a consistent state before handing off control to the client.

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

More information about the webkit-unassigned mailing list