CLOSED FIXED 14069
[S60]: Plug-in request for execution of JavaScript through NpnGetUrl API not working
https://bugs.webkit.org/show_bug.cgi?id=14069
Summary [S60]: Plug-in request for execution of JavaScript through NpnGetUrl API not ...
Mahesh Kulkarni
Reported 2007-06-11 06:56:09 PDT
A scriptable plug-in can request for a page to load through NPN_GetURL . It can also execute javascript using same API by sending javascript: in the url followed by script. This API doesnt seems to be responding for simple javascript alert. - Use case: http://waplabdc.nokia-boston.com/browser/users/MaheshKK/geturl.html - Steps to reproduce: 1) Open the link: http://waplabdc.nokia-boston.com/browser/users/MaheshKK/geturl.html 2) Select flash 2) Click on "Enter" button - Actual outcome: Nothing happens after clicking on Enter. - Expected outcome: It should display a alert message "Hello"
Attachments
Resolve URL bug (772 bytes, patch)
2007-06-11 07:00 PDT, Mahesh Kulkarni
Sachin.Padma: review-
Attached ChangeLog (1.48 KB, patch)
2007-06-14 07:01 PDT, Mahesh Kulkarni
Sachin.Padma: review+
Mahesh Kulkarni
Comment 1 2007-06-11 07:00:59 PDT
Created attachment 14935 [details] Resolve URL bug _REASON: When "javascript:" is come in requested URL, browser tries to resolve that URL with base URL. This process encodes special characters (in this case double quotes). Then URL is sent to javascript execution. Javascript engine fails to understand the content as URL has encoded script. _SOLUTION: When "javascript:" is present in the URL, do not resolve URL. Even if javascript makes a requests for a URL then that will be taken care by script engine. _MODULE_EFFECTED: webKit/plugin plugin_loader _TSW ERR_ID: MKUI-73GEH4
Sachin Padma
Comment 2 2007-06-13 06:56:13 PDT
Comment on attachment 14935 [details] Resolve URL bug Please attach changelog
Mahesh Kulkarni
Comment 3 2007-06-14 07:01:05 PDT
Created attachment 15020 [details] Attached ChangeLog Attached changeLog. With the fix of not resolving URL
Sachin Padma
Comment 4 2007-06-15 10:36:16 PDT
Comment on attachment 15020 [details] Attached ChangeLog r=me. In future please attach changelog to the beginning of the file
Sachin Padma
Comment 5 2007-06-15 10:39:27 PDT
landed in ccb in r23548 and in 3.1m in r23549
Bradley Morrison
Comment 6 2008-04-09 11:39:25 PDT
Bulk closing of all s60 platform bugs. Sorry for the noise!
Note You need to log in before you can comment on or make changes to this bug.