Manual crash uploads for mac and win

chrome/browser/ui/webui/crashes_ui.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/webui/crashes_ui.h"
 
 #include <stddef.h>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/sys_info.h"
 #include "base/values.h"
 #include "build/build_config.h"
 #include "chrome/browser/crash_upload_list/crash_upload_list.h"
 #include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
+#include "chrome/browser/metrics/metrics_reporting_state.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/chromium_strings.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/crash/core/browser/crashes_ui_util.h"
 #include "components/version_info/version_info.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_data_source.h"
 #include "content/public/browser/web_ui_message_handler.h"
@@ skipping 56 matching lines @@
   void HandleRequestCrashes(const base::ListValue* args);
 
 #if defined(OS_CHROMEOS)
   // Asynchronously triggers crash uploading. Called from JS.
   void HandleRequestUploads(const base::ListValue* args);
 #endif
 
   // Sends the recent crashes list JS.
   void UpdateUI();
 
+#if defined(OS_WIN) || defined(OS_MACOSX)
+  // Asynchronously requests a user triggered upload. Called from JS.
+  void HandleRequestSingleCrashUpload(const base::ListValue* args);
+#endif
+
   scoped_refptr<CrashUploadList> upload_list_;
   bool list_available_;
   bool first_load_;
 
   DISALLOW_COPY_AND_ASSIGN(CrashesDOMHandler);
 };
 
 CrashesDOMHandler::CrashesDOMHandler()
     : list_available_(false), first_load_(true) {
   upload_list_ = CreateCrashUploadList(this);
 }
 
 CrashesDOMHandler::~CrashesDOMHandler() {
   upload_list_->ClearDelegate();
 }
 
 void CrashesDOMHandler::RegisterMessages() {
   upload_list_->LoadUploadListAsynchronously();
   web_ui()->RegisterMessageCallback(
       crash::kCrashesUIRequestCrashList,
       base::Bind(&CrashesDOMHandler::HandleRequestCrashes,
                  base::Unretained(this)));
 
 #if defined(OS_CHROMEOS)
   web_ui()->RegisterMessageCallback(
       crash::kCrashesUIRequestCrashUpload,
       base::Bind(&CrashesDOMHandler::HandleRequestUploads,
                  base::Unretained(this)));
 #endif
+
+#if defined(OS_WIN) || defined(OS_MACOSX)
+  web_ui()->RegisterMessageCallback(
+      crash::kCrashesUIRequestSingleCrashUpload,
+      base::Bind(&CrashesDOMHandler::HandleRequestSingleCrashUpload,
+                 base::Unretained(this)));
+#endif
 }
 
 void CrashesDOMHandler::HandleRequestCrashes(const base::ListValue* args) {
   if (first_load_) {
     first_load_ = false;
     if (list_available_)
       UpdateUI();
   } else {
     list_available_ = false;
     upload_list_->LoadUploadListAsynchronously();
@@ skipping 19 matching lines @@
 void CrashesDOMHandler::UpdateUI() {
   bool crash_reporting_enabled =
       ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled();
 
   bool system_crash_reporter = false;
 #if defined(OS_CHROMEOS)
   // Chrome OS has a system crash reporter.
   system_crash_reporter = true;
 #endif
 
+  bool support_manual_uploads = false;

[Mark Mentovai, 2016/08/25 20:25:11]

Put “const bool support_manual_uploads = …” in both branches of the #if that
follows, so that it’s impossible to accidentally change the value.

[gayane -on leave until 09-2017, 2016/08/26 00:12:54]

Done.

   base::ListValue crash_list;
+#if defined(OS_WIN) || defined(OS_MACOSX)
+  // Maunal uploads currently are supported only for Windows and Mac platforms.
+  // Allow manual uploads only if crash uploads are not disabled by policy.

[Mark Mentovai, 2016/08/25 20:25:11]

s/Windows and Mac/Crashpad-using/

[gayane -on leave until 09-2017, 2016/08/26 00:12:55]

Done.

+  support_manual_uploads =
+      crash_reporting_enabled || IsMetricsReportingPolicyManaged();

[Mark Mentovai, 2016/08/25 20:25:11]

We want to support manual uploads if the user has disabled crash reporting too.
That’s a major reason to do this, in fact. But we don’t want to support it if
policy says “no”.

[gayane -on leave until 09-2017, 2016/08/26 00:12:55]

Thanks. There was a logic problem here. Now manual uploads would be off only if
crash reporting is disabled and policy is on.

+  // Show crash reports regardless of |crash_reporting_enabled| so that users
+  // can manually uploads those reports.
+  crash::UploadListToValue(upload_list_.get(), &crash_list);
+#else
+  // Don't show crash reports list if crash reporting is disbled for platforms
+  // where manual uploads are not supported.
   if (crash_reporting_enabled)
     crash::UploadListToValue(upload_list_.get(), &crash_list);
+#endif
 
   base::FundamentalValue enabled(crash_reporting_enabled);
   base::FundamentalValue dynamic_backend(system_crash_reporter);
+  base::FundamentalValue manual_uploads(support_manual_uploads);
   base::StringValue version(version_info::GetVersionNumber());
   base::StringValue os_string(base::SysInfo::OperatingSystemName() + " " +
                               base::SysInfo::OperatingSystemVersion());
 
   std::vector<const base::Value*> args;
   args.push_back(&enabled);
   args.push_back(&dynamic_backend);
+  args.push_back(&manual_uploads);
   args.push_back(&crash_list);
   args.push_back(&version);
   args.push_back(&os_string);
   web_ui()->CallJavascriptFunctionUnsafe(crash::kCrashesUIUpdateCrashList,
                                          args);
 }
 
+#if defined(OS_WIN) || defined(OS_MACOSX)
+void CrashesDOMHandler::HandleRequestSingleCrashUpload(

[Mark Mentovai, 2016/08/25 20:25:11]

I think it’s a good idea to check that policy says it’s OK to do the upload
here, once C++ has control again. Otherwise, we’re trusting the
(user-inspectable) JS code to maintain the correct value of
support_manual_uploads. Dangerous.

[gayane -on leave until 09-2017, 2016/08/26 00:12:55]

Done.

+    const base::ListValue* args) {
+  std::string local_id;
+  if (!args || !args->GetString(0, &local_id))
+    return;
+  upload_list_->RequestSingleCrashUploadAsync(local_id);
+}
+#endif
+
 }  // namespace
 
 ///////////////////////////////////////////////////////////////////////////////
 //
 // CrashesUI
 //
 ///////////////////////////////////////////////////////////////////////////////
 
 CrashesUI::CrashesUI(content::WebUI* web_ui) : WebUIController(web_ui) {
   web_ui->AddMessageHandler(new CrashesDOMHandler());
 
   // Set up the chrome://crashes/ source.
   Profile* profile = Profile::FromWebUI(web_ui);
   content::WebUIDataSource::Add(profile, CreateCrashesUIHTMLSource());
 }
 
 // static
 base::RefCountedMemory* CrashesUI::GetFaviconResourceBytes(
       ui::ScaleFactor scale_factor) {
   return ResourceBundle::GetSharedInstance().LoadDataResourceBytesForScale(
       IDR_CRASH_SAD_FAVICON, scale_factor);
 }