Fix intermittent searchbar_spec.js test failure

We need to wait for search results before sending new ones, or we end up
skipping over valid results.

Signed-off-by: Chris Lord <chris.lord@collabora.com>
Change-Id: I957873ee8c80ac8c1ed04a33510b3c6b46886360

browser/src/control/Search.js

@@ -11,7 +11,25 @@
 /* global app */
 
 L.Map.include({
+	_onSearch: function() {
+		this.off('search', this._onSearch, this);
+		if (this._pendingSearch) {
+			this.search.bind(this, ...this._pendingSearch)();
+			this._pendingSearch = null;
+		}
+	},
+
 	search: function (text, backward, replaceString,  command, expand) {
+		if (this._docLayer._searchRequested) {
+			// If we're waiting for a search result, wait for the response before
+			// sending a new one.
+			// This reduces traffic and also avoids the bug where fast successive
+			// searches can skip over results.
+			this.on('search', this._onSearch, this);
+			this._pendingSearch = arguments;
+			return;
+		}
+
 		if (backward === undefined) {
 			backward = false;
 		}