Add missing checks that default search contains search term replacement.

Add missing checks that default search contains search term replacement.


BUG=116952
TEST=TemplateURLServiceTest.*


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

[Peter Kasting, 2012-04-23 18:01:17]

LGTM.  I wonder if we should add some code to the policy checker so that it will
report policy errors with a specified default search URL that does not support
replacement.

[commit-bot: I haz the power, 2012-04-23 18:47:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ivankr@chromium.org/10184009/1

[commit-bot: I haz the power, 2012-04-23 18:47:19]

Can't apply patch for file
chrome/browser/search_engines/template_url_service.cc.
While running patch -p1 --forward --force;
patching file chrome/browser/search_engines/template_url_service.cc
Hunk #1 succeeded at 612 (offset 6 lines).
Hunk #2 FAILED at 620.
1 out of 2 hunks FAILED -- saving rejects to file
chrome/browser/search_engines/template_url_service.cc.rej

[Ivan Korotkov, 2012-04-23 18:47:24]

Makes sense for the sake of transparency for sysadmins. I'm not familiar with
policy-related code, though.

[commit-bot: I haz the power, 2012-04-23 18:56:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ivankr@chromium.org/10184009/3

[Peter Kasting, 2012-04-23 20:13:33]

On 2012/04/23 18:01:17, Peter Kasting wrote:
> LGTM.  I wonder if we should add some code to the policy checker so that it
will
> report policy errors with a specified default search URL that does not support
> replacement.

Ah, as it turns out we already do this.

[commit-bot: I haz the power, 2012-04-23 20:29:12]

Change committed as 133510