Problem description: When inspecting accessibility in LibreOffice using Accessibility Inspector you can only see the frame in the accessibility tree. I'm trying to bisect this to see if I can find at what commit this issue arouse. It might take a few days before I have the results from that though. Steps to reproduce: 1. Start Accessibility Inspector 2. Start LibreOfficeDev.app 3. Move the mouse over different parts of LibreOffice Current behavior: Only the title bar of the frame seems to trigger an update in Accessibility inspector. Expected behavior: When you move the cursor over a button for example, the accessibility information of the button should be displayed. Likewise for other objects. Operating System: Mac OS X Version: 4.3.0.0.alpha0+ Master
So quite a few commits did not build around the time that the issue was introduced. I suppose I need to dig a bit deeper to find the offending commit. Heres the results of my bisect session. There are only 'skip'ped commits left to test. The first bad commit could be any of: 3440440dbf433d98673d114959d862d6bbcf5de2 a41d064568f3e69238a60c0df803c746b03b2b34 ab27cf774edc3c27c2be181e1cb1a67a4551536f ab80ea279bea37f4dd919417433cfd72ca28e8bd 130833f80e89774269108cf30b2d1155a00354ce de45a49273542c9046e0cfc2b75c863573eb6c38 059cc67245a4f0a62589a0c877aea85e8859c435 c5138cad87575283cdf0670355f2b2cddae93fc8 b38629ae210b204a6d24d6e9c5c62eaaf563d494 fb9d14f6a6deb00ecd09288e5696e8df7f8f5c0e 215f87fb53164a5fc9af13acdad7fdada2117b60 be7d87bea2611eb316b9b379aebc031179d4f794 0fb67dfdb6493cb7f6fc3b97af938300f1891af4 ef8c2098a8528c0ca3cd9c66e23f4420f8660d19 4eaa08f1d4e110dd3c3f464eaf9e7c526b7c28f7 92b1348ae8e0b0cd8fa5dca34ce55fc01dc6e02f 461e0f9eaf9895ecd0eba67b35a4d2cec8a5e5c5 8c73b16f5f18f3bc1dbf9ff6c1475db56b44d304 b415ec8cad8224c58d149626ddc3365575aa855e We cannot bisect more!
Adding Bug 74121 - ACCESSIBILITY: Accessibility broken on Mac OS X in master to mac accessibility tracking bug
If I understand things correctly this is the commit that broke the build: http://cgit.freedesktop.org/libreoffice/core/commit/?id=461e0f9eaf9895ecd0eba67b35a4d2cec8a5e5c5 I checked that one out and only applied the patches called Fix fallout from 461e0f9eaf9895ecd0eba67b35a4d2cec8a5e5c5 This made it possible to build again but the accessibility is broken so I suppose there is still something left that needs some adjustment. Note that I can find the elements, toolbar, menu etc., by navigating the accessibility Hierarchy. The main windows has children as usual so it might be somewhere in the focus tracking.
Running Accessibility Verifier on LibreOffice gives a lot of window errors such as: error: AXButton - "Open", AXWindow (could not find) is not the correct window, should be AXWindow(AXStandardWindow) - "LibreOfficeDev" These errors where not present before the patch I mention in the previous comment.
I can confirm this with VoiceOver as well. I found this while testing if bug 74470 happens on Master as well. With VoiceOver on, Master builds freeze completely and have to be shut down via the Mac equivalent of Task manager.
(In reply to comment #3) > If I understand things correctly this is the commit that broke the build: > http://cgit.freedesktop.org/libreoffice/core/commit/ > ?id=461e0f9eaf9895ecd0eba67b35a4d2cec8a5e5c5 Caolán - Please take a look at this commit and see if you can determine what in the patch degraded accessibility on OSX. Thanks!
(In reply to comment #4) > Running Accessibility Verifier on LibreOffice gives a lot of window errors > such as: > error: AXButton - "Open", AXWindow (could not find) is not the correct > window, should be AXWindow(AXStandardWindow) - "LibreOfficeDev" > > These errors where not present before the patch I mention in the previous > comment. Hi Niklas, Please keep track of this change on Windows, and re-test after we've fixed the issues on OSX. It's quite possible that a patch may fix one OS but not the other; if Windows needs additional patches, it would be great for us to be on top of the problem so we may fix things sooner rather than later. Best,
@Robinson, (In reply to comment #7) > (In reply to comment #4) > ... > Please keep track of this change on Windows, and re-test after we've fixed > the issues on OSX. I don't think Niklas suggested this was in any way associated with MS Windows. AX types are all Apple accessibility functions and this issue is limited to OS X. Of course it could end up otherwise, we'll watch it across the OS builds, :)
(In reply to comment #8) > @Robinson, > (In reply to comment #7) > > (In reply to comment #4) > > ... > > Please keep track of this change on Windows, and re-test after we've fixed > > the issues on OSX. > > I don't think Niklas suggested this was in any way associated with MS > Windows. AX types are all Apple accessibility functions and this issue is > limited to OS X. Ah, verily it is true. (note to self: double-check bug comments when half-asleep...and by double-check, I probably mean triple-check) > Of course it could end up otherwise, we'll watch it across the OS builds, :) Indeed, indeed. Thank you for your diligence in this regard!
I've found two issues in the original commit and sent a patch for review [1] that fixes the problem with the window errors (see comment 4). Note however that accessibility is still broken on master after my patch. Since Accessibility has been broken on master for Mac for the last two months I'm now building from the commit that broke accessibility and then gradually applying the fixup-patches as well as my patch to see if something else has broken accessibility since or if there is still some loose ends from the mentioned commit. [1] https://gerrit.libreoffice.org/#/c/7837/
Niklas Johansson committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=37ee9b402c2264daa7f8a8bc96443d380a054cf5 fdo#74121 - Accessibility broken on Mac OS X (master) The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
The bug was quite interesting, and I had came to the same conclusion as you before I noticed your patch. AOO had "fixed" it to "look" more right but actually the method is expected to return what seems like a "wrong" thing, a NSWindow* even if it declared to return a NSView*... Fun. Thanks!
Seems the commit that originally broke accessibility caused even more problems. I've finished the build that I mention i comment 10 and it contains the same issues as master, LibreOffice easily freezes with VoiceOver running. What catches my eye is that accessible actions for buttons etc. is no longer reported in Accessibility Inspector so it might be related to that.
It seems I was a bit sloppy with my testing, the issue with missing Accessible Actions and a LibreOffice that easily freezes was introduced before the commit http://cgit.freedesktop.org/libreoffice/core/commit/?id=461e0f9eaf9895ecd0eba67b35a4d2cec8a5e5c5 So I suppose I need to go on and Bisect that issue as well.
The issue described in the first comment is no longer true and so I'm closing this as resolved, fixed. As for comment 14, I opened a new bug [1] for it since it's unrelated of the original issue. [1] https://www.libreoffice.org/bugzilla/show_bug.cgi?id=74983
(In reply to Niklas Johansson from comment #0) > Steps to reproduce: > 1. Start Accessibility Inspector Any idea how to do that on OS X 10.10? Where could I find that Accessibility Inspector?
Open Xcode and go to the menu Xcode -> Open Developer Tool -> Accessibility Inspector. I don't really remember if I had to do anything for the Accessibility Inspector to be added to that menu, at least I don't think so.
(In reply to Niklas Johansson from comment #17) > Open Xcode and go to the menu Xcode -> Open Developer Tool -> Accessibility > Inspector. Yes, thanks.
Migrating Whiteboard tags to Keywords: (a11y -> accessibility) [NinjaEdit]