yRevert 57093 - dom-ui options: record user metrics for pref changes....

chrome/browser/dom_ui/advanced_options_handler.cc

 // Copyright (c) 2010 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 "chrome/browser/dom_ui/advanced_options_handler.h"
 
 #include "app/l10n_util.h"
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/utf_string_conversions.h"
@@ skipping 200 matching lines @@
   select_folder_dialog_->SelectFile(
       SelectFileDialog::SELECT_FOLDER,
       l10n_util::GetStringUTF16(IDS_OPTIONS_DOWNLOADLOCATION_BROWSE_TITLE),
       pref_service->GetFilePath(prefs::kDownloadDefaultDirectory),
       NULL, 0, FILE_PATH_LITERAL(""),
       dom_ui_->tab_contents()->view()->GetTopLevelNativeWindow(), NULL);
 }
 
 void AdvancedOptionsHandler::FileSelected(const FilePath& path, int index,
                                           void* params) {
-  UserMetricsRecordAction(UserMetricsAction("Options_SetDownloadDirectory"),
-                          NULL);
   default_download_location_.SetValue(path);
   SetupDownloadLocationPath();
 }
 
 void AdvancedOptionsHandler::HandleAutoOpenButton(const ListValue* args) {
-  UserMetricsRecordAction(UserMetricsAction("Options_ResetAutoOpenFiles"),
-                          NULL);
   DCHECK(dom_ui_);
   DownloadManager* manager = dom_ui_->GetProfile()->GetDownloadManager();
   if (manager) manager->ResetAutoOpenFiles();
 }
 
 #if defined(OS_WIN)
 void AdvancedOptionsHandler::HandleCheckRevocationCheckbox(
     const ListValue* args) {
-  UserMetricsRecordAction(UserMetricsAction(metric.c_str()), NULL);
   std::string checked_str = WideToUTF8(ExtractStringValue(args));
-  std::string metric =
-      (checked_str == "true" ? "Options_CheckCertRevocation_Enable"
-                             : "Options_CheckCertRevocation_Disable");
   net::SSLConfigServiceWin::SetRevCheckingEnabled(checked_str == "true");
 }
 
 void AdvancedOptionsHandler::HandleUseSSL2Checkbox(const ListValue* args) {
-  UserMetricsRecordAction(UserMetricsAction(metric.c_str()), NULL);
   std::string checked_str = WideToUTF8(ExtractStringValue(args));
-  std::string metric =
-      (checked_str == "true" ? "Options_SSL2_Enable"
-                             : "Options_SSL2_Disable");
   net::SSLConfigServiceWin::SetSSL2Enabled(checked_str == "true");
 }
 
 void AdvancedOptionsHandler::HandleShowGearsSettings(const ListValue* args) {
   UserMetricsRecordAction(UserMetricsAction("Options_GearsSettings"), NULL);
   GearsSettingsPressed(
       dom_ui_->tab_contents()->view()->GetTopLevelNativeWindow());
 }
 #endif
 
@@ skipping 68 matching lines @@
   }
   FundamentalValue checkRevocationValue(checkRevocationSetting);
   dom_ui_->CallJavascriptFunction(
       L"options.AdvancedOptions.SetCheckRevocationCheckboxState",
       checkRevocationValue);
   FundamentalValue useSSLValue(useSSLSetting);
   dom_ui_->CallJavascriptFunction(
       L"options.AdvancedOptions.SetUseSSL2CheckboxStatechecked", useSSLValue);
 }
 #endif