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); |
} |