Changeset 142055 in webkit

Timestamp:

Feb 6, 2013, 4:47:03 PM (12 years ago)

Author:

enrica@apple.com

Message:

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):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/WebPage/FindController.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+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>
 

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

@@ -165 +165 @@
             // Get rid of the overlay.
             m_webPage->uninstallPageOverlay(m_findPageOverlay, true);
-        }
-        
-        ASSERT(!m_findPageOverlay);
+        }        
     } else {
         if (!m_findPageOverlay) {