Bug 15225

Summary:

[s60] plugin cannot cancel a stream request it had issued earlier. TSW ID: RKAN-7597VV

Product:

WebKit

Reporter:

Mahesh Kulkarni <maheshk>

Component:

Plug-ins

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED INVALID

Severity:

Critical

CC:

Sachin.Padma

Priority:

Version:

523.x (Safari 3)

Hardware:

S60 Hardware

OS:

S60 3rd edition

URL:

http:// www.youtube.com/

Attachments:

Description	Flags
3.1m Patch	none

Mahesh Kulkarni

Reported 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.

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 attachment proposed patch, testcase, etc.

Mahesh Kulkarni

Comment 1 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.

Eric Seidel (no email)

Comment 2 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?

Mahesh Kulkarni

Comment 3 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.

Bradley Morrison

Comment 4 2008-04-09 11:31:55 PDT

Bulk update of s60 bugs - tagging and closing.

Bradley Morrison

Comment 5 2008-04-09 11:39:37 PDT

Bulk closing of all s60 platform bugs. Sorry for the noise!

Joel Parks

Comment 6 2011-03-21 11:53:38 PDT

re-purposing InTSW keyword for use by QtWebkit team

Note You need to log in before you can comment on or make changes to this bug.