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

Unified Diff: Source/devtools/front_end/sources/SourcesSearchScope.js

Issue 400673002: DevTools: Fix dirty files handling in search across all files. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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 | « LayoutTests/http/tests/inspector/search/sources-search-scope-in-files-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/sources/SourcesSearchScope.js
diff --git a/Source/devtools/front_end/sources/SourcesSearchScope.js b/Source/devtools/front_end/sources/SourcesSearchScope.js
index d4940cf310170ae1817fa7d3333e9c62a0f9d010..69b6774649431c38c0b187c79a324df430d91dde 100644
--- a/Source/devtools/front_end/sources/SourcesSearchScope.js
+++ b/Source/devtools/front_end/sources/SourcesSearchScope.js
@@ -158,7 +158,7 @@ WebInspector.SourcesSearchScope.prototype = {
files = files.sort(String.naturalOrderComparator);
files = files.intersectOrdered(filesMathingFileQuery, String.naturalOrderComparator);
var dirtyFiles = this._projectFilesMatchingFileQuery(project, this._searchConfig, true);
- files = files.mergeOrdered(dirtyFiles);
+ files = files.mergeOrdered(dirtyFiles, String.naturalOrderComparator);
if (!files.length) {
progress.done();
« no previous file with comments | « LayoutTests/http/tests/inspector/search/sources-search-scope-in-files-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698