Change email to be OPT-IN on Chrome and OPT-OUT on cros.

This CL changes the email box to be checked for chrome os and unchecked for Chrome.

R=jhawkins@chromium.org
BUG=140946


Review URL: https://chromiumcodereview.appspot.com/10834194

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150226 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/feedback.html

@@ -29,7 +29,7 @@
     <!-- User e-mail -->
     <div id="user-email" class="input-text-container checkbox">
       <label>
-        <input id="user-email-checkbox" type="checkbox" checked>
+        <input id="user-email-checkbox" type="checkbox">
         <span id="user-email-label" i18n-content="user-email"></span>
       </label>
       <input id="user-email-text">

chrome/browser/resources/feedback.js

@@ -284,6 +284,7 @@ function setupDialogDefaults(defaults) {
     $('page-url-checkbox').checked = false;
   // User e-mail.
   $('user-email-text').value = defaults.userEmail;
+  $('user-email-checkbox').checked = defaults.emailCheckboxDefault;
 
   // Are screenshots disabled?
   if (!defaults.disableScreenshots)

chrome/browser/ui/webui/feedback_ui.cc

@@ -476,6 +476,16 @@ void FeedbackHandler::HandleGetDialogDefaults(const ListValue*) {
   std::string user_email = GetUserEmail();
   dialog_defaults.SetString("userEmail", user_email);
 
+  // Set email checkbox to checked by default for cros, unchecked for Chrome.
+  dialog_defaults.SetBoolean(
+      "emailCheckboxDefault",
+#if defined(OS_CHROMEOS)
+      true);
+#else
+      false);
+#endif
+
+
 #if defined(OS_CHROMEOS)
   // Trigger the request for system information here.
   chromeos::system::SyslogsProvider* provider =