From 0924d424e444d57dd95c647652a11f2d655c11a0 Mon Sep 17 00:00:00 2001 From: Joshua Peraza Date: Thu, 19 Jan 2017 11:18:41 -0800 Subject: Populate stack frames with unloaded module info. This CL hits lots of source files because: 1. An update to the CodeModule virtual class. I added an is_loaded method to specify whether the module is loaded. There were several mocks/test classes that needed to be updated with an implementation. An alternative to this route would be to modify MinidumpUnloadedModule::code_file to prepend "Unloaded_" to the module name. 2. Added an unloaded_modules parameter to StackFrameSymbolizer::FillSourceLineInfo. BUG= Change-Id: Ic9c7f7c7b7e932a154a5d4ccf292c1527d8da09f Reviewed-on: https://chromium-review.googlesource.com/430241 Reviewed-by: Ivan Penkov --- src/google_breakpad/processor/minidump.h | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/google_breakpad/processor/minidump.h') diff --git a/src/google_breakpad/processor/minidump.h b/src/google_breakpad/processor/minidump.h index 1bfc7d9e..bff38bf3 100644 --- a/src/google_breakpad/processor/minidump.h +++ b/src/google_breakpad/processor/minidump.h @@ -399,6 +399,7 @@ class MinidumpModule : public MinidumpObject, virtual string debug_identifier() const; virtual string version() const; virtual CodeModule* Copy() const; + virtual bool is_unloaded() const { return false; } // Getter and setter for shrink_down_delta. This is used when the address // range for a module is shrunk down due to address range conflicts with @@ -775,6 +776,7 @@ class MinidumpUnloadedModule : public MinidumpObject, string debug_identifier() const override; string version() const override; CodeModule* Copy() const override; + bool is_unloaded() const override { return true; } uint64_t shrink_down_delta() const override; void SetShrinkDownDelta(uint64_t shrink_down_delta) override; -- cgit v1.2.1