Closed
Bug 339371
Opened 18 years ago
Closed 18 years ago
Need [NSApp run] on the stack
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mark, Assigned: mark)
References
Details
Attachments
(2 files)
|
6.08 KB,
patch
|
jaas
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
|
3.70 KB,
text/plain
|
Details |
When we're running under Cocoa, we should make sure to run with [NSApp run] beneath us on the stack. Josh and I are seeing weird crashes otherwise. Testcase: attempt to log in to an https site that uses http authentication.
|
Assignee | |
Comment 1•18 years ago
|
||
Attachment #223490 -
Flags: review?(joshmoz)
Attachment #223490 -
Flags: review?(joshmoz) → review+
|
|
Assignee | |
Updated•18 years ago
|
Attachment #223490 -
Flags: superreview?(darin)
|
||
Updated•18 years ago
|
Attachment #223490 -
Flags: superreview?(darin) → superreview+
I am still seeing weird crashes with this patch (I've been using it all day).
|
|
Assignee | |
Comment 3•18 years ago
|
||
I wonder if this is caused by running after terminate (if the terminate you see on the stack isn't the first terminate) - in that case, I have a patch...somewhere...that might help...
|
||
Comment 4•18 years ago
|
||
*** Bug 339942 has been marked as a duplicate of this bug. ***
|
|
Assignee | |
Comment 5•18 years ago
|
||
Checked in on the trunk. May need more work (see comment 2).
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•