Closed
Bug 1076977
Opened 11 years ago
Closed 11 years ago
Crash reporter doesn't come up anymore an MacOSX
Categories
(Toolkit :: Crash Reporting, defect)
Tracking
()
VERIFIED
FIXED
mozilla35
People
(Reporter: martijn.martijn, Assigned: spohl)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
|
1.19 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
I noticed that when the browser crashes, the crash reporter doesn't come up anymore.
Str:
- Go to https://bugzilla.mozilla.org/attachment.cgi?id=8408204 (testcase from bug 997709)
- Print the testcase, or go to Preview
Expected result (until bug 997709 gets fixed):
- Crash and crash reporter appearing
Actual result:
- Crash and no crash reporter appearing
Syd, could this have anything to do with the Mac v2 codesigning application layout change.
| Reporter | ||
Comment 1•11 years ago
|
||
Build Machine
bld-lion-r5-003
Source
Built from https://hg.mozilla.org/mozilla-central/rev/2399d1ae89e9
Build platform
target
x86_64-apple-darwin11.2.0
| Reporter | ||
Updated•11 years ago
|
Keywords: regression
Comment 2•11 years ago
|
||
I can reproduce this as well. Almost certainly fallout from the v2 bundle changes.
Simpler STR:
1) Run Firefox
2) kill -ABRT <firefox pid>
Blocks: 1050944
| Assignee | ||
Comment 3•11 years ago
|
||
This may or may not be a duplicate of bug 1075691. The fix for bug 1075691 just landed on inbound. Martijn, would you be able to verify once the inbound build finished compiling? The cset is https://hg.mozilla.org/integration/mozilla-inbound/rev/cabd17a5837a and I'll be happy to post the URL for the build once it becomes available, if necessary.
Flags: needinfo?(martijn.martijn)
| Assignee | ||
Comment 4•11 years ago
|
||
Hmm, a quick test with a local debug build with the patch from bug 1075691 applied didn't seem to work either. Should debug builds trigger the crash reporter as well, or only optimized builds?
| Reporter | ||
Comment 5•11 years ago
|
||
I think I know where to find it from here: https://treeherder.mozilla.org/ui/#/jobs?repo=mozilla-inbound
Afaik, debug build don't trigger the crash reporter.
| Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Martijn Wargers [:mwargers] (QA) from comment #5)
> I think I know where to find it from here:
> https://treeherder.mozilla.org/ui/#/jobs?repo=mozilla-inbound
Awesome!
> Afaik, debug build don't trigger the crash reporter.
Will try with a local optimized build then. Thanks!
| Assignee | ||
Comment 7•11 years ago
|
||
I can't get the crash reporter to open with an optimized build either. Would still like confirmation from the build on inbound but will start looking into it.
| Reporter | ||
Comment 8•11 years ago
|
||
Ok, I tried with http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-macosx64/1412270423/firefox-35.0a1.en-US.mac.dmg
That doesn't trigger the crash reporter on a crash.
Flags: needinfo?(martijn.martijn)
| Assignee | ||
Comment 9•11 years ago
|
||
Confirmed locally that this fixes the issue.
Attachment #8499158 -
Flags: review?(benjamin)
Updated•11 years ago
|
Attachment #8499158 -
Flags: review?(benjamin) → review+
| Assignee | ||
Comment 11•11 years ago
|
||
Comment 12•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
| Reporter | ||
Comment 13•11 years ago
|
||
Works again in latest Nightly, thanks.
Status: RESOLVED → VERIFIED
Comment 14•11 years ago
|
||
I think that this is a perfect candidate for Moztrap if we don't have a test yet. I do not see a way to get this easily automated, even not with Mozmill. Juan, mind adding it to Moztrap.
Flags: needinfo?(jbecerra)
Flags: in-moztrap?(jbecerra)
Hardware: x86 → All
Updated•11 years ago
|
Flags: needinfo?(jbecerra)
Comment 15•11 years ago
|
||
There's already a MozTrap case for this:
https://moztrap.mozilla.org/manage/case/8736/
| Reporter | ||
Updated•11 years ago
|
Flags: in-moztrap?(jbecerra) → in-moztrap+
Comment 16•11 years ago
|
||
Landed on aurora in the Mac V2 signing combined patch in bug 1047584
status-firefox34:
--- → fixed
status-firefox35:
--- → fixed
Updated•11 years ago
|
Updated•11 years ago
|
Flags: qe-verify+
Comment 17•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•