Add WebView-specific whitelist for crash keys.

components/crash/content/app/crash_reporter_client.h

 // Copyright 2013 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.
 
 #ifndef COMPONENTS_CRASH_CONTENT_APP_CRASH_REPORTER_CLIENT_H_
 #define COMPONENTS_CRASH_CONTENT_APP_CRASH_REPORTER_CLIENT_H_
 
 #include <stddef.h>
 
 #include <string>
+#include <vector>
 
 #include "base/strings/string16.h"
 #include "build/build_config.h"
 
 #if !defined(OS_WIN)
 namespace base {
 class FilePath;
 }
 #endif
 
@@ skipping 116 matching lines @@
 #if defined(OS_WIN)
   virtual bool GetCrashMetricsLocation(base::string16* metrics_dir);
 #else
   virtual bool GetCrashMetricsLocation(base::FilePath* metrics_dir);
 #endif
 
   // Register all of the potential crash keys that can be sent to the crash
   // reporting server. Returns the size of the union of all keys.
   virtual size_t RegisterCrashKeys();
 
+  virtual bool UseCrashKeysWhiteList();
+
+  virtual std::vector<const char*> GetWhiteListedCrashKeys();

[Tobias Sargeant, 2017/02/28 22:18:55]

Because the whitelist is not accessed in a compromised context (only when
setting the crash key value), WDYT about having only:

virtual void GetWhiteListedCrashKeys(std::unordererd_set<std::string>*
whitelist);

I don't think separate Use* and Get* methods are really needed. The default
implementation can just not add any whitelisted keys.

[gsennton, 2017/03/01 15:53:02]

If there aren't any whitelisted keys, shouldn't that mean that you don't want
any keys to be uploaded? :)

+
   // Returns true if running in unattended mode (for automated testing).
   virtual bool IsRunningUnattended();
 
   // Returns true if the user has given consent to collect stats.
   virtual bool GetCollectStatsConsent();
 
   // Returns true if the client is currently in the chosen sample that will
   // report stats and crashes. Crashes should only be reported if this function
   // returns true and GetCollectStatsConsent returns true.
   virtual bool GetCollectStatsInSample();
@@ skipping 17 matching lines @@
   virtual bool ShouldEnableBreakpadMicrodumps();
 #endif
 
   // Returns true if breakpad should run in the given process type.
   virtual bool EnableBreakpadForProcess(const std::string& process_type);
 };
 
 }  // namespace crash_reporter
 
 #endif  // COMPONENTS_CRASH_CONTENT_APP_CRASH_REPORTER_CLIENT_H_