Title: [117080] trunk/Source/WebCore
Revision
117080
Author
[email protected]
Date
2012-05-15 09:13:05 -0700 (Tue, 15 May 2012)

Log Message

Web Inspector: AdvancedSearchController is not stopped then view is hidden.
https://bugs.webkit.org/show_bug.cgi?id=86466

Patch by Eugene Klyuchnikov <[email protected]> on 2012-05-15
Reviewed by Vsevolod Vlasov.

AdvancedSearchController overrides "wasHidden" method instead of "willHide".

UI event-flow, not tested.

* inspector/front-end/AdvancedSearchController.js:
(WebInspector.SearchView.prototype.willHide): Renamed function.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (117079 => 117080)


--- trunk/Source/WebCore/ChangeLog	2012-05-15 16:05:23 UTC (rev 117079)
+++ trunk/Source/WebCore/ChangeLog	2012-05-15 16:13:05 UTC (rev 117080)
@@ -1,3 +1,17 @@
+2012-05-15  Eugene Klyuchnikov  <[email protected]>
+
+        Web Inspector: AdvancedSearchController is not stopped then view is hidden.
+        https://bugs.webkit.org/show_bug.cgi?id=86466
+
+        Reviewed by Vsevolod Vlasov.
+
+        AdvancedSearchController overrides "wasHidden" method instead of "willHide".
+
+        UI event-flow, not tested.
+
+        * inspector/front-end/AdvancedSearchController.js:
+        (WebInspector.SearchView.prototype.willHide): Renamed function.
+
 2012-05-15  Jocelyn Turcotte  <[email protected]>
 
         [Qt] http/tests/xmlhttprequest/xmlhttprequest-check-head-readystate-for-404.html times out

Modified: trunk/Source/WebCore/inspector/front-end/AdvancedSearchController.js (117079 => 117080)


--- trunk/Source/WebCore/inspector/front-end/AdvancedSearchController.js	2012-05-15 16:05:23 UTC (rev 117079)
+++ trunk/Source/WebCore/inspector/front-end/AdvancedSearchController.js	2012-05-15 16:13:05 UTC (rev 117080)
@@ -356,7 +356,7 @@
         this.focus();
     },
 
-    wasHidden: function()
+    willHide: function()
     {
         this._controller.stopSearch();
     },
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to