Offer user to send feedback from profile error dialog

chrome/browser/ui/profile_error_dialog.cc

 // Copyright (c) 2012 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/profile_error_dialog.h"
 
 #include "base/auto_reset.h"
 #include "base/base_switches.h"
 #include "base/command_line.h"
+#include "base/files/file_path.h"
 #include "base/metrics/histogram.h"
 #include "build/build_config.h"
+#include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/browser/ui/simple_message_box.h"
 #include "chrome/grit/chromium_strings.h"
+#include "chrome/grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
-void ShowProfileErrorDialog(ProfileErrorType type, int message_id) {
+namespace {
+
+#if !defined(OS_ANDROID)
+constexpr char kProfileErrorFeedbackCategory[] = "FEEDBACK_PROFILE_ERROR";
+#endif  // !defined(OS_ANDROID)
+
+}  // namespace
+
+void ShowProfileErrorDialog(const base::FilePath& corrupted_file,
+                            ProfileErrorType type,
+                            int message_id) {
 #if defined(OS_ANDROID)
   NOTIMPLEMENTED();
 #else
   UMA_HISTOGRAM_ENUMERATION("Profile.ProfileError", type, PROFILE_ERROR_END);
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kNoErrorDialogs))
+          switches::kNoErrorDialogs)) {
     return;
+  }
 
   static bool is_showing_profile_error_dialog = false;
   if (!is_showing_profile_error_dialog) {
     base::AutoReset<bool> resetter(&is_showing_profile_error_dialog, true);
-    chrome::ShowWarningMessageBox(NULL,
-                                  l10n_util::GetStringUTF16(IDS_PRODUCT_NAME),
-                                  l10n_util::GetStringUTF16(message_id));
+    if (chrome::ShowWarningMessageBoxWithCheckbox(
+            nullptr, l10n_util::GetStringUTF16(IDS_PROFILE_ERROR_DIALOG_TITLE),
+            l10n_util::GetStringUTF16(message_id),
+            l10n_util::GetStringUTF16(IDS_PROFILE_ERROR_DIALOG_CHECKBOX))) {
+      std::string feedback_description =
+          l10n_util::GetStringUTF8(IDS_PROFILE_ERROR_FEEDBACK_DESCRIPTION);
+      feedback_description +=
+          "\nCorrupted file: " + corrupted_file.AsUTF8Unsafe();

[Scott Hess - ex-Googler, 2016/06/29 05:13:48]

May be worth replacing the profile dir with .../ .  The fact that my profile dir
is /Users/shess/Library/Google/Chrome is probably not important, and the "shess"
may influence whether I want to allow the upload.

[afakhry, 2016/06/29 18:57:18]

We need to distinguish between the default profile DBs and the user profile DBs
to see which one gets corrupted the most. I'll think about a way to tell which
one is which even when we remove the dir.

[Scott Hess - ex-Googler, 2016/06/29 19:17:27]

Bit of clarification from offline: My concern is that having the entire dir from
root for Win/OSX/Linux would result in exposing the user's homedir, which might
give them pause.  Stripping the user-data-dir would remove the homedir issue,
but would leave the profile name in there.  I think for Win/OSX/Linux, this will
look like "Default", "Profile 1", "Profile 2", etc, so I think that seems
reasonable.  On ChromiumOS is appears to be a complicated hash, which also seems
reasonable.

Another option would be to strip the profile dir and put back a prefix.  Like
Default/, Secondary/, etc.  I kind of suspect that the only important cases are
Default/ and "not Default/", because it sounds like Default/ is used before
login, so there might be differences in the access patterns or how orderly the
shutdown of that profile is.

[Having a different profile before login makes me wonder if there aren't better
ways to handle that profile, for instance by putting things like History on a
tmpfs.]

+      chrome::ShowFeedbackPage(
+          nullptr,
+          feedback_description,
+          kProfileErrorFeedbackCategory);
+    }
   }
 #endif
 }