ASSERT(!m_findPageOverlay) in FindController.cpp after r140769.
authorenrica@apple.com <enrica@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Feb 2013 00:47:03 +0000 (00:47 +0000)
committerenrica@apple.com <enrica@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Feb 2013 00:47:03 +0000 (00:47 +0000)
https://bugs.webkit.org/show_bug.cgi?id=109105.

Reviewed by Tim Horton.

In r140769 we changed the way the overlay is destroyed,
therefore the assert is no longer valid and should be removed.

* WebProcess/WebPage/FindController.cpp:
(WebKit::FindController::updateFindUIAfterPageScroll):

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

Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/WebPage/FindController.cpp

index f96a365..ddc4ad9 100644 (file)
@@ -1,3 +1,16 @@
+2013-02-06  Enrica Casucci  <enrica@apple.com>
+
+        ASSERT(!m_findPageOverlay) in FindController.cpp after r140769.
+        https://bugs.webkit.org/show_bug.cgi?id=109105.
+
+        Reviewed by Tim Horton.
+
+        In r140769 we changed the way the overlay is destroyed,
+        therefore the assert is no longer valid and should be removed.
+
+        * WebProcess/WebPage/FindController.cpp:
+        (WebKit::FindController::updateFindUIAfterPageScroll):
+
 2013-02-06  Anders Carlsson  <andersca@apple.com>
 
         Dispatch storage manager messages to the storage work queue
index f3feccd..704183d 100644 (file)
@@ -164,9 +164,7 @@ void FindController::updateFindUIAfterPageScroll(bool found, const String& strin
         if (m_findPageOverlay) {
             // Get rid of the overlay.
             m_webPage->uninstallPageOverlay(m_findPageOverlay, true);
-        }
-        
-        ASSERT(!m_findPageOverlay);
+        }        
     } else {
         if (!m_findPageOverlay) {
             RefPtr<PageOverlay> findPageOverlay = PageOverlay::create(this);