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

Side by Side Diff: components/search_engines/template_url_unittest.cc

Issue 2343523002: [TTS] Request Now on Tap data in the Resolve request. (Closed)
Patch Set: Updated comments and rebased only. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 6
7 #include "base/base_paths.h" 7 #include "base/base_paths.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 1672 matching lines...) Expand 10 before | Expand all | Expand 10 after
1683 TemplateURLData data; 1683 TemplateURLData data;
1684 search_terms_data_.set_google_base_url("http://bar/"); 1684 search_terms_data_.set_google_base_url("http://bar/");
1685 data.SetURL("http://bar/_/contextualsearch?" 1685 data.SetURL("http://bar/_/contextualsearch?"
1686 "{google:contextualSearchVersion}" 1686 "{google:contextualSearchVersion}"
1687 "{google:contextualSearchContextData}"); 1687 "{google:contextualSearchContextData}");
1688 1688
1689 TemplateURL url(data); 1689 TemplateURL url(data);
1690 TemplateURLRef::SearchTermsArgs search_terms_args(ASCIIToUTF16("foo")); 1690 TemplateURLRef::SearchTermsArgs search_terms_args(ASCIIToUTF16("foo"));
1691 std::string result = url.url_ref().ReplaceSearchTerms(search_terms_args, 1691 std::string result = url.url_ref().ReplaceSearchTerms(search_terms_args,
1692 search_terms_data_); 1692 search_terms_data_);
1693 EXPECT_EQ("http://bar/_/contextualsearch?ctxsl_resolve=1", result); 1693 EXPECT_EQ("http://bar/_/contextualsearch?ctxsl_coca=0", result);
1694 1694
1695 TemplateURLRef::SearchTermsArgs::ContextualSearchParams params( 1695 TemplateURLRef::SearchTermsArgs::ContextualSearchParams params(
1696 1, 6, 11, "allen", "woody+allen+movies", "www.wikipedia.org", 1696 1, 6, 11, "allen", "woody+allen+movies", "www.wikipedia.org",
1697 "utf-8", true); 1697 "utf-8", 1);
1698 search_terms_args.contextual_search_params = params; 1698 search_terms_args.contextual_search_params = params;
1699 result = url.url_ref().ReplaceSearchTerms(search_terms_args, 1699 result = url.url_ref().ReplaceSearchTerms(search_terms_args,
1700 search_terms_data_); 1700 search_terms_data_);
1701 EXPECT_EQ("http://bar/_/contextualsearch?" 1701 EXPECT_EQ("http://bar/_/contextualsearch?"
1702 "ctxs=1&" 1702 "ctxs=1&"
1703 "ctxs_start=6&" 1703 "ctxs_start=6&"
1704 "ctxs_end=11&" 1704 "ctxs_end=11&"
1705 "q=allen&" 1705 "q=allen&"
1706 "ctxs_content=woody+allen+movies&" 1706 "ctxs_content=woody+allen+movies&"
1707 "ctxsl_url=www.wikipedia.org&" 1707 "ctxsl_url=www.wikipedia.org&"
1708 "ctxs_encoding=utf-8&" 1708 "ctxs_encoding=utf-8&"
1709 "ctxsl_resolve=1", 1709 "ctxsl_coca=1",
1710 result); 1710 result);
1711 } 1711 }
1712 1712
1713 TEST_F(TemplateURLTest, GenerateKeyword) { 1713 TEST_F(TemplateURLTest, GenerateKeyword) {
1714 ASSERT_EQ(ASCIIToUTF16("foo"), 1714 ASSERT_EQ(ASCIIToUTF16("foo"),
1715 TemplateURL::GenerateKeyword(GURL("http://foo"))); 1715 TemplateURL::GenerateKeyword(GURL("http://foo")));
1716 // www. should be stripped. 1716 // www. should be stripped.
1717 ASSERT_EQ(ASCIIToUTF16("foo"), 1717 ASSERT_EQ(ASCIIToUTF16("foo"),
1718 TemplateURL::GenerateKeyword(GURL("http://www.foo"))); 1718 TemplateURL::GenerateKeyword(GURL("http://www.foo")));
1719 // Make sure we don't get a trailing '/'. 1719 // Make sure we don't get a trailing '/'.
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
1854 GURL("https://www.foo.org/search?q=Y+Z"), 1854 GURL("https://www.foo.org/search?q=Y+Z"),
1855 search_terms_data_, &search_terms)); 1855 search_terms_data_, &search_terms));
1856 EXPECT_EQ(base::ASCIIToUTF16("Y Z"), search_terms); 1856 EXPECT_EQ(base::ASCIIToUTF16("Y Z"), search_terms);
1857 EXPECT_TRUE(url.ExtractSearchTermsFromURL( 1857 EXPECT_TRUE(url.ExtractSearchTermsFromURL(
1858 GURL("https://www.foo.org/s#q=123"), 1858 GURL("https://www.foo.org/s#q=123"),
1859 search_terms_data_, &search_terms)); 1859 search_terms_data_, &search_terms));
1860 EXPECT_EQ(base::ASCIIToUTF16("123"), search_terms); 1860 EXPECT_EQ(base::ASCIIToUTF16("123"), search_terms);
1861 1861
1862 search_terms_data_.set_google_base_url("http://www.google.com/"); 1862 search_terms_data_.set_google_base_url("http://www.google.com/");
1863 } 1863 }
OLDNEW
« components/search_engines/template_url.cc ('K') | « components/search_engines/template_url.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698