[result database] Empty search will return too many tests to display

https://bugs.webkit.org/show_bug.cgi?id=213898

Reviewed by Jonathan Bedard.

* resultsdbpy/resultsdbpy/view/static/js/search.js: Trim the search string before fetching the tests, ingore empty string on paste event


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

Author: [email protected]

Tools/ChangeLog

@@ -1,3 +1,12 @@
+2020-07-06  Zhifei Fang  <[email protected]>
+
+        [result database] Empty search will return too many tests to display
+        https://bugs.webkit.org/show_bug.cgi?id=213898
+
+        Reviewed by Jonathan Bedard.
+
+        * resultsdbpy/resultsdbpy/view/static/js/search.js: Trim the search string before fetching the tests, ingore empty string on paste event
+
 2020-07-06  Geoffrey Garen  <[email protected]>
 
         RunLoop::dispatch should only call wakeUp when needed

Tools/resultsdbpy/resultsdbpy/view/static/js/search.js

@@ -149,12 +149,14 @@ function SearchBar(callback, suites) {
                 if (event.keyCode == 0x28 /* DOM_VK_DOWN */ || event.keyCode == 0x26 /* DOM_VK_UP */)
                     return;
 
+                const inputValue = element.value.trimStart().trimEnd();
+                if (!inputValue)
+                    return;
                 let myDispatch = Date.now();
-
                 candidates = {};
                 suites.forEach(suite => {
                     inFlight += 1;
-                    fetch(`api/${suite}/tests?limit=${LIMIT}&test=${element.value}`).then(response => {
+                    fetch(`api/${suite}/tests?limit=${LIMIT}&test=${inputValue}`).then(response => {
                         inFlight -= 1;
                         if (myDispatch < currentDispatch) {
                             candidatesRef.setState({});
@@ -215,7 +217,8 @@ function SearchBar(callback, suites) {
                             ignoring = false;
                         return;
                     }
-                    tests.add(str);
+                    if (str)
+                        tests.add(str);
                 });
                 tests = [...tests];
                 if (tests.length <= 1)