Do not generate a microdump if there are no webview pointers on the stack.

components/crash/content/app/breakpad_linux.cc

Index: components/crash/content/app/breakpad_linux.cc
diff --git a/components/crash/content/app/breakpad_linux.cc b/components/crash/content/app/breakpad_linux.cc
index 9ebc33f4f01ce975604b921303e43c3af59a04cb..2d9e8ae86658476778795a737c8133c67a430a37 100644
--- a/components/crash/content/app/breakpad_linux.cc
+++ b/components/crash/content/app/breakpad_linux.cc
@@ -110,7 +110,11 @@ class MicrodumpInfo {
   MicrodumpInfo()
       : microdump_build_fingerprint_(nullptr),
         microdump_product_info_(nullptr),
-        microdump_gpu_fingerprint_(nullptr) {}
+        microdump_gpu_fingerprint_(nullptr),
+        microdump_process_type_(nullptr),
+        interest_range_start_(0ul),
+        interest_range_end_(0ul),
+        suppress_microdump_based_on_interest_range_(false) {}
 
   // The order in which SetGpuFingerprint and Initialize are called
   // may be dependent on the timing of the availability of GPU
@@ -126,6 +130,7 @@ class MicrodumpInfo {
   // which a microdump is generated, then the GPU fingerprint will be
   // UNKNOWN.
   void SetGpuFingerprint(const std::string& gpu_fingerprint);
+  void SetMicrodumpInterestRange(uintptr_t start, uintptr_t end);
   void Initialize(const std::string& process_type,
                   const char* product_name,
                   const char* product_version,
@@ -137,6 +142,9 @@ class MicrodumpInfo {
   const char* microdump_product_info_;
   const char* microdump_gpu_fingerprint_;
   const char* microdump_process_type_;
+  uintptr_t interest_range_start_;
+  uintptr_t interest_range_end_;
+  bool suppress_microdump_based_on_interest_range_;
 };
 
 base::LazyInstance<MicrodumpInfo> g_microdump_info =
@@ -872,6 +880,22 @@ void MicrodumpInfo::SetGpuFingerprint(const std::string& gpu_fingerprint) {
   }
 }
 
+void MicrodumpInfo::SetMicrodumpInterestRange(uintptr_t start, uintptr_t end) {
+  interest_range_start_ = start;
+  interest_range_end_ = end;
+  suppress_microdump_based_on_interest_range_ = true;
+
+  if (g_microdump) {
+    MinidumpDescriptor descriptor(g_microdump->minidump_descriptor());
+    google_breakpad::MicrodumpExtraInfo* microdump_extra_info =
+        descriptor.microdump_extra_info();
+    microdump_extra_info->interest_range_start = interest_range_start_;
+    microdump_extra_info->interest_range_end = interest_range_end_;
+    microdump_extra_info->suppress_microdump_based_on_interest_range = true;
+    g_microdump->set_minidump_descriptor(descriptor);
+  }
+}
+
 void MicrodumpInfo::Initialize(const std::string& process_type,
                                const char* product_name,
                                const char* product_version,
@@ -886,31 +910,36 @@ void MicrodumpInfo::Initialize(const std::string& process_type,
                             process_type == kBrowserProcessType;
 
   MinidumpDescriptor descriptor(MinidumpDescriptor::kMicrodumpOnConsole);
+  google_breakpad::MicrodumpExtraInfo* microdump_extra_info =
+      descriptor.microdump_extra_info();
 
   if (product_name && product_version) {
     microdump_product_info_ =
         strdup((product_name + std::string(":") + product_version).c_str());
     ANNOTATE_LEAKING_OBJECT_PTR(microdump_product_info_);
-    descriptor.microdump_extra_info()->product_info = microdump_product_info_;
+    microdump_extra_info->product_info = microdump_product_info_;
   }
 
   microdump_process_type_ =
       strdup(process_type.empty() ? kBrowserProcessType : process_type.c_str());
   ANNOTATE_LEAKING_OBJECT_PTR(microdump_process_type_);
-  descriptor.microdump_extra_info()->process_type = microdump_process_type_;
+  microdump_extra_info->process_type = microdump_process_type_;
 
   if (android_build_fp) {
     microdump_build_fingerprint_ = strdup(android_build_fp);
     ANNOTATE_LEAKING_OBJECT_PTR(microdump_build_fingerprint_);
-    descriptor.microdump_extra_info()->build_fingerprint =
-        microdump_build_fingerprint_;
+    microdump_extra_info->build_fingerprint = microdump_build_fingerprint_;
   }
 
   if (microdump_gpu_fingerprint_) {
-    descriptor.microdump_extra_info()->gpu_fingerprint =
-        microdump_gpu_fingerprint_;
+    microdump_extra_info->gpu_fingerprint = microdump_gpu_fingerprint_;
   }
 
+  microdump_extra_info->interest_range_start = interest_range_start_;
+  microdump_extra_info->interest_range_end = interest_range_end_;
+  microdump_extra_info->suppress_microdump_based_on_interest_range =
+      suppress_microdump_based_on_interest_range_;
+
   g_microdump =
       new ExceptionHandler(descriptor, nullptr, MicrodumpCrashDone,
                            reinterpret_cast<void*>(is_browser_process),
@@ -935,7 +964,7 @@ void MicrodumpInfo::Initialize(const std::string& process_type,
         GetCrashReporterClient()->GetAndroidCrashSignalFD();
     if (g_signal_code_pipe_fd != -1)
       g_microdump->set_crash_handler(WriteSignalCodeToPipe);
-  }
+    }

[Robert Sesek, 2016/12/05 16:19:18]

Incorrect indent.

[Tobias Sargeant, 2016/12/06 16:48:33]

Done.

 }
 
 #else
@@ -1934,6 +1963,10 @@ void AddGpuFingerprintToMicrodumpCrashHandler(
     const std::string& gpu_fingerprint) {
   g_microdump_info.Get().SetGpuFingerprint(gpu_fingerprint);
 }
+
+void SetWebViewTextAddrRange(uintptr_t start, uintptr_t end) {
+  g_microdump_info.Get().SetMicrodumpInterestRange(start, end);
+}
 #endif  // OS_ANDROID
 
 bool IsCrashReporterEnabled() {