Bug 14434 - [S60] Video continue to play after closing the page view.
Summary: [S60] Video continue to play after closing the page view.
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 523.x (Safari 3)
Hardware: S60 Emulator S60 3rd edition
: P2 Normal
Assignee: Nobody
URL:
Keywords: PlatformOnly
Depends on:
Blocks:
 
Reported: 2007-06-27 12:11 PDT by Hong Zhao
Modified: 2011-03-21 11:53 PDT (History)
0 users

See Also:


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-
Details | Formatted Diff | Diff
Update the former patch based on the review notes. (5.49 KB, patch)
2007-06-28 11:45 PDT, Hong Zhao
yongjun.zhang: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hong Zhao 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.
Comment 1 Hong Zhao 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.
Comment 2 Yongjun Zhang 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
;
Comment 3 Hong Zhao 2007-06-28 11:45:33 PDT
Created attachment 15294 [details]
Update the former patch based on the review notes.
Comment 4 Yongjun Zhang 2007-06-29 12:27:08 PDT
Comment on attachment 15294 [details]
Update the former patch based on the review notes. 

r = me
Comment 5 Yongjun Zhang 2007-06-29 12:28:33 PDT
landed on r23884.
Comment 6 Bradley Morrison 2008-04-09 11:39:40 PDT
Bulk closing of all s60 platform bugs. 

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