Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1528)

Unified Diff: Source/devtools/front_end/components/SearchableView.js

Issue 669043002: DevTools: Fix the way search works when switching between editor tabs. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/devtools/front_end/source_frame/CodeMirrorTextEditor.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/components/SearchableView.js
diff --git a/Source/devtools/front_end/components/SearchableView.js b/Source/devtools/front_end/components/SearchableView.js
index 43af570b31df68f33ae58f53d08e43d62da09a2b..0500ba88f80851680bdd36eac4dfa0d6b5cba3aa 100644
--- a/Source/devtools/front_end/components/SearchableView.js
+++ b/Source/devtools/front_end/components/SearchableView.js
@@ -207,14 +207,14 @@ WebInspector.SearchableView.prototype = {
{
this._caseSensitiveButton.toggled = !this._caseSensitiveButton.toggled;
this._saveSetting();
- this._performSearch(true, true);
+ this._performSearch(false, true);
},
_toggleRegexSearch: function()
{
this._regexButton.toggled = !this._regexButton.toggled;
this._saveSetting();
- this._performSearch(true, true);
+ this._performSearch(false, true);
},
/**
@@ -365,6 +365,16 @@ WebInspector.SearchableView.prototype = {
this._matchesElement.textContent = "";
},
+ refreshSearch: function()
+ {
+ if (!this._searchIsVisible)
+ return;
+ this._clearSearch();
lushnikov 2014/10/22 11:12:37 these three lines are "resetSearch()" essentially
+ this._updateReplaceVisibility();
+ this._matchesElement.textContent = "";
+ this._performSearch(false, false);
+ },
+
/**
* @return {boolean}
*/
« no previous file with comments | « no previous file | Source/devtools/front_end/source_frame/CodeMirrorTextEditor.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698