Chrome Cleaner UI: Add logs upload permission checkbox to the dialog

chrome/browser/ui/webui/settings/chrome_cleanup_handler.cc

 // Copyright (c) 2017 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/ui/webui/settings/chrome_cleanup_handler.h"
 
+#include <string>
+
 #include "base/command_line.h"
 #include "base/feature_list.h"
 #include "base/synchronization/lock.h"
 #include "base/values.h"
 #include "chrome/browser/safe_browsing/chrome_cleaner/srt_field_trial_win.h"
 #include "chrome/grit/generated_resources.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "ui/base/l10n/l10n_util.h"
 
@@ skipping 122 matching lines @@
 
   CallJavascriptFunction("cr.webUIListenerCallback",
                          base::Value("chrome-cleanup-on-dismiss"));
   // TODO(proberge): Show a prompt to reboot the system.
 }
 
 void ChromeCleanupHandler::HandleStartCleanup(const base::ListValue* args) {
   DCHECK_EQ(0U, args->GetSize());
 
   controller_->ReplyWithUserResponse(
-      profile_, ChromeCleanerController::UserResponse::kAccepted);
+      profile_, ChromeCleanerController::UserResponse::kAcceptedWithoutLogs);

[robertshield, 2017/06/29 13:47:21]

Shouldn't this take the value of the toggle in the web ui?

[robertshield, 2017/06/29 13:56:28]

Nm, just saw your comment in the mail. Maybe a TODO?

[proberge, 2017/06/29 13:56:29]

Yes. @alito please add a TODO under my name above this line.

[alito, 2017/06/29 17:14:27]

Added a TODO assigned to proberge@.

[alito, 2017/06/29 17:14:27]

Done.

 }
 
 }  // namespace settings