Bug 13648

Summary: [S60] Open to viewer option doesn't work in browser in video content
Product: WebKit Reporter: Hong Zhao <hong.a.zhao>
Component: Plug-insAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal Keywords: PlatformOnly
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: S60 Hardware   
OS: S60 3rd edition   
Attachments:
Description Flags
3.1m: Save the rtsp link to the plugin window.
Sachin.Padma: review-
Save the selected rtsp link to plugin window in 3.1m.
Sachin.Padma: review-
Save rtsp link to plugin window in CCB.
Sachin.Padma: review-
Set rtsp url to top url in plugin window in 3.1m
Sachin.Padma: review-
Merge the comments from review into solution in 3.1m.
Sachin.Padma: review+
Merge review comments into solution. Sachin.Padma: review+

Description Hong Zhao 2007-05-09 13:17:51 PDT
TSW# ID:  SLON-72V9HL  

1) Browse to http://testsuite.nokiaboston.com/content/S60NG_Video_Plugin/default.html 

2) Choose 1. Video Plugin / 2. Object stream
--> Embedded video starts

3) Select Video object

4) From option menu select Open to viewer

-> It doesn't have any effect. Video isn't opened to real one player.
Comment 1 Hong Zhao 2007-05-09 13:33:55 PDT
Created attachment 14439 [details]
3.1m: Save the rtsp link to the plugin window.
Comment 2 Sachin Padma 2007-05-10 07:27:57 PDT
Comment on attachment 14439 [details]
3.1m: Save the rtsp link to the plugin window. 

iTopUrl should be saved at same place for rtsp and nonrtsp urls
Comment 3 Hong Zhao 2007-05-10 08:10:27 PDT
Created attachment 14467 [details]
Save the selected rtsp link to plugin window in 3.1m.
Comment 4 Sachin Padma 2007-05-10 08:21:42 PDT
Comment on attachment 14467 [details]
Save the selected rtsp link to plugin window in 3.1m.

iTopUrl should be set to null after it is deleted. Also function name should be changed to SetTopUrl.
Comment 5 Hong Zhao 2007-05-10 08:34:31 PDT
Created attachment 14469 [details]
Save rtsp link to plugin window in CCB.
Comment 6 Hong Zhao 2007-05-10 08:48:34 PDT
Created attachment 14470 [details]
Set rtsp url to top url in plugin window in 3.1m
Comment 7 Sachin Padma 2007-05-10 12:55:25 PDT
Comment on attachment 14469 [details]
Save rtsp link to plugin window in CCB.

SetTopUrl is a leaving function so should be an L function
Comment 8 Hong Zhao 2007-05-10 19:17:42 PDT
Created attachment 14481 [details]
Merge the  comments from review into solution in 3.1m.
Comment 9 Hong Zhao 2007-05-10 19:19:04 PDT
Created attachment 14482 [details]
Merge review comments into solution.
Comment 10 Sachin Padma 2007-05-14 08:55:19 PDT
Comment on attachment 14481 [details]
Merge the  comments from review into solution in 3.1m.

r=me
Comment 11 Sachin Padma 2007-05-14 08:55:40 PDT
Comment on attachment 14482 [details]
Merge review comments into solution.

r=me
Comment 12 Sachin Padma 2007-05-14 08:56:59 PDT
landed in ccb in r21457 and in 3.1m in r21458
Comment 13 Bradley Morrison 2008-04-09 11:39:27 PDT
Bulk closing of all s60 platform bugs. 

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