| Index: third_party/WebKit/Source/core/html/AutoplayUmaHelper.cpp
|
| diff --git a/third_party/WebKit/Source/core/html/AutoplayUmaHelper.cpp b/third_party/WebKit/Source/core/html/AutoplayUmaHelper.cpp
|
| index b359769b8a007dbf862405f1673040b029a7fa7a..bc6fe41590ece6524fdd9c2b7c6f5c45bd8c768c 100644
|
| --- a/third_party/WebKit/Source/core/html/AutoplayUmaHelper.cpp
|
| +++ b/third_party/WebKit/Source/core/html/AutoplayUmaHelper.cpp
|
| @@ -81,7 +81,7 @@ void AutoplayUmaHelper::didMoveToNewDocument(Document& oldDocument)
|
|
|
| void AutoplayUmaHelper::onVisibilityChangedForVideoMutedPlayMethod(bool isVisible)
|
| {
|
| - if (!isVisible)
|
| + if (!isVisible || !m_videoMutedPlayMethodVisibilityObserver)
|
| return;
|
|
|
| recordVideoAutoplayMutedPlayMethodBecomesVisibleUma(true);
|
| @@ -115,7 +115,7 @@ void AutoplayUmaHelper::handlePlayingEvent()
|
| {
|
| if (m_source == AutoplaySource::Method && m_element->isHTMLVideoElement() && m_element->muted()) {
|
| if (!m_videoMutedPlayMethodVisibilityObserver) {
|
| - m_videoMutedPlayMethodVisibilityObserver = new ElementVisibilityObserver(m_element, WTF::bind(&AutoplayUmaHelper::onVisibilityChangedForVideoMutedPlayMethod, wrapPersistent(this)));
|
| + m_videoMutedPlayMethodVisibilityObserver = new ElementVisibilityObserver(m_element, WTF::bind(&AutoplayUmaHelper::onVisibilityChangedForVideoMutedPlayMethod, wrapWeakPersistent(this)));
|
| m_videoMutedPlayMethodVisibilityObserver->start();
|
| m_element->document().domWindow()->addEventListener(EventTypeNames::unload, this, false);
|
| }
|
|
|