From: Jiyeon Kim Date: Fri, 28 Jun 2013 08:23:47 +0000 (+0900) Subject: Video doesn't play after repeating play/back in TizenWinset widget X-Git-Tag: submit/tizen_2.2/20130714.131554~80 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2771c0a951d3cd44e43a0eadb08e2a7850825be5;p=framework%2Fweb%2Fwebkit-efl.git Video doesn't play after repeating play/back in TizenWinset widget [Title] video doesn't play after repeating play/back in TizenWinset widget [Problem] N_SE-43410 [Cause] In this case, media element did not suspend and just removed from parent node. At this time, media element coudln't release resource and stay pause. [Solution] Add supend and resume api in insertedInto and removedFrom api for releasing resource Change-Id: Ife3182e4e4553b65ce227cd397adb97c80da09f2 --- diff --git a/Source/WebCore/html/HTMLMediaElement.cpp b/Source/WebCore/html/HTMLMediaElement.cpp index 78fe1d1..72db174 100755 --- a/Source/WebCore/html/HTMLMediaElement.cpp +++ b/Source/WebCore/html/HTMLMediaElement.cpp @@ -537,6 +537,10 @@ Node::InsertionNotificationRequest HTMLMediaElement::insertedInto(ContainerNode* HTMLElement::insertedInto(insertionPoint); if (insertionPoint->inDocument() && !getAttribute(srcAttr).isEmpty() && m_networkState == NETWORK_EMPTY) scheduleLoad(MediaResource); + +#if ENABLE(TIZEN_GSTREAMER_VIDEO) + resume(); +#endif configureMediaControls(); return InsertionDone; } @@ -547,7 +551,11 @@ void HTMLMediaElement::removedFrom(ContainerNode* insertionPoint) LOG(Media, "HTMLMediaElement::removedFromDocument"); configureMediaControls(); if (m_networkState > NETWORK_EMPTY) +#if ENABLE(TIZEN_GSTREAMER_VIDEO) + suspend(PageWillBeSuspended); +#else pause(); +#endif if (m_isFullscreen) exitFullscreen(); }