CC AnimationHost: Fix LinkHighlights detach scope.

third_party/WebKit/Source/web/WebViewImpl.cpp

Index: third_party/WebKit/Source/web/WebViewImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebViewImpl.cpp b/third_party/WebKit/Source/web/WebViewImpl.cpp
index 2a1421fcebcdfe88fdb3f5540a407512fabf53ca..0114670454c6b84d1525cafc0230ac5216c6b476 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.cpp
+++ b/third_party/WebKit/Source/web/WebViewImpl.cpp
@@ -2681,6 +2681,7 @@ bool WebViewImpl::isAcceleratedCompositingActive() const
 void WebViewImpl::willCloseLayerTreeView()
 {
     if (m_linkHighlightsTimeline) {
+        m_linkHighlights.clear();

[pdr., 2015/11/04 02:53:02]

Can you help me understand this line? Why do we only clear m_linkHighlights if
there's a highlight timeline?

[loyso (OOO), 2015/11/04 04:16:30]

Ermmm. We can clear m_linkHighlights in any case, definitely.

Currently two compositor animation systems co-exist in cc.
m_linkHighlightsTimeline presence indicates that we are running in "external
AnimatioHost mode (enable-compositor-animationtimelines).

So i just wanted to keep those behaviors isolated.

         detachCompositorAnimationTimeline(m_linkHighlightsTimeline.get());
         m_linkHighlightsTimeline.clear();
     }