Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(821)

Unified Diff: components/search_engines/template_url_unittest.cc

Issue 2343523002: [TTS] Request Now on Tap data in the Resolve request. (Closed)
Patch Set: Responded to Peters comments: inlined a conversion and a few other nits. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/search_engines/template_url.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/search_engines/template_url_unittest.cc
diff --git a/components/search_engines/template_url_unittest.cc b/components/search_engines/template_url_unittest.cc
index e0a413a5684344dd2854eb9744d362e9dabf44fa..672579bf4746ca340abf256bc23d33c75d3b3a12 100644
--- a/components/search_engines/template_url_unittest.cc
+++ b/components/search_engines/template_url_unittest.cc
@@ -1690,24 +1690,24 @@ TEST_F(TemplateURLTest, ContextualSearchParameters) {
TemplateURLRef::SearchTermsArgs search_terms_args(ASCIIToUTF16("foo"));
std::string result = url.url_ref().ReplaceSearchTerms(search_terms_args,
search_terms_data_);
- EXPECT_EQ("http://bar/_/contextualsearch?ctxsl_resolve=1", result);
+ EXPECT_EQ("http://bar/_/contextualsearch?ctxsl_coca=0", result);
TemplateURLRef::SearchTermsArgs::ContextualSearchParams params(
- 1, 6, 11, "allen", "woody+allen+movies", "www.wikipedia.org",
- "utf-8", true);
+ 1, 6, 11, "allen", "woody+allen+movies", "www.wikipedia.org", "utf-8", 1);
search_terms_args.contextual_search_params = params;
result = url.url_ref().ReplaceSearchTerms(search_terms_args,
search_terms_data_);
- EXPECT_EQ("http://bar/_/contextualsearch?"
- "ctxs=1&"
- "ctxs_start=6&"
- "ctxs_end=11&"
- "q=allen&"
- "ctxs_content=woody+allen+movies&"
- "ctxsl_url=www.wikipedia.org&"
- "ctxs_encoding=utf-8&"
- "ctxsl_resolve=1",
- result);
+ EXPECT_EQ(
+ "http://bar/_/contextualsearch?"
+ "ctxs=1&"
+ "ctxs_start=6&"
+ "ctxs_end=11&"
+ "q=allen&"
+ "ctxs_content=woody+allen+movies&"
+ "ctxsl_url=www.wikipedia.org&"
+ "ctxs_encoding=utf-8&"
+ "ctxsl_coca=1",
+ result);
}
TEST_F(TemplateURLTest, GenerateKeyword) {
« no previous file with comments | « components/search_engines/template_url.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698