1684903 - Add nsAutoreleasePools to functions that interacts with NSScreen.

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Attachment: Bug 1684903 - Add nsAutoreleasePools to functions that interacts with NSScreen.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/111897d51399
Add nsAutoreleasePools to functions that interacts with NSScreen. r=mstange

Comment 3

[Jeff Muizelaar [:jrmuizel]]

* thread #1, name = 'MainThread', queue = 'com.apple.main-thread', stop reason = breakpoint 1.1
  * frame #0: 0x00007fff6b02c68c libobjc.A.dylib`objc_autoreleaseNoPool
    frame #1: 0x00007fff6b02c68a libobjc.A.dylib`AutoreleasePoolPage::autoreleaseNoPage(objc_object*) + 282
    frame #2: 0x00007fff6b013636 libobjc.A.dylib`objc_object::rootAutorelease2() + 32
    frame #3: 0x00007fff2f3dc6c2 AppKit`+[_NSScreenConfiguration latestScreens] + 465
    frame #4: 0x00000001046e8a3c XUL`mozilla::widget::GfxInfo::FindMonitors(this=<unavailable>, aCx=0x000000011552e000, aOutArray=JS::HandleObject @ 0x00007ffeefbfcec8) at GfxInfo.mm:500:28 [opt]
    frame #5: 0x00000001046826f3 XUL`mozilla::widget::GfxInfoBase::GetMonitors(this=0x0000000115794ac0, aCx=0x000000011552e000, aResult=JS::MutableHandleValue @ r15) at GfxInfoBase.cpp:1467:17 [opt]
    frame #6: 0x0000000101c496de XUL`NS_InvokeByIndex at xptcinvoke_asm_x86_64_unix.S:101
    frame #7: 0x0000000102637139 XUL`XPCWrappedNative::CallMethod(XPCCallContext&, XPCWrappedNative::CallMode) [inlined] CallMethodHelper::Invoke(this=<unavailable>) at XPCWrappedNative.cpp:1620:10 [opt]
    frame #8: 0x0000000102637119 XUL`XPCWrappedNative::CallMethod(XPCCallContext&, XPCWrappedNative::CallMode) [inlined] CallMethodHelper::Call(this=<unavailable>) at XPCWrappedNative.cpp:1176 [opt]
    frame #9: 0x0000000102636630 XUL`XPCWrappedNative::CallMethod(ccx=<unavailable>, mode=<unavailable>) at XPCWrappedNative.cpp:1142 [opt]
    frame #10: 0x000000010263867f XUL`XPC_WN_CallMethod(cx=<unavailable>, argc=<unavailable>, vp=<unavailable>) at XPCWrappedNativeJSOps.cpp:925:10 [opt]
    frame #11: 0x0000000105baedce XUL`js::InternalCallOrConstruct(JSContext*, JS::CallArgs const&, js::MaybeConstruct, js::CallReason) [inlined] CallJSNative(cx=0x000000011552e000, native=(XUL`XPC_WN_CallMethod(JSContext*, unsigned int, JS::Value*) at XPCWrappedNativeJSOps.cpp:901), reason=<unavailable>, args=0x00007ffeefbfd5a8)(JSContext*, unsigned int, JS::Value*), js::CallReason, JS::CallArgs const&) at Interpreter.cpp:503:13 [opt]
    frame #12: 0x0000000105baed81 XUL`js::InternalCallOrConstruct(cx=0x000000011552e000, args=0x00007ffeefbfd5a8, construct=<unavailable>, reason=<unavailable>) at Interpreter.cpp:594 [opt]
    frame #13: 0x0000000105baf6ef XUL`InternalCall(cx=<unavailable>, args=<unavailable>, reason=<unavailable>) at Interpreter.cpp:647:10 [opt] [artificial]
    frame #14: 0x0000000105ba8aef XUL`Interpret(JSContext*, js::RunState&) [inlined] js::CallFromStack(cx=0x000000011552e000, args=0x00007ffeefbfd5a8) at Interpreter.cpp:651:10 [opt]
    frame #15: 0x0000000105ba8ae5 XUL`Interpret(cx=0x000000011552e000, state=<unavailable>) at Interpreter.cpp:3309 [opt]
    frame #16: 0x0000000105b9fd21 XUL`js::RunScript(cx=0x000000011552e000, state=0x00007ffeefbfd9b0) at Interpreter.cpp:473:13 [opt]
    frame #17: 0x0000000105baf134 XUL`js::InternalCallOrConstruct(cx=0x000000011552e000, args=0x00007ffeefbfda38, construct=NO_CONSTRUCT, reason=<unavailable>) at Interpreter.cpp:619:13 [opt]
    frame #18: 0x0000000105bb00d2 XUL`js::CallGetter(JSContext*, JS::Handle<JS::Value>, JS::Handle<JS::Value>, JS::MutableHandle<JS::Value>) [inlined] InternalCall(cx=0x000000011552e000, args=0x00007ffeefbfda38, reason=Getter) at Interpreter.cpp:647:10 [opt]
    frame #19: 0x0000000105bb008a XUL`js::CallGetter(JSContext*, JS::Handle<JS::Value>, JS::Handle<JS::Value>, JS::MutableHandle<JS::Value>) [inlined] js::Call(cx=0x000000011552e000, fval=JS::HandleValue @ r12, thisv=JS::HandleValue @ r15, args=0x00007ffeefbfda38, rval=JS::MutableHandleValue @ r14, reason=Getter) at Interpreter.cpp:664 [opt]
    frame #20: 0x0000000105bb007b XUL`js::CallGetter(cx=0x000000011552e000, thisv=JS::HandleValue @ r15, getter=JS::HandleValue @ r12, rval=JS::MutableHandleValue @ r14) at Interpreter.cpp:788 [opt]
    frame #21: 0x0000000105d7cc83 XUL`js::NativeGetProperty(JSContext*, JS::Handle<js::NativeObject*>, JS::Handle<JS::Value>, JS::Handle<JS::PropertyKey>, JS::MutableHandle<JS::Value>) at NativeObject.cpp:2131:12 [opt]
    frame #22: 0x0000000105d7cc4a XUL`js::NativeGetProperty(JSContext*, JS::Handle<js::NativeObject*>, JS::Handle<JS::Value>, JS::Handle<JS::PropertyKey>, JS::MutableHandle<JS::Value>) [inlined] bool GetExistingProperty<(js::AllowGC)1>(cx=0x000000011552e000, receiver=js::MaybeRooted<JS::Value, js::CanGC>::HandleType @ 0x00007ffeefbfdad0, obj=<unavailable>, shape=<unavailable>, vp=js::MaybeRooted<JS::Value, js::CanGC>::MutableHandleType @ 0x00007ffeefbfdad8)1>::HandleType, js::MaybeRooted<js::NativeObject*, (js::AllowGC)1>::HandleType, js::MaybeRooted<js::Shape*, (js::AllowGC)1>::HandleType, js::MaybeRooted<JS::Value, (js::AllowGC)1>::MutableHandleType) at NativeObject.cpp:2161 [opt]
    frame #23: 0x0000000105d7cc4a XUL`js::NativeGetProperty(JSContext*, JS::Handle<js::NativeObject*>, JS::Handle<JS::Value>, JS::Handle<JS::PropertyKey>, JS::MutableHandle<JS::Value>) at NativeObject.cpp:2306 [opt]
    frame #24: 0x0000000105d7c196 XUL`js::NativeGetProperty(cx=0x000000011552e000, obj=<unavailable>, receiver=JS::HandleValue @ 0x00007ffeefbfdad0, id=<unavailable>, vp=JS::MutableHandleValue @ 0x00007ffeefbfdad8) at NativeObject.cpp:2343 [opt]
    frame #25: 0x0000000105bb2172 XUL`js::GetProperty(JSContext*, JS::Handle<JS::Value>, JS::Handle<js::PropertyName*>, JS::MutableHandle<JS::Value>) [inlined] js::GetProperty(cx=<unavailable>, obj=<unavailable>, receiver=<unavailable>, id=<unavailable>, vp=<unavailable>) at ObjectOperations-inl.h:116:10 [opt]
    frame #26: 0x0000000105bb2145 XUL`js::GetProperty(JSContext*, JS::Handle<JS::Value>, JS::Handle<js::PropertyName*>, JS::MutableHandle<JS::Value>) [inlined] js::GetProperty(cx=0x000000011552e000, obj=<unavailable>, receiver=<unavailable>, name=<unavailable>, vp=MutableHandle<JS::Value> @ r14) at ObjectOperations-inl.h:123 [opt]
    frame #27: 0x0000000105bb2125 XUL`js::GetProperty(cx=0x000000011552e000, v=JS::HandleValue @ r12, name=js::HandlePropertyName @ r15, vp=JS::MutableHandleValue @ r14) at Interpreter.cpp:4648 [opt]
    frame #28: 0x0000000105ba6063 XUL`Interpret(JSContext*, js::RunState&) [inlined] GetPropertyOperation(cx=0x000000011552e000, fp=<unavailable>, script=<unavailable>, pc=<unavailable>, lval=<unavailable>, vp=<unavailable>) at Interpreter.cpp:219:10 [opt]
    frame #29: 0x0000000105ba6019 XUL`Interpret(cx=<unavailable>, state=<unavailable>) at Interpreter.cpp:3000 [opt]
    frame #30: 0x0000000105b9fd21 XUL`js::RunScript(cx=0x000000011552e000, state=0x00007ffeefbfe100) at Interpreter.cpp:473:13 [opt]
    frame #31: 0x0000000105baf134 XUL`js::InternalCallOrConstruct(cx=0x000000011552e000, args=0x00007ffeefbfe1c8, construct=NO_CONSTRUCT, reason=<unavailable>) at Interpreter.cpp:619:13 [opt]
    frame #32: 0x0000000105baf6ef XUL`InternalCall(cx=<unavailable>, args=<unavailable>, reason=<unavailable>) at Interpreter.cpp:647:10 [opt] [artificial]
    frame #33: 0x0000000105baf74c XUL`js::Call(cx=0x000000011552e000, fval=JS::HandleValue @ scalar, thisv=JS::HandleValue @ scalar, args=0x00007ffeefbfe1c8, rval=JS::MutableHandleValue @ r14, reason=<unavailable>) at Interpreter.cpp:664:8 [opt]
    frame #34: 0x0000000105eced4d XUL`JS_CallFunctionValue(cx=0x000000011552e000, obj=JS::HandleObject @ 0x00007ffeefbfe1a8, fval=JS::HandleValue @ rbx, args=0x00007ffeefbfe330, rval=JS::MutableHandleValue @ r13) at jsapi.cpp:2798:10 [opt]
    frame #35: 0x0000000102632728 XUL`nsXPCWrappedJS::CallMethod(this=<unavailable>, methodIndex=<unavailable>, info=<unavailable>, nativeParams=<unavailable>) at XPCWrappedJSClass.cpp:970:17 [opt]
    frame #36: 0x0000000101c4a9f5 XUL`::PrepareAndDispatch(self=0x0000000110373900, methodIndex=<unavailable>, args=<unavailable>, gpregs=0x00007ffeefbfe740, fpregs=0x00007ffeefbfe770) at xptcstubs_x86_64_darwin.cpp:117:37 [opt]
    frame #37: 0x0000000101c4987b XUL`SharedStub + 91
    frame #38: 0x0000000101c132b7 XUL`NS_CreateServicesFromCategory(aCategory=<unavailable>, aOrigin=0x0000000000000000, aObserverTopic="profile-after-change", aObserverData=<unavailable>) at nsCategoryManager.cpp:686:19 [opt]
    frame #39: 0x0000000105ad4fdd XUL`nsXREDirProvider::DoStartup(this=0x00007ffeefbfecd0) at nsXREDirProvider.cpp:982:11 [opt]
    frame #40: 0x0000000105acc3b7 XUL`XREMain::XRE_mainRun(this=0x00007ffeefbfec90) at nsAppRunner.cpp:4913:16 [opt]
    frame #41: 0x0000000105acd122 XUL`XREMain::XRE_main(this=0x00007ffeefbfec90, argc=5, argv=0x00007ffeefbff260, aConfig=<unavailable>) at nsAppRunner.cpp:5330:8 [opt]
    frame #42: 0x0000000105acd468 XUL`XRE_main(argc=<unavailable>, argv=<unavailable>, aConfig=<unavailable>) at nsAppRunner.cpp:5389:21 [opt]
    frame #43: 0x00000001000027ac firefox`main [inlined] do_main(argc=<unavailable>, argv=0x00007ffeefbff260, envp=0x00007ffeefbff290) at nsBrowserApp.cpp:219:22 [opt]
    frame #44: 0x000000010000251a firefox`main(argc=<unavailable>, argv=<unavailable>, envp=0x00007ffeefbff290) at nsBrowserApp.cpp:337 [opt]
    frame #45: 0x00007fff6c1c4cc9 libdyld.dylib`start + 1
(lldb) bt 20

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/111897d51399