Bug 15225 - [s60] plugin cannot cancel a stream request it had issued earlier. TSW ID: RKAN-7597VV
Summary: [s60] plugin cannot cancel a stream request it had issued earlier. TSW ID: R...
Status: CLOSED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 523.x (Safari 3)
Hardware: S60 Hardware S60 3rd edition
: P1 Critical
Assignee: Nobody
URL: http:// www.youtube.com/
Keywords:
Depends on:
Blocks:
 
Reported: 2007-09-17 05:49 PDT by Mahesh Kulkarni
Modified: 2011-03-21 11:53 PDT (History)
1 user (show)

See Also:

Attachments
3.1m Patch (2.47 KB, patch)
2007-09-17 06:14 PDT, Mahesh Kulkarni 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mahesh Kulkarni 2007-09-17 05:49:47 PDT 
TSWID: RKAN-7597VV

- Use case: 

Plugin cannot cancel a stream request it had issued earlier. 
As returning error code from write stream cancels all transaction and deletes plugin.

- Actual outcome: Borwser deletes the plugin when NPP_Write returns with KErrCancel. 

- Expected outcome: Browser should not delete the plugin for cancel stream request from plugin. Instead browser should just cancel the transaction which is returned error.
Comment 1 Mahesh Kulkarni 2007-09-17 06:14:08 PDT 
Created attachment 16306 [details]
3.1m Patch

Do not cancel all transaction and delete plugin. Instead just cancel the requested transaction which is got from returning error code from plugin NPP_Write.
Comment 2 Eric Seidel (no email) 2008-01-05 19:55:31 PST 
It's now 4 months since this patch was marked as review.  It would be nice to get it out of the review queue.  Sachin?
Comment 3 Mahesh Kulkarni 2008-01-06 21:31:26 PST 
This change has been reviewed but i think missed out to update here. No need for review. Making patch obsolete.
Comment 4 Bradley Morrison 2008-04-09 11:31:55 PDT 
Bulk update of s60 bugs - tagging and closing.
Comment 5 Bradley Morrison 2008-04-09 11:39:37 PDT 
Bulk closing of all s60 platform bugs. 

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