Bug 105906 - [BlackBerry] Handle re-entrant destruction of MediaPlayerPrivate while a dialog is up
Summary: [BlackBerry] Handle re-entrant destruction of MediaPlayerPrivate while a dial...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Max Feil
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-01 22:04 PST by Max Feil
Modified: 2013-01-04 09:31 PST (History)
8 users (show)

See Also:


Attachments
Patch (4.85 KB, patch)
2013-01-01 22:14 PST, Max Feil
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Max Feil 2013-01-01 22:04:13 PST
Now that dialogs are not truly modal anymore, we need to handle the case where MediaPlayerPrivate gets destroyed by re-entrant calls while a dialog is up. The re-entrant calls come via the nested event handler that is set up by the dialog code. This nested event handler does defer page loads, but this is not enough to ensure that MediaPlayerPrivate objects don't get destroyed. They can be destroyed by simply changing the source of an audio or video element, for example in a canplaythrough event handler.

This patch covers only the WebCore side of the changes required. Most of the work is in the platform repository. See PR272949.
Comment 1 Max Feil 2013-01-01 22:14:58 PST
Created attachment 181008 [details]
Patch
Comment 2 Max Feil 2013-01-01 22:17:03 PST
Comment on attachment 181008 [details]
Patch

BlackBerry Specific patch, already reviewed by George Staikos.
Comment 3 WebKit Review Bot 2013-01-01 22:20:33 PST
Comment on attachment 181008 [details]
Patch

Rejecting attachment 181008 [details] from review queue.

mfeil@rim.com does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your reviewer rights.
Comment 4 WebKit Review Bot 2013-01-01 22:21:10 PST
Comment on attachment 181008 [details]
Patch

Rejecting attachment 181008 [details] from commit-queue.

mfeil@rim.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 5 Max Feil 2013-01-01 22:22:22 PST
Comment on attachment 181008 [details]
Patch

Alrighty then.
Comment 6 Max Feil 2013-01-03 16:25:50 PST
I'd like to upstream this change, which is already reviewed and landed downstream...
Comment 7 WebKit Review Bot 2013-01-04 09:31:28 PST
Comment on attachment 181008 [details]
Patch

Clearing flags on attachment: 181008

Committed r138805: <http://trac.webkit.org/changeset/138805>
Comment 8 WebKit Review Bot 2013-01-04 09:31:32 PST
All reviewed patches have been landed.  Closing bug.