Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: toolkit/components/search/nsSearchService.js

Issue 29529787: Issue 4655 - Removing an installed search engine replaces it with a new search option (Closed)
Patch Set: Created Aug. 28, 2017, 8 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: toolkit/components/search/nsSearchService.js
===================================================================
--- a/toolkit/components/search/nsSearchService.js
+++ b/toolkit/components/search/nsSearchService.js
@@ -3936,18 +3936,16 @@ SearchService.prototype = {
aCount.value = engines.length;
return engines;
},
getVisibleEngines: function SRCH_SVC_getVisible(aCount) {
this._ensureInitialized();
LOG("getVisibleEngines: getting all visible engines");
var engines = this._getSortedEngines(false);
- // Capped visible engine count at 5 according to https://issues.adblockplus.org/ticket/3128
- engines = engines.slice(0, Math.min(engines.length, 5));
aCount.value = engines.length;
return engines;
},
getDefaultEngines: function SRCH_SVC_getDefault(aCount) {
this._ensureInitialized();
function isDefault(engine) {
return engine._isDefault;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld