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

Unified Diff: components/search_engines/template_url_unittest.cc

Issue 492963005: Stop using UIThreadSearchTermsData for search_engines related tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use make_scoped_ptr Created 6 years, 4 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.gypi ('k') | components/search_engines/testing_search_terms_data.h » ('j') | 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 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];
« no previous file with comments | « components/search_engines.gypi ('k') | components/search_engines/testing_search_terms_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698