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-ins

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Normal

Keywords:

PlatformOnly

Priority:

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+

Hong Zhao

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

Attachments
3.1m: Save the rtsp link to the plugin window. (2.63 KB, patch) 2007-05-09 13:33 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Save the selected rtsp link to plugin window in 3.1m. (2.74 KB, patch) 2007-05-10 08:10 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Save rtsp link to plugin window in CCB. (3.35 KB, patch) 2007-05-10 08:34 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Set rtsp url to top url in plugin window in 3.1m (2.80 KB, patch) 2007-05-10 08:48 PDT, Hong Zhao	Sachin.Padma: review-	Details Formatted Diff Diff
Merge the comments from review into solution in 3.1m. (2.78 KB, patch) 2007-05-10 19:17 PDT, Hong Zhao	Sachin.Padma: review+	Details Formatted Diff Diff
Merge review comments into solution. (3.34 KB, patch) 2007-05-10 19:19 PDT, Hong Zhao	Sachin.Padma: review+	Details Formatted Diff Diff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.

Hong Zhao

Comment 1 2007-05-09 13:33:55 PDT

Created attachment 14439 [details] 3.1m: Save the rtsp link to the plugin window.

Sachin Padma

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

Hong Zhao

Comment 3 2007-05-10 08:10:27 PDT

Created attachment 14467 [details] Save the selected rtsp link to plugin window in 3.1m.

Sachin Padma

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

Hong Zhao

Comment 5 2007-05-10 08:34:31 PDT

Created attachment 14469 [details] Save rtsp link to plugin window in CCB.

Hong Zhao

Comment 6 2007-05-10 08:48:34 PDT

Created attachment 14470 [details] Set rtsp url to top url in plugin window in 3.1m

Sachin Padma

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

Hong Zhao

Comment 8 2007-05-10 19:17:42 PDT

Created attachment 14481 [details] Merge the comments from review into solution in 3.1m.

Hong Zhao

Comment 9 2007-05-10 19:19:04 PDT

Created attachment 14482 [details] Merge review comments into solution.

Sachin Padma

Comment 10 2007-05-14 08:55:19 PDT

Comment on attachment 14481 [details] Merge the comments from review into solution in 3.1m. r=me

Sachin Padma

Comment 11 2007-05-14 08:55:40 PDT

Comment on attachment 14482 [details] Merge review comments into solution. r=me

Sachin Padma

Comment 12 2007-05-14 08:56:59 PDT

landed in ccb in r21457 and in 3.1m in r21458

Bradley Morrison

Comment 13 2008-04-09 11:39:27 PDT

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

Joel Parks

Comment 14 2011-03-21 11:53:26 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.