ASSERT(!m_overlay) reached in TapHighlightController.cpp:73

https://bugs.webkit.org/show_bug.cgi?id=77117

Reviewed by Tor Arne Vestbø.

The PageOverlay does only get uninstalled after the fade-out animation finishes,
so do not assert on !m_overlay after the uninstallPageOverlay call.

* WebProcess/WebPage/TapHighlightController.cpp:
(WebKit::TapHighlightController::hideHighlight):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@106012 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebKit2/ChangeLog b/Source/WebKit2/ChangeLog
index f22afe0..82b1542 100644 (file)
--- a/Source/WebKit2/ChangeLog
+++ b/Source/WebKit2/ChangeLog
@@ -1,3 +1,16 @@
+2012-01-26  Andras Becsi  <andras.becsi@nokia.com>
+
+        ASSERT(!m_overlay) reached in TapHighlightController.cpp:73
+        https://bugs.webkit.org/show_bug.cgi?id=77117
+
+        Reviewed by Tor Arne Vestbø.
+
+        The PageOverlay does only get uninstalled after the fade-out animation finishes,
+        so do not assert on !m_overlay after the uninstallPageOverlay call.
+
+        * WebProcess/WebPage/TapHighlightController.cpp:
+        (WebKit::TapHighlightController::hideHighlight):
+
 2012-01-26  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
 
         WebPreferencesStore: use DEFINE_STATIC_LOCAL for overrides HashMap

diff --git a/Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp b/Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp
index 5408107..1c36a4d 100644 (file)
--- a/Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp
+++ b/Source/WebKit2/WebProcess/WebPage/TapHighlightController.cpp
@@ -70,7 +70,6 @@ void TapHighlightController::hideHighlight()
 {
     if (m_overlay)
         m_webPage->uninstallPageOverlay(m_overlay, /* fadeout */ true);
-    ASSERT(!m_overlay);
 }
 
 void TapHighlightController::pageOverlayDestroyed(PageOverlay*)