Add WebView-specific whitelist for crash keys.

android_webview/common/crash_reporter/aw_microdump_crash_reporter.cc

 // Copyright 2015 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 "android_webview/common/crash_reporter/aw_microdump_crash_reporter.h"
 
 #include "android_webview/common/aw_descriptors.h"
 #include "android_webview/common/aw_paths.h"
 #include "android_webview/common/aw_version_info_values.h"
 #include "android_webview/common/crash_reporter/crash_keys.h"
 #include "base/android/build_info.h"
 #include "base/base_paths_android.h"
+#include "base/debug/crash_logging.h"
 #include "base/debug/dump_without_crashing.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/path_service.h"
 #include "base/scoped_native_library.h"
 #include "base/synchronization/lock.h"
 #include "build/build_config.h"
 #include "components/crash/content/app/breakpad_linux.h"
 #include "components/crash/content/app/crash_reporter_client.h"
 #include "content/public/common/content_switches.h"
 
 namespace android_webview {
 namespace crash_reporter {
 
 namespace {
 
 class AwCrashReporterClient : public ::crash_reporter::CrashReporterClient {
  public:
   AwCrashReporterClient()
       : dump_fd_(kAndroidMinidumpDescriptor), crash_signal_fd_(-1) {}
 
   // Does not use lock, can only be called immediately after creation.
   void set_crash_signal_fd(int fd) { crash_signal_fd_ = fd; }
 
   // crash_reporter::CrashReporterClient implementation.
   size_t RegisterCrashKeys() override;
+  bool UseCrashKeysWhiteList() override { return true; }
+  const char* const* GetCrashKeyWhiteList() override;
 
   bool IsRunningUnattended() override { return false; }
   bool GetCollectStatsConsent() override {
 #if defined(GOOGLE_CHROME_BUILD)
     // TODO(gsennton): Enabling minidump-generation unconditionally means we
     // will generate minidumps even if the user doesn't consent to minidump
     // uploads. However, we will check user-consent before uploading any
     // minidumps, if we do not have user consent we will delete the minidumps.
     // We should investigate whether we can avoid generating minidumps
     // altogether if we don't have user consent, see crbug.com/692485
@@ skipping 30 matching lines @@
  private:
   int dump_fd_;
   int crash_signal_fd_;
   DISALLOW_COPY_AND_ASSIGN(AwCrashReporterClient);
 };
 
 size_t AwCrashReporterClient::RegisterCrashKeys() {
   return crash_keys::RegisterWebViewCrashKeys();
 }
 
+const char* const* AwCrashReporterClient::GetCrashKeyWhiteList() {
+  return crash_keys::kWebViewCrashKeyWhiteList;
+}
+
 base::LazyInstance<AwCrashReporterClient>::Leaky g_crash_reporter_client =
     LAZY_INSTANCE_INITIALIZER;
 
 bool g_enabled = false;
 
 #if defined(ARCH_CPU_X86_FAMILY)
 bool SafeToUseSignalHandler() {
   // N+ shared library namespacing means that we are unable to dlopen
   // libnativebridge (because it isn't in the NDK). However we know
   // that, were we able to, the tests below would pass, so just return
@@ skipping 106 matching lines @@
 bool IsCrashReporterEnabled() {
   return breakpad::IsCrashReporterEnabled();
 }
 
 void SuppressDumpGeneration() {
   breakpad::SuppressDumpGeneration();
 }
 
 }  // namespace crash_reporter
 }  // namespace android_webview