Closed Bug 1491580 Opened 6 years ago Closed 6 years ago

Crash in -[ChildView initWithFrame:geckoChild:]

Categories

(Core :: Widget: Cocoa, defect)

Product:
Core
Component:
Widget: Cocoa
Version:
64 Branch
Platform:
Unspecified
macOS
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla64
Tracking Flags:
Tracking Status
firefox-esr60 --- unaffected
firefox62 --- unaffected
firefox63 --- fixed
firefox64 --- fixed

People

(Reporter: calixte, Assigned: hiro)

References

(Blocks 1 open bug)

Details

(Keywords: crash, regression)

Crash Data

Attachments

(1 file)

Bug 1491580 - Use NSWorkspaceAccessibilityDisplayOptionsDidChangeNotification only for MacOSX 10.10 or later versions. r=mstange
46 bytes, text/x-phabricator-request
mstange
: review+
pascalc
: approval-mozilla-beta+
Details | Review
This bug was filed from the Socorro interface and is report bp-06333735-f132-4acc-94f7-140310180915. ============================================================= Top 10 frames of crashing thread: 0 XUL -[ChildView initWithFrame:geckoChild:] widget/cocoa/nsChildView.mm:3375 1 XUL nsChildView::Create widget/cocoa/nsChildView.mm:526 2 XUL nsBaseWidget::CreateChild widget/nsBaseWidget.cpp:442 3 XUL nsView::CreateWidgetForParent view/nsView.cpp:616 4 XUL nsDocumentViewer::MakeWindow layout/base/nsDocumentViewer.cpp:2608 5 XUL nsDocumentViewer::InitInternal layout/base/nsDocumentViewer.cpp:983 6 XUL nsDocumentViewer::Init layout/base/nsDocumentViewer.cpp:771 7 XUL nsDocShell::Embed docshell/base/nsDocShell.cpp:8882 8 XUL nsDocShell::CreateAboutBlankContentViewer docshell/base/nsDocShell.cpp:7585 9 XUL nsAppShellService::JustCreateTopWindow xpfe/appshell/nsWebShellWindow.cpp:229 ============================================================= There are 27 crashes (from 3 installations) in nightly 64 with buildid 20180915101434. In analyzing the backtrace, the regression may have been introduced by patch [1] to fix bug 1486971. [1] https://hg.mozilla.org/mozilla-central/rev?node=e92f3e990b35
Flags: needinfo?(hikezoe)
Component: CSS Parsing and Computation → Widget: Cocoa
All these crashes occurred on 10.9. I'm not exactly sure which code is crashing here - is it just the access of the notification name (which is 10.10+), or is it the call to addObserver? Either way, wrapping the whole statement in a macOS version check should fix the crash. This is pretty urgent to fix because it's a startup crash. It means that Firefox can't start at all on 10.9. Sorry for missing it in the review.
That's totally my fault! I thought the notification name is just a string, so it happens nothing on older versions. So sorry for that. https://treeherder.mozilla.org/#/jobs?repo=try&revision=e5375d4a9d94c0a9b9daf51d8082123ff8e83b50 Hope this works.
Assignee: nobody → hikezoe
Status: NEW → ASSIGNED
Flags: needinfo?(hikezoe)
I have to run right now, it's a kindergarten event today, will be back within 8 hours or so.
Comment on attachment 9009412 [details] Bug 1491580 - Use NSWorkspaceAccessibilityDisplayOptionsDidChangeNotification only for MacOSX 10.10 or later versions. r=mstange Markus Stange [:mstange] has approved the revision.
Attachment #9009412 - Flags: review+
Pushed by mstange@themasta.com: https://hg.mozilla.org/integration/autoland/rev/431d82d82470 Use NSWorkspaceAccessibilityDisplayOptionsDidChangeNotification only for MacOSX 10.10 or later versions. r=mstange
Thank you Markus for reviewing and landing it!
https://hg.mozilla.org/mozilla-central/rev/431d82d82470
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Verified as fixed: it was broken here and it's back to normal.
Status: RESOLVED → VERIFIED
Great. Thanks for fixing it so quickly Hiro!
Blocks: 1492284
Comment on attachment 9009412 [details] Bug 1491580 - Use NSWorkspaceAccessibilityDisplayOptionsDidChangeNotification only for MacOSX 10.10 or later versions. r=mstange Approval Request Comment [Feature/Bug causing the regression]: bug 1486971 (but it's actually regressed only on nightly) [User impact if declined]: Without this and with bug 1486971 firefox on MacOSX 10.9 crashes [Is this code covered by automated tests?]: A test case in bug 1486971 covers this but unfortunately we don't run the test on MacOSX 10.9. We are using MacOSX10.10 on our infra IIUC. [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: I don't think we need manual test [List of other uplifts needed for the feature/fix]: This should be uplifted with bug 1486971. [Is the change risky?]: Very low [Why is the change risky/not risky?]: This just checks the MacOSX version, but it's really important for the crash. [String changes made/needed]: None
Attachment #9009412 - Flags: approval-mozilla-beta?
Comment on attachment 9009412 [details] Bug 1491580 - Use NSWorkspaceAccessibilityDisplayOptionsDidChangeNotification only for MacOSX 10.10 or later versions. r=mstange Uplift approved for 63 beta 8, thanks.
Attachment #9009412 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
This crash is still happening on 63 release, but apparently only to Mac users running 10.10: https://bit.ly/2ETsrgW. I can open a new bug if necessary.
Marcia, yes, please open a new bug for that. We should definitely fix the crash.
(In reply to Hiroyuki Ikezoe (:hiro) from comment #15) > Marcia, yes, please open a new bug for that. We should definitely fix the > crash. Filed Bug 1503424 for the 10.10 crashes.
Thank you, Marcia!
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: