Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | toolkit/qa/unoapi VCLXAccessibleMenuItem::doAccessibleAction deadlock | ||||||
---|---|---|---|---|---|---|---|
Product: | gsl | Reporter: | Stephan Bergmann <stephan.bergmann.secondary> | ||||
Component: | code | Assignee: | philipp.lohmann | ||||
Status: | CLOSED FIXED | QA Contact: | issues@gsl <issues> | ||||
Severity: | Trivial | ||||||
Priority: | P3 | CC: | issues | ||||
Version: | DEV300m76 | ||||||
Target Milestone: | 3.4.0 | ||||||
Hardware: | Mac | ||||||
OS: | Mac OS X, all | ||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||
Developer Difficulty: | --- | ||||||
Attachments: |
|
Description
Stephan Bergmann
2010-04-21 15:08:08 UTC
Created attachment 69031 [details]
deadlocking stacks
worked around for now by disabling the affected tests in toolkit/qa/unoapi/toolkit.sce, see <http://hg.services.openoffice.org/cws/sb120/rev/e8b5766f7e32>; please revert when fixing this issue target The reason for this is that vcl's menus can be popped up without modality whereas mac's naitve menus cannot. The resulting deadlock from the original report could be worked around by executing the native menu only in the solar thread; but that is not a fix for the issue at hand: now the menu runs until the user closes it (selecting an entry or clicking outside), which also brings the test to a halt. The workaround now is to not use the native menus in this case; instead using normal vcl menus. This does not test the actual product anymore, but actually the testtool needs to do the same for similar reasons (actually testtool's case is more involved, but I digress). I don't know however whether there will be further side effects in this case given that now vcl popups are used without a vcl menubar. Some testing did not result in immediate problems however and the test at least runs through. fixed in CWS pl07 see issue 113489 for a follow-up problem verified (fix was transplanted to CWS sb127, which was run many times on many machines to check subsequenttests status) seen in DEV300m87, closing |