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

Side by Side Diff: chrome/browser/search_engines/template_url_unittest.cc

Issue 292983003: Use different Access Point on iOS for phone and tablet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 7 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "base/base_paths.h" 5 #include "base/base_paths.h"
6 #include "base/command_line.h" 6 #include "base/command_line.h"
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/browser/browser_process.h" 10 #include "chrome/browser/browser_process.h"
(...skipping 642 matching lines...) Expand 10 before | Expand all | Expand 10 after
653 EXPECT_EQ(test_data[i].expected_result, result.spec()); 653 EXPECT_EQ(test_data[i].expected_result, result.spec());
654 } 654 }
655 } 655 }
656 656
657 TEST_F(TemplateURLTest, RLZ) { 657 TEST_F(TemplateURLTest, RLZ) {
658 base::string16 rlz_string; 658 base::string16 rlz_string;
659 #if defined(ENABLE_RLZ) 659 #if defined(ENABLE_RLZ)
660 std::string brand; 660 std::string brand;
661 if (google_util::GetBrand(&brand) && !brand.empty() && 661 if (google_util::GetBrand(&brand) && !brand.empty() &&
662 !google_util::IsOrganic(brand)) { 662 !google_util::IsOrganic(brand)) {
663 RLZTracker::GetAccessPointRlz(RLZTracker::CHROME_OMNIBOX, &rlz_string); 663 RLZTracker::GetAccessPointRlz(RLZTracker::ChromeOmnibox(), &rlz_string);
664 } 664 }
665 #elif defined(OS_ANDROID) 665 #elif defined(OS_ANDROID)
666 SearchTermsDataAndroid::rlz_parameter_value_.Get() = 666 SearchTermsDataAndroid::rlz_parameter_value_.Get() =
667 ASCIIToUTF16("android_test"); 667 ASCIIToUTF16("android_test");
668 rlz_string = SearchTermsDataAndroid::rlz_parameter_value_.Get(); 668 rlz_string = SearchTermsDataAndroid::rlz_parameter_value_.Get();
669 #endif 669 #endif
670 670
671 TemplateURLData data; 671 TemplateURLData data;
672 data.SetURL("http://bar/?{google:RLZ}{searchTerms}"); 672 data.SetURL("http://bar/?{google:RLZ}{searchTerms}");
673 TemplateURL url(NULL, data); 673 TemplateURL url(NULL, data);
674 EXPECT_TRUE(url.url_ref().IsValid()); 674 EXPECT_TRUE(url.url_ref().IsValid());
675 ASSERT_TRUE(url.url_ref().SupportsReplacement()); 675 ASSERT_TRUE(url.url_ref().SupportsReplacement());
676 GURL result(url.url_ref().ReplaceSearchTerms( 676 GURL result(url.url_ref().ReplaceSearchTerms(
677 TemplateURLRef::SearchTermsArgs(ASCIIToUTF16("x")))); 677 TemplateURLRef::SearchTermsArgs(ASCIIToUTF16("x"))));
678 ASSERT_TRUE(result.is_valid()); 678 ASSERT_TRUE(result.is_valid());
679 std::string expected_url = "http://bar/?"; 679 std::string expected_url = "http://bar/?";
680 if (!rlz_string.empty()) 680 if (!rlz_string.empty())
681 expected_url += "rlz=" + base::UTF16ToUTF8(rlz_string) + "&"; 681 expected_url += "rlz=" + base::UTF16ToUTF8(rlz_string) + "&";
682 expected_url += "x"; 682 expected_url += "x";
683 EXPECT_EQ(expected_url, result.spec()); 683 EXPECT_EQ(expected_url, result.spec());
684 } 684 }
685 685
686 #if !defined(OS_ANDROID) && !defined(OS_IOS) 686 #if !defined(OS_ANDROID) && !defined(OS_IOS)
687 TEST_F(TemplateURLTest, RLZFromAppList) { 687 TEST_F(TemplateURLTest, RLZFromAppList) {
688 base::string16 rlz_string; 688 base::string16 rlz_string;
689 #if defined(ENABLE_RLZ) 689 #if defined(ENABLE_RLZ)
690 std::string brand; 690 std::string brand;
691 if (google_util::GetBrand(&brand) && !brand.empty() && 691 if (google_util::GetBrand(&brand) && !brand.empty() &&
692 !google_util::IsOrganic(brand)) { 692 !google_util::IsOrganic(brand)) {
693 RLZTracker::GetAccessPointRlz(RLZTracker::CHROME_APP_LIST, &rlz_string); 693 RLZTracker::GetAccessPointRlz(RLZTracker::ChromeAppList(), &rlz_string);
694 } 694 }
695 #endif 695 #endif
696 696
697 TemplateURLData data; 697 TemplateURLData data;
698 data.SetURL("http://bar/?{google:RLZ}{searchTerms}"); 698 data.SetURL("http://bar/?{google:RLZ}{searchTerms}");
699 TemplateURL url(NULL, data); 699 TemplateURL url(NULL, data);
700 EXPECT_TRUE(url.url_ref().IsValid()); 700 EXPECT_TRUE(url.url_ref().IsValid());
701 ASSERT_TRUE(url.url_ref().SupportsReplacement()); 701 ASSERT_TRUE(url.url_ref().SupportsReplacement());
702 TemplateURLRef::SearchTermsArgs args(ASCIIToUTF16("x")); 702 TemplateURLRef::SearchTermsArgs args(ASCIIToUTF16("x"));
703 args.from_app_list = true; 703 args.from_app_list = true;
(...skipping 647 matching lines...) Expand 10 before | Expand all | Expand 10 after
1351 TemplateURLRef::SearchTermsArgs search_terms_args(ASCIIToUTF16("foo")); 1351 TemplateURLRef::SearchTermsArgs search_terms_args(ASCIIToUTF16("foo"));
1352 search_terms_args.session_token = "SESSIONTOKENGOESHERE"; 1352 search_terms_args.session_token = "SESSIONTOKENGOESHERE";
1353 std::string result = url.url_ref().ReplaceSearchTerms(search_terms_args); 1353 std::string result = url.url_ref().ReplaceSearchTerms(search_terms_args);
1354 EXPECT_EQ("http://bar/search?q=foo&psi=SESSIONTOKENGOESHERE&xssi=t", result); 1354 EXPECT_EQ("http://bar/search?q=foo&psi=SESSIONTOKENGOESHERE&xssi=t", result);
1355 1355
1356 TemplateURL url2(NULL, data); 1356 TemplateURL url2(NULL, data);
1357 search_terms_args.session_token = ""; 1357 search_terms_args.session_token = "";
1358 result = url.url_ref().ReplaceSearchTerms(search_terms_args); 1358 result = url.url_ref().ReplaceSearchTerms(search_terms_args);
1359 EXPECT_EQ("http://bar/search?q=foo&xssi=t", result); 1359 EXPECT_EQ("http://bar/search?q=foo&xssi=t", result);
1360 } 1360 }
OLDNEW
« no previous file with comments | « chrome/browser/search_engines/template_url_service.cc ('k') | chrome/browser/ui/browser_commands.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698