If we remove a search engine from our prepopulate data, remove it from the us...

If we remove a search engine from our prepopulate data, remove it from the user's list too if they haven't edited it or made it default.

BUG=none
TEST=Covered by unittest

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=30508

[sky, 2009-10-29 19:46:36]

http://codereview.chromium.org/339071/diff/1/3
File chrome/browser/search_engines/template_url_model.cc (right):

http://codereview.chromium.org/339071/diff/1/3#newcode770
Line 770: Remove(template_url);
I believe this is going to remove all keywords that were generated by Chrome as
the user browsers. You should only remove if prepopulate_id() != 0.

[Peter Kasting, 2009-10-29 20:30:30]

http://codereview.chromium.org/339071/diff/1/3
File chrome/browser/search_engines/template_url_model.cc (right):

http://codereview.chromium.org/339071/diff/1/3#newcode770
Line 770: Remove(template_url);
On 2009/10/29 19:46:36, sky wrote:
> I believe this is going to remove all keywords that were generated by Chrome
as
> the user browsers. You should only remove if prepopulate_id() != 0.

Look at how |id_to_turl| is populated at the top of the function.

[sky, 2009-10-29 20:33:49]

My mistake. LGTM