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

components/chrome_cleaner/public/interfaces/chrome_prompt.mojom

Index: components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
diff --git a/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom b/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
index e935de9d570882ed42d6c9f35c91afd2d3a1c904..0208967f5a8830e58ad9cd9e66f9bf9fd2b7d3d6 100644
--- a/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
+++ b/components/chrome_cleaner/public/interfaces/chrome_prompt.mojom
@@ -16,16 +16,15 @@ module chrome_cleaner.mojom;
 [Extensible]
 enum PromptAcceptance {
   UNSPECIFIED = 0,
-  // The Chrome prompt was not shown to the user (for example, due to an
-  // experiment run that shouldn't prompt the user or because the user has
-  // been prompted recently).
-  NOT_SHOWN = 1,

[proberge, 2017/06/29 13:55:53]

Is removing NOT_SHOWN intended?

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

It is intentional. Chrome is not sending this to the Cleaner from anywhere and I
don't see a need for it. I think this was introduced while we were still
considering having a single unified reporter and cleaner binary.

-  // The user explicitly accepted the Chrome prompt.
-  ACCEPTED = 2,
+  // The user explicitly accepted the cleanup operation and cleaner logs
+  // upload is allowed.
+  ACCEPTED_WITH_LOGS = 1,
+  // The user explicitly accepted the cleanup operation and cleaner logs
+  // upload is not allowed.
+  ACCEPTED_WITHOUT_LOGS = 2,
   // The user explicitly denied the Chrome prompt.
   DENIED = 3,
-  // The user didn't interact with the Chrome prompt after a while.
-  IGNORED = 4,
+  NUM_VALUES,
 };
 
 struct FilePath {