Bug 1466242 Look for Sprintf.h instead of Assertions.h in the symbolstore test r?ted draft
authorTom Ritter <tom@mozilla.com>
Tue, 05 Jun 2018 10:44:03 -0500
changeset 804171 7a1d6e782500797855a116bc866e1cb3b6970dfc
parent 804170 05f570cb0fe70b2d21e87f057e7b189ebd9c8d19
push id112318
push userbmo:tom@mozilla.com
push dateTue, 05 Jun 2018 16:26:28 +0000
reviewersted
bugs1466242
milestone62.0a1
Bug 1466242 Look for Sprintf.h instead of Assertions.h in the symbolstore test r?ted It turns out sometimes (in the LTO+CFI case at least) Assertions.h will not be present in the opt build, presumably because it was optimized out. MozReview-Commit-ID: GB3GIoSdIUK
toolkit/crashreporter/tools/unit-symbolstore.py
--- a/toolkit/crashreporter/tools/unit-symbolstore.py
+++ b/toolkit/crashreporter/tools/unit-symbolstore.py
@@ -532,13 +532,13 @@ class TestFunctional(HelperMixin, unitte
             for line in match_lines:
                 filename = line.split(None, 2)[2]
                 self.assertEqual('hg:', filename[:3])
         # Check that nsBrowserApp.cpp is listed as a FILE line, and that
         # it was properly mapped to the source repo.
         check_hg_path(file_lines, 'nsBrowserApp.cpp')
         # Also check Assertions.h to verify that files from dist/include
         # are properly mapped.
-        check_hg_path(file_lines, 'mfbt/Assertions.h')
+        check_hg_path(file_lines, 'mfbt/Sprintf.h')
 
 
 if __name__ == '__main__':
     mozunit.main()