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

Issue 29529787: Issue 4655 - Removing an installed search engine replaces it with a new search option (Closed)

Created:
Aug. 28, 2017, 8 p.m. by diegocarloslima
Modified:
Sept. 13, 2017, 6:26 p.m.
Reviewers:
anton, jens
CC:
René Jeschke, Felix Dahlke
Visibility:
Public.

Description

Issue 4655 - Removing an installed search engine replaces it with a new search option

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2 lines) Patch
M toolkit/components/search/nsSearchService.js View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 3
diegocarloslima
Aug. 28, 2017, 8:01 p.m. (2017-08-28 20:01:23 UTC) #1
jens
On 2017/08/28 20:01:23, diegocarloslima wrote: LGTM
Aug. 29, 2017, 12:19 p.m. (2017-08-29 12:19:05 UTC) #2
anton
Sept. 1, 2017, 5:07 a.m. (2017-09-01 05:07:29 UTC) #3
On 2017/08/28 20:01:23, diegocarloslima wrote:

LGTM

Powered by Google App Engine
This is Rietveld