Omnibox - Strip Extra Whitespace from Custom Search Engine Names
Whitespace in the form of tabs and line feeds and the like can screw
things up.
This fixes the fresh.amazon.com bug. (I tested it interatively.)
By the way, there is no need to do a migration pass.
The data will be correctly loaded when read from the
database. (I checked.) While it's true the database
will be out of date until written again, given that
the actual data used is correct, I don't think that's
an issue.
TBR=rdevlin.cronin,isherman,lazyboy,pvalenzuela,kmadhusu,avi
rdevlin.cronin - this trivial template URL data API change required some changes in chrome/browser/extensions
isherman - this trivial template URL data API change required some changes in chrome/browser/importer
lazyboy - this trivial template URL data API change required some changes in chrome/browser/renderer_context_menu
pvalenzuela - this trivial template URL data API change required some changes in chrome/browser/sync/test/integration
kmadhusu - this trivial template URL data API change required some changes in chrome/browser/ui/search/
avi - this trivial template URL data API change required some changes in chrome/browser/ui/cocoa/browser/
BUG=485357
Review URL: https://codereview.chromium.org/1135163002
Cr-Commit-Position: refs/heads/master@{#329742}
diff --git a/chrome/browser/chrome_content_browser_client_unittest.cc b/chrome/browser/chrome_content_browser_client_unittest.cc
index 6d37b76..6b35cb94 100644
--- a/chrome/browser/chrome_content_browser_client_unittest.cc
+++ b/chrome/browser/chrome_content_browser_client_unittest.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/metrics/field_trial.h"
+#include "base/strings/utf_string_conversions.h"
#include "chrome/browser/search_engines/template_url_service_factory.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/tabs/tab_strip_model.h"
@@ -163,6 +164,7 @@
ui_test_utils::WaitForTemplateURLServiceToLoad(template_url_service);
TemplateURLData data;
+ data.SetShortName(base::ASCIIToUTF16("foo.com"));
data.SetURL("http://foo.com/url?bar={searchTerms}");
data.new_tab_url = new_tab_page_url.spec();
TemplateURL* template_url = new TemplateURL(data);
