| 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 0474033d015349ec6034bfa934f696b75890cca8..063ac2396f31872fefc2c27a6e0e93d6dbfd71ca 100644
|
| --- a/components/search_engines/template_url_unittest.cc
|
| +++ b/components/search_engines/template_url_unittest.cc
|
| @@ -12,109 +12,27 @@
|
| #include "components/search_engines/search_engines_switches.h"
|
| #include "components/search_engines/search_terms_data.h"
|
| #include "components/search_engines/template_url.h"
|
| +#include "components/search_engines/testing_search_terms_data.h"
|
| #include "testing/gtest/include/gtest/gtest.h"
|
|
|
| using base::ASCIIToUTF16;
|
|
|
| -// TestSearchTermsData --------------------------------------------------------
|
| -
|
| -// Simple implementation of SearchTermsData.
|
| -class TestSearchTermsData : public SearchTermsData {
|
| - public:
|
| - explicit TestSearchTermsData(const std::string& google_base_url);
|
| -
|
| - virtual std::string GoogleBaseURLValue() const OVERRIDE;
|
| - virtual base::string16 GetRlzParameterValue(
|
| - bool from_app_list) const OVERRIDE;
|
| - virtual std::string GetSearchClient() const OVERRIDE;
|
| - virtual std::string GoogleImageSearchSource() const OVERRIDE;
|
| - virtual bool EnableAnswersInSuggest() const OVERRIDE;
|
| - virtual bool IsShowingSearchTermsOnSearchResultsPages() const OVERRIDE;
|
| - virtual int OmniboxStartMargin() const OVERRIDE;
|
| -
|
| - void set_google_base_url(const std::string& google_base_url) {
|
| - google_base_url_ = google_base_url;
|
| - }
|
| - void set_search_client(const std::string& search_client) {
|
| - search_client_ = search_client;
|
| - }
|
| - void set_enable_answers_in_suggest(bool enable_answers_in_suggest) {
|
| - enable_answers_in_suggest_ = enable_answers_in_suggest;
|
| - }
|
| - void set_is_showing_search_terms_on_search_results_pages(bool value) {
|
| - is_showing_search_terms_on_search_results_pages_ = value;
|
| - }
|
| - void set_omnibox_start_margin(int omnibox_start_margin) {
|
| - omnibox_start_margin_ = omnibox_start_margin;
|
| - }
|
| -
|
| - private:
|
| - std::string google_base_url_;
|
| - std::string search_client_;
|
| - bool enable_answers_in_suggest_;
|
| - bool is_showing_search_terms_on_search_results_pages_;
|
| - int omnibox_start_margin_;
|
| -
|
| - DISALLOW_COPY_AND_ASSIGN(TestSearchTermsData);
|
| -};
|
| -
|
| -TestSearchTermsData::TestSearchTermsData(const std::string& google_base_url)
|
| - : google_base_url_(google_base_url),
|
| - enable_answers_in_suggest_(false),
|
| - is_showing_search_terms_on_search_results_pages_(false) {
|
| -}
|
| -
|
| -std::string TestSearchTermsData::GoogleBaseURLValue() const {
|
| - return google_base_url_;
|
| -}
|
| -
|
| -base::string16 TestSearchTermsData::GetRlzParameterValue(
|
| - bool from_app_list) const {
|
| - return ASCIIToUTF16(
|
| - from_app_list ? "rlz_parameter_from_app_list" : "rlz_parameter");
|
| -}
|
| -
|
| -std::string TestSearchTermsData::GetSearchClient() const {
|
| - return search_client_;
|
| -}
|
| -
|
| -std::string TestSearchTermsData::GoogleImageSearchSource() const {
|
| - return "google_image_search_source";
|
| -}
|
| -
|
| -bool TestSearchTermsData::EnableAnswersInSuggest() const {
|
| - return enable_answers_in_suggest_;
|
| -}
|
| -
|
| -bool TestSearchTermsData::IsShowingSearchTermsOnSearchResultsPages() const {
|
| - return is_showing_search_terms_on_search_results_pages_;
|
| -}
|
| -
|
| -int TestSearchTermsData::OmniboxStartMargin() const {
|
| - return omnibox_start_margin_;
|
| -}
|
| -
|
| -// TemplateURLTest ------------------------------------------------------------
|
| -
|
| class TemplateURLTest : public testing::Test {
|
| public:
|
| TemplateURLTest() : search_terms_data_("http://www.google.com/") {}
|
| void CheckSuggestBaseURL(const std::string& base_url,
|
| const std::string& base_suggest_url) const;
|
|
|
| - TestSearchTermsData search_terms_data_;
|
| + TestingSearchTermsData search_terms_data_;
|
| };
|
|
|
| void TemplateURLTest::CheckSuggestBaseURL(
|
| const std::string& base_url,
|
| const std::string& base_suggest_url) const {
|
| - TestSearchTermsData search_terms_data(base_url);
|
| + TestingSearchTermsData search_terms_data(base_url);
|
| EXPECT_EQ(base_suggest_url, search_terms_data.GoogleBaseSuggestURLValue());
|
| }
|
|
|
| -
|
| -// Actual tests ---------------------------------------------------------------
|
| -
|
| TEST_F(TemplateURLTest, Defaults) {
|
| TemplateURLData data;
|
| EXPECT_FALSE(data.show_in_default_list);
|
| @@ -258,7 +176,7 @@ TEST_F(TemplateURLTest, URLRefTestImageURLWithPOST) {
|
| search_args.image_original_size = gfx::Size(10, 10);
|
| // Replacement operation with no post_data buffer should still return
|
| // the parsed URL.
|
| - TestSearchTermsData search_terms_data("http://X");
|
| + TestingSearchTermsData search_terms_data("http://X");
|
| GURL result(url.image_url_ref().ReplaceSearchTerms(
|
| search_args, search_terms_data));
|
| ASSERT_TRUE(result.is_valid());
|
| @@ -383,7 +301,7 @@ TEST_F(TemplateURLTest, URLRefTestSearchTermsUsingTermsData) {
|
| "http://example.com/complete/" }
|
| };
|
|
|
| - TestSearchTermsData search_terms_data("http://example.com/e/");
|
| + TestingSearchTermsData search_terms_data("http://example.com/e/");
|
| TemplateURLData data;
|
| for (size_t i = 0; i < ARRAYSIZE_UNSAFE(search_term_cases); ++i) {
|
| const SearchTermsCase& value = search_term_cases[i];
|
|
|