Bug 14434

Summary: [S60] Video continue to play after closing the page view.
Product: WebKit Reporter: Hong Zhao <hong.a.zhao>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal Keywords: PlatformOnly
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: S60 Emulator   
OS: S60 3rd edition   
Attachments:
Description Flags
Remove the plugin windows if the page is closed and meanwhile the page is the very first one opened inside the Browser.
yongjun.zhang: review-
Update the former patch based on the review notes. yongjun.zhang: review+

Hong Zhao
Reported 2007-06-27 12:11:53 PDT
TSW#: ID: YZHG-73HPPG After closing the page view that has video plugin playback, the audio part of the video is still playing. This happens only when it's very first page. - Steps to reproduce: 1. Directly load the page http://testsuite.nokia-boston.com/content/S60NG_video_plugin/ondemand.html from a newly launched browser, after a few seconds, the video will start playing. Make sure the SK2 is "Close". 2. Press SK2 to close page view. - Actual outcome: Phone goes back to bookmarks view but the audio part of the video playback is still on going. User has to exit browser to stop it. - Expected outcome: Phone goes back to bookmarks view and video playback stops completely.
Attachments
Remove the plugin windows if the page is closed and meanwhile the page is the very first one opened inside the Browser. (6.60 KB, patch)
2007-06-27 12:28 PDT, Hong Zhao
yongjun.zhang: review-
Update the former patch based on the review notes. (5.49 KB, patch)
2007-06-28 11:45 PDT, Hong Zhao
yongjun.zhang: review+
Hong Zhao
Comment 1 2007-06-27 12:28:47 PDT
Created attachment 15275 [details] Remove the plugin windows if the page is closed and meanwhile the page is the very first one opened inside the Browser.
Yongjun Zhang
Comment 2 2007-06-28 11:28:27 PDT
Comment on attachment 15275 [details] Remove the plugin windows if the page is closed and meanwhile the page is the very first one opened inside the Browser. no need to change BrCtrl.h, pls remove RemovePluginWindows and implement the logic in case statment ECommandUnLoadPluginWindows ;
Hong Zhao
Comment 3 2007-06-28 11:45:33 PDT
Created attachment 15294 [details] Update the former patch based on the review notes.
Yongjun Zhang
Comment 4 2007-06-29 12:27:08 PDT
Comment on attachment 15294 [details] Update the former patch based on the review notes. r = me
Yongjun Zhang
Comment 5 2007-06-29 12:28:33 PDT
landed on r23884.
Bradley Morrison
Comment 6 2008-04-09 11:39:40 PDT
Bulk closing of all s60 platform bugs. Sorry for the noise!
Joel Parks
Comment 7 2011-03-21 11:53:32 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.