[Cleanup] Remove the no longer used GoogleSearch.AccessPoint metric.

chrome/browser/google/google_search_counter_android_unittest.cc

Index: chrome/browser/google/google_search_counter_android_unittest.cc
diff --git a/chrome/browser/google/google_search_counter_android_unittest.cc b/chrome/browser/google/google_search_counter_android_unittest.cc
deleted file mode 100644
index 0bd7d12cad60b9e5c2d8bb9b903f3a4bc632e9ee..0000000000000000000000000000000000000000
--- a/chrome/browser/google/google_search_counter_android_unittest.cc
+++ /dev/null
@@ -1,179 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/memory/scoped_ptr.h"
-#include "base/message_loop/message_loop.h"
-#include "chrome/browser/google/google_search_counter.h"
-#include "chrome/browser/google/google_search_counter_android.h"
-#include "chrome/browser/prerender/prerender_manager.h"
-#include "chrome/test/base/testing_profile.h"
-#include "components/google/core/browser/google_search_metrics.h"
-#include "content/public/browser/navigation_controller.h"
-#include "content/public/browser/navigation_details.h"
-#include "content/public/browser/navigation_entry.h"
-#include "content/public/browser/notification_service.h"
-#include "content/public/browser/notification_types.h"
-#include "content/public/test/test_browser_thread.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-
-class MockSearchMetrics : public GoogleSearchMetrics {
- public:
-  MOCK_CONST_METHOD2(RecordAndroidGoogleSearch,
-                     void(AccessPoint ap, bool prerender_enabled));
-};
-
-}  // namespace
-
-class GoogleSearchCounterAndroidTest : public testing::Test {
- protected:
-  GoogleSearchCounterAndroidTest();
-  ~GoogleSearchCounterAndroidTest() override;
-
-  // testing::Test
-  void SetUp() override;
-  void TearDown() override;
-
-  // Test if |url| is a Google search for specific types. When |is_omnibox| is
-  // true, this method will append Omnibox identifiers to the simulated URL
-  // navigation. If |expected_metric| is set and not AP_BOUNDARY, we'll also use
-  // the Search Metrics mock class to ensure that the type of metric recorded is
-  // correct. Note that when |expected_metric| is AP_BOUNDARY, we strictly
-  // forbid any metrics from being logged at all. See implementation below for
-  // details.
-  void TestGoogleSearch(const std::string& url,
-                        bool is_omnibox,
-                        GoogleSearchMetrics::AccessPoint expected_metric,
-                        bool expected_prerender_enabled);
-
- private:
-  void ExpectMetricsLogged(GoogleSearchMetrics::AccessPoint ap,
-                           bool prerender_enabled);
-
-  // Needed to pass PrerenderManager's DCHECKs.
-  base::MessageLoop message_loop_;
-  content::TestBrowserThread ui_thread_;
-  scoped_ptr<TestingProfile> profile_;
-  scoped_ptr<GoogleSearchCounterAndroid> search_counter_;
-  // Weak ptr. Actual instance owned by GoogleSearchCounter.
-  ::testing::StrictMock<MockSearchMetrics>* mock_search_metrics_;
-  prerender::PrerenderManager::PrerenderManagerMode original_prerender_mode_;
-};
-
-GoogleSearchCounterAndroidTest::GoogleSearchCounterAndroidTest()
-    : ui_thread_(content::BrowserThread::UI, &message_loop_),
-      profile_(new TestingProfile()),
-      search_counter_(new GoogleSearchCounterAndroid(profile_.get())),
-      mock_search_metrics_(NULL),
-      original_prerender_mode_(
-          prerender::PrerenderManager::PRERENDER_MODE_DISABLED) {
-}
-
-GoogleSearchCounterAndroidTest::~GoogleSearchCounterAndroidTest() {
-}
-
-void GoogleSearchCounterAndroidTest::SetUp() {
-  // Keep a weak ptr to MockSearchMetrics so we can run expectations. The
-  // GoogleSearchCounter singleton will own and clean up MockSearchMetrics.
-  mock_search_metrics_ = new ::testing::StrictMock<MockSearchMetrics>;
-  GoogleSearchCounter::GetInstance()->SetSearchMetricsForTesting(
-      mock_search_metrics_);
-  original_prerender_mode_ = prerender::PrerenderManager::GetMode();
-  prerender::PrerenderManager::SetMode(
-      prerender::PrerenderManager::PRERENDER_MODE_ENABLED);
-}
-
-void GoogleSearchCounterAndroidTest::TearDown() {
-  mock_search_metrics_ = NULL;
-  prerender::PrerenderManager::SetMode(original_prerender_mode_);
-}
-
-void GoogleSearchCounterAndroidTest::TestGoogleSearch(
-    const std::string& url,
-    bool is_omnibox,
-    GoogleSearchMetrics::AccessPoint expected_metric,
-    bool expected_prerender_enabled) {
-  content::LoadCommittedDetails details;
-  scoped_ptr<content::NavigationEntry> entry(
-      content::NavigationEntry::Create());
-  if (is_omnibox) {
-    entry->SetTransitionType(ui::PageTransitionFromInt(
-        ui::PAGE_TRANSITION_GENERATED |
-            ui::PAGE_TRANSITION_FROM_ADDRESS_BAR));
-  }
-  entry->SetURL(GURL(url));
-  details.entry = entry.get();
-
-  // Since the internal mocked metrics object is strict, if |expect_metrics| is
-  // false, the absence of this call to ExpectMetricsLogged will be noticed and
-  // cause the test to complain, as expected. We use this behaviour to test
-  // negative test cases (such as bad searches).
-  if (expected_metric != GoogleSearchMetrics::AP_BOUNDARY)
-    ExpectMetricsLogged(expected_metric, expected_prerender_enabled);
-
-  // For now we don't care about the notification source, but when we start
-  // listening for additional access points, we will have to pass in a valid
-  // controller.
-  search_counter_->Observe(
-      content::NOTIFICATION_NAV_ENTRY_COMMITTED,
-      content::Source<content::NavigationController>(NULL),
-      content::Details<content::LoadCommittedDetails>(&details));
-}
-
-void GoogleSearchCounterAndroidTest::ExpectMetricsLogged(
-    GoogleSearchMetrics::AccessPoint ap, bool prerender_enabled) {
-  EXPECT_CALL(*mock_search_metrics_,
-              RecordAndroidGoogleSearch(ap, prerender_enabled)).Times(1);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, EmptySearch) {
-  TestGoogleSearch(std::string(), false, GoogleSearchMetrics::AP_BOUNDARY,
-                   true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, GoodOmniboxSearch) {
-  TestGoogleSearch("http://www.google.com/search?q=something", true,
-                   GoogleSearchMetrics::AP_OMNIBOX, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, BadOmniboxSearch) {
-  TestGoogleSearch("http://www.google.com/search?other=something", true,
-                   GoogleSearchMetrics::AP_BOUNDARY, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, EmptyOmniboxSearch) {
-  TestGoogleSearch(std::string(), true, GoogleSearchMetrics::AP_BOUNDARY, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, GoodOtherSearch) {
-  TestGoogleSearch("http://www.google.com/search?q=something", false,
-                   GoogleSearchMetrics::AP_OTHER, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, BadOtherSearch) {
-  TestGoogleSearch("http://www.google.com/search?other=something", false,
-                   GoogleSearchMetrics::AP_BOUNDARY, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, SearchAppSearch) {
-  TestGoogleSearch("http://www.google.com/webhp?source=search_app#q=something",
-                   false, GoogleSearchMetrics::AP_SEARCH_APP, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, SearchAppStart) {
-  // Starting the search app takes you to this URL, but it should not be
-  // considered an actual search event. Note that this URL is not considered an
-  // actual search because it has no query string parameter ("q").
-  TestGoogleSearch("http://www.google.com/webhp?source=search_app",
-                   false, GoogleSearchMetrics::AP_BOUNDARY, true);
-}
-
-TEST_F(GoogleSearchCounterAndroidTest, GoodOmniboxSearch_PrerenderDisabled) {
-  prerender::PrerenderManager::SetMode(
-      prerender::PrerenderManager::PRERENDER_MODE_DISABLED);
-  TestGoogleSearch("http://www.google.com/search?q=something", true,
-                   GoogleSearchMetrics::AP_OMNIBOX, false);
-}