Omnibox Metrics - Remove Percent Match Histograms

components/omnibox/browser/omnibox_edit_model.cc

 // Copyright 2012 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 "components/omnibox/browser/omnibox_edit_model.h"
 
 #include <algorithm>
 #include <string>
 #include <utility>
 
@@ skipping 65 matching lines @@
 
 // Histogram name which counts the number of milliseconds a user takes
 // between focusing and editing the omnibox.
 const char kFocusToEditTimeHistogram[] = "Omnibox.FocusToEditTime";
 
 // Histogram name which counts the number of milliseconds a user takes
 // between focusing and opening an omnibox match.
 const char kFocusToOpenTimeHistogram[] =
     "Omnibox.FocusToOpenTimeAnyPopupState2";
 
-// Split the percentage match histograms into buckets based on the width of the
-// omnibox.
-const int kPercentageMatchHistogramWidthBuckets[] = { 400, 700, 1200 };
-
-void RecordPercentageMatchHistogram(const base::string16& old_text,
-                                    const base::string16& new_text,
-                                    ui::PageTransition transition,
-                                    int omnibox_width) {
-  size_t avg_length = (old_text.length() + new_text.length()) / 2;
-
-  int percent = 0;
-  if (!old_text.empty() && !new_text.empty()) {
-    size_t shorter_length = std::min(old_text.length(), new_text.length());
-    base::string16::const_iterator end(old_text.begin() + shorter_length);
-    base::string16::const_iterator mismatch(
-        std::mismatch(old_text.begin(), end, new_text.begin()).first);
-    size_t matching_characters = mismatch - old_text.begin();
-    percent = static_cast<float>(matching_characters) / avg_length * 100;
-  }
-
-  // TODO(treib,mpearson): Do we want to keep these histograms? Most of the
-  // previously-logged cases don't exist anymore. crbug.com/627747
-  std::string histogram_name;
-  if (ui::PageTransitionTypeIncludingQualifiersIs(
-          transition, ui::PAGE_TRANSITION_TYPED)) {
-    histogram_name = "InstantExtended.PercentageMatchV2_URLtoURL";
-    UMA_HISTOGRAM_PERCENTAGE(histogram_name, percent);
-  } else {
-    histogram_name = "InstantExtended.PercentageMatchV2_URLtoQuery";
-    UMA_HISTOGRAM_PERCENTAGE(histogram_name, percent);
-  }
-
-  std::string suffix = "large";
-  for (size_t i = 0; i < arraysize(kPercentageMatchHistogramWidthBuckets);
-       ++i) {
-    if (omnibox_width < kPercentageMatchHistogramWidthBuckets[i]) {
-      suffix = base::IntToString(kPercentageMatchHistogramWidthBuckets[i]);
-      break;
-    }
-  }
-
-  // Cannot rely on UMA histograms macro because the name of the histogram is
-  // generated dynamically.
-  base::HistogramBase* counter = base::LinearHistogram::FactoryGet(
-      histogram_name + "_" + suffix, 1, 101, 102,
-      base::Histogram::kUmaTargetedHistogramFlag);
-  counter->Add(percent);
-}
-
 }  // namespace
 
 
 // OmniboxEditModel::State ----------------------------------------------------
 
 OmniboxEditModel::State::State(bool user_input_in_progress,
                                const base::string16& user_text,
                                const base::string16& gray_text,
                                const base::string16& keyword,
                                bool is_keyword_hint,
@@ skipping 602 matching lines @@
   }
 
   // Get the current text before we call RevertAll() which will clear it.
   base::string16 current_text = view_->GetText();
 
   if (disposition != WindowOpenDisposition::NEW_BACKGROUND_TAB) {
     base::AutoReset<bool> tmp(&in_revert_, true);
     view_->RevertAll();  // Revert the box to its unedited state.
   }
 
-  RecordPercentageMatchHistogram(
-      permanent_text_, current_text, match.transition, view_->GetWidth());
-
   // Track whether the destination URL sends us to a search results page
   // using the default search provider.
   TemplateURLService* template_url_service = client_->GetTemplateURLService();
   if (template_url_service &&
       template_url_service->IsSearchResultsPageFromDefaultSearchProvider(
           match.destination_url)) {
     base::RecordAction(
         base::UserMetricsAction("OmniboxDestinationURLIsSearchOnDSP"));
   }
 
@@ skipping 721 matching lines @@
   // Update state and notify view if the omnibox has focus and the caret
   // visibility changed.
   const bool was_caret_visible = is_caret_visible();
   focus_state_ = state;
   if (focus_state_ != OMNIBOX_FOCUS_NONE &&
       is_caret_visible() != was_caret_visible)
     view_->ApplyCaretVisibility();
 
   client_->OnFocusChanged(focus_state_, reason);
 }