Fixed the default value for 'crash report' on EULA screen....

chrome/browser/chromeos/login/eula_view.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/chromeos/login/eula_view.h"
 
 #include <signal.h>
 #include <sys/types.h>
 #include <string>
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/basictypes.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/customization_document.h"
 #include "chrome/browser/chromeos/login/network_screen_delegate.h"
 #include "chrome/browser/chromeos/login/rounded_rect_painter.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "chrome/browser/options_util.h"
-#include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profile_manager.h"
 #include "chrome/browser/renderer_host/site_instance.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/views/dom_view.h"
 #include "chrome/common/native_web_keyboard_event.h"
-#include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/installer/util/google_update_settings.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "views/controls/button/checkbox.h"
 #include "views/controls/button/native_button.h"
 #include "views/controls/label.h"
 #include "views/grid_layout.h"
 #include "views/layout_manager.h"
@@ skipping 109 matching lines @@
     } else {
       LOG(INFO) << "No eula found for locale: " << locale;
     }
   } else {
     LOG(ERROR) << "No manifest found.";
   }
   return GURL();
 }
 
 void EulaView::Init() {
-  // Handle preference that enables crash/statistics reporting. We do
-  // not monitor its change in background since we are in OOBE wizard.
-  metrics_reporting_enabled_.Init(prefs::kMetricsReportingEnabled,
-                                  g_browser_process->local_state(), NULL);
-
   // Use rounded rect background.
   views::Painter* painter = CreateWizardPainter(
       &BorderDefinition::kScreenBorder);
   set_background(
       views::Background::CreateBackgroundPainter(true, painter));
 
   // Layout created controls.
   views::GridLayout* layout = new views::GridLayout(this);
   SetLayoutManager(layout);
   SetUpGridLayout(layout);
@@ skipping 15 matching lines @@
   box_view->SetLayoutManager(new FillLayoutWithBorder());
   layout->AddView(box_view);
 
   google_eula_view_ = new DOMView();
   box_view->AddChildView(google_eula_view_);
 
   layout->AddPaddingRow(0, kRelatedControlSmallVerticalSpacing);
   layout->StartRow(0, SINGLE_CONTROL_WITH_SHIFT_ROW);
   usage_statistics_checkbox_ = new views::Checkbox();
   usage_statistics_checkbox_->SetMultiLine(true);
-  usage_statistics_checkbox_->SetChecked(
-      metrics_reporting_enabled_.GetValue());
+
+  // TODO(zelidrag): http://crosbug/6367 - Change default settings for checked
+  // and enabled state of usage_statistics_checkbox_ before the product is
+  // released.
+  usage_statistics_checkbox_->SetChecked(true);
   layout->AddView(usage_statistics_checkbox_);
-  usage_statistics_checkbox_->SetEnabled(
-      metrics_reporting_enabled_.IsManaged());
+  usage_statistics_checkbox_->SetEnabled(false);
 
   layout->StartRow(0, SINGLE_LINK_WITH_SHIFT_ROW);
   learn_more_link_ = new views::Link();
   learn_more_link_->SetController(this);
   layout->AddView(learn_more_link_);
 
   layout->AddPaddingRow(0, kRelatedControlSmallVerticalSpacing);
   layout->StartRow(0, SINGLE_CONTROL_ROW);
   oem_eula_label_ = new views::Label(std::wstring(), label_font);
   layout->AddView(oem_eula_label_, 1, 1,
@@ skipping 74 matching lines @@
   Layout();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // views::ButtonListener implementation:
 
 void EulaView::ButtonPressed(views::Button* sender, const views::Event& event) {
   if (sender == continue_button_) {
     if (usage_statistics_checkbox_) {
       bool enable_reporting = usage_statistics_checkbox_->checked();
-      if (metrics_reporting_enabled_.GetValue() != enable_reporting) {
-        enable_reporting =
-            OptionsUtil::ResolveMetricsReportingEnabled(enable_reporting);
-        metrics_reporting_enabled_.SetValueIfNotManaged(enable_reporting);
+      enable_reporting =
+          OptionsUtil::ResolveMetricsReportingEnabled(enable_reporting);
 #if defined(USE_LINUX_BREAKPAD)
-        if (enable_reporting)
-          InitCrashReporter();
+      if (enable_reporting)
+        InitCrashReporter();
 #endif
-      }
     }
     observer_->OnExit(ScreenObserver::EULA_ACCEPTED);
   }
   // TODO(glotov): handle cancel button.
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // views::LinkController implementation:
 
 void EulaView::LinkActivated(views::Link* source, int event_flags) {
@@ skipping 25 matching lines @@
   }
 }
 
 void EulaView::HandleKeyboardEvent(const NativeWebKeyboardEvent& event) {
   views::Widget* widget = GetWidget();
   if (widget && event.os_event && !event.skip_in_browser)
     static_cast<views::WidgetGtk*>(widget)->HandleKeyboardEvent(event.os_event);
 }
 
 }  // namespace chromeos