Bug 15094

Summary: [S60] OfferKeyEventL on Browser Control causes KERN-EXEC 3
Product: WebKit Reporter: Misha <mtutunik>
Component: WebKit APIAssignee: Misha <mtutunik>
Status: CLOSED INVALID    
Severity: Critical Keywords: PlatformOnly
Priority: P1    
Version: 523.x (Safari 3)   
Hardware: S60 Hardware   
OS: S60 3rd edition   
Attachments:
Description Flags
Patch file for PluginWin.cpp
Sachin.Padma: review-
Patch Sachin.Padma: review+

Description Misha 2007-08-27 13:06:53 PDT
iBrCtlInterface->OfferKeyEventL( aKeyEvent, aType ); with parameters aType
EEventKeyUp and aKeyEvent.iScanCode 167 (EStdKeyDevice3) and keyEvent.iCode =
EKeyDevice3 causes KERN-EXEC 3.

Steps to reproduce:

1. 	Start BrCtl.
2.	A Flash file is playing within a XHTML page. 
4.	Press the CENTER key to give the Flash object focus.
5.	Observe that if you use the Up and Down arrow keys the Flash object is not scrolling within the XHTML page. Conclusion: The Flash object has focus.
6.	Press the application key to return to the shell.
7.	Hold the application key to list the running applications and select BrCtl using the CENTER key.

Actual outcome

BrCtl has closed.

Expected outcome

BrCtl is still in the list of running applications.
Comment 1 Misha 2007-08-27 13:10:15 PDT
Created attachment 16136 [details]
Patch file for PluginWin.cpp

If BrCtl is created without cursor capability WebKitCursor is NULL. The fix includes NULL check for WebKitCursor in PluginWin.
Comment 2 Sachin Padma 2007-08-31 11:31:43 PDT
Comment on attachment 16136 [details]
Patch file for PluginWin.cpp

This should be modified so that in Tab navigation mode we do not notify plugins of the position and activated and deactivated cases.
Comment 3 Misha 2007-09-04 09:05:08 PDT
Created attachment 16196 [details]
Patch
Comment 4 Sachin Padma 2007-09-06 06:59:18 PDT
Comment on attachment 16196 [details]
Patch

r=me
Comment 5 Bradley Morrison 2008-04-09 11:31:55 PDT
Bulk update of s60 bugs - tagging and closing.
Comment 6 Bradley Morrison 2008-04-09 11:39:37 PDT
Bulk closing of all s60 platform bugs. 

Sorry for the noise!
Comment 7 Joel Parks 2011-03-21 11:53:37 PDT
re-purposing InTSW keyword for use by QtWebkit team