Bug 14387 – [S60] Pointer event in a blank browser window cusses crash

• Status: CLOSED
• Resolution: FIXED
• Component: New Bugs

[Sachin Padma]

- Steps to reproduce:
Open BrowserNG, Option->Goto address, fill a non-existed local address to the address field, e.g. file://c:/ anything.htm, press open button. After  no gateway error message, a blank browser window is shown up. Click on the blank browser window	

- Actual outcome:
Crash

- Expected outcome:
Should not  crash

- Other info:
        I watched following calling sequence before crash: 
CPointerEventHandler::HandlePointerEventL -> CWebCoreBridge::ActivateEvent  ->  KWQKHTMLPart::activateNodeAtPoint

If taking a look at the last function,  renderer() returns null when there is no html document.  The return value of renderer() should be checked.
void KWQKHTMLPart::activateNodeAtPoint(const QPoint &point)
{
    // On ActivateEvent, clear the "don't submit form twice" data member.
    prepareForUserAction();

    RenderObject::NodeInfo renderInfo(true, false);
renderer()->layer()->hitTest(renderInfo, point.x(), point.y());

TSW ID: YDAI-73V998

[Sachin Padma]

Created attachment 15222 [details]
Null pointer check

[zalan]

Comment on attachment 15222 [details]
Null pointer check

r=me

[Sachin Padma]

landed in r23792

[Bradley Morrison]

Bulk closing of all s60 platform bugs. 

Sorry for the noise!

[Joel Parks]

re-purposing InTSW keyword for use by QtWebkit team