summaryrefslogtreecommitdiffstats
path: root/accessibility
Commit message (Collapse)AuthorAgeFilesLines
* typo: choosen -> chosenThomas Arnhold2014-04-141-1/+1
|
* typo: accesible -> accessibleThomas Arnhold2014-04-141-1/+1
|
* It's "its", not "it's"Tor Lillqvist2014-04-104-5/+5
| | | | Change-Id: Ieaa787afd7cc622b4750a2ee8f17f6dad934ba63
* Remove unused functionsStephan Bergmann2014-04-091-3/+0
| | | | Change-Id: I49ee400e665160daf02574a0962ba7bd0239091f
* fdo#43157 : clean up more OSL_POSTCONDMichaël Lefèvre2014-04-041-3/+4
| | | | | | | Change-Id: Iaca31b4e12c258b762b3d88fbad0fb08b4f1fc16 Reviewed-on: https://gerrit.libreoffice.org/8832 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Remove unused functionsStephan Bergmann2014-04-031-25/+6
| | | | Change-Id: Iac147aef89e7f8be2f3d16e434c76d57a950d591
* remove unnecessary scope qualifier from sal_Bool usesNoel Grandin2014-04-032-25/+25
| | | | | | i.e. convert "::sal_Bool" to "sal_Bool" Change-Id: Ie5943aee4fee617bf2670655558927ed25b7e067
* Explicitly mark overriding destructors as "virtual"Stephan Bergmann2014-04-015-5/+5
| | | | | | | | It appears that the C++ standard allows overriding destructors to be marked "override," but at least some MSVC versions complain about it, so at least make sure such destructors are explicitly marked "virtual." Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
* First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann2014-03-2669-1237/+1237
| | | | | | | ...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
* svtools: sal_Bool->boolNoel Grandin2014-03-252-8/+8
| | | | Change-Id: Ifd3e643dbc6755839ad4af73ae141fd115ddb4f4
* Reduce IAccessibleTabListBox to what's actually usedStephan Bergmann2014-03-251-2/+2
| | | | | | | | ...the calls to getAccessibleChild(0) and getHeaderBar(BBYTE_COLUMNHEADERBAR) ultimately all ended up at AccessibleBrowseBox::implGetHeaderBar(BBTYPE_COLUMNHEADERBAR). Change-Id: Ie25659e19d0d1ce2dcde2d1440ecc69b4fa265b0
* Remove dead codeStephan Bergmann2014-03-254-73/+0
| | | | Change-Id: Ia9035189bd01d3a3e5e3222760fc392f3f78880f
* Remove dead codeStephan Bergmann2014-03-2520-113/+4
| | | | Change-Id: Id522113f1ba0c9e7b1356acc01a79e0e90a5e752
* svtools: sal_Bool->boolNoel Grandin2014-03-248-12/+12
| | | | Change-Id: Ic765d1a28960dbeca9501852b5e84cfdfe6ac47e
* Use cppu::BaseMutex and remove need for Paragraph/ParagraphImpl splitStephan Bergmann2014-03-182-175/+163
| | | | Change-Id: I790e14feb50fefe3b2efa505118b332d065970fc
* Find places where OUString and OString are passed by value.Noel Grandin2014-03-182-4/+4
| | | | | | | | | | It's not very efficient, because we generally end up copying it twice - once into the parameter and again into the destination OUString. So I create a clang plugin that finds such places and generates a warning so that we can convert them to pass-by-reference. Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
* fdo#63154 Remove some solar.h referencesAlexandre Vicenzi2014-03-152-6/+5
| | | | | | | | | | Remove some solar.h from tools. Replace sal_uLong by size_t, sal_uInt32 and sal_Int32. Change-Id: I38961db046337a3ea4ef75d64afa424d81391f34 Reviewed-on: https://gerrit.libreoffice.org/8582 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* OAccessibleContextWrapperHelper does not implement XAccessibleContextStephan Bergmann2014-03-141-3/+3
| | | | | | | ...but rather some (non-virtual) base functionality for use in derived classes (that do implement XAccessibleContext) Change-Id: Idb0023906108db22bb9696245f07b9a4c053a0d1
* a crash sometimes seen in a11y in baseCaolán McNamara2014-03-131-14/+15
| | | | Change-Id: Id8b7c8394a4e8bc1477c34056b8014d311186d7e
* No need for implGetBounds to be SAL_CALLStephan Bergmann2014-03-1113-13/+13
| | | | Change-Id: I4eb1f998de57a4ceeb7990c6e4a964ce0d07ea88
* accessibility: simplify deprecated XTypeProvider.getImplementationIdStephan Bergmann2014-03-109-75/+9
| | | | Change-Id: I7f8848adf245a4fdf56bff67cc15ee0efb818c43
* made ListBox handle more than 64k elements, fdo#61520 relatedEike Rathke2014-03-055-34/+36
| | | | | | | | | | | | ListBox and related now handle up to sal_Int32 elements correctly. sal_Int32 instead of sal_Size or size_t because of UNO and a11y API. Also disentangled some of the mess of SvTreeList and other containers regarding sal_uInt16, sal_uLong, long, size_t, ... type mixtures. Change-Id: Idb6e0ae689dc5bc2cf980721972b57b0261e688a Reviewed-on: https://gerrit.libreoffice.org/8460 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
* remove unused code in accessibility::DocumentNoel Grandin2014-03-052-41/+0
| | | | | | | accessibility::Document::justifySelection(TextPaM&, TextPaM&) accessibility::Document::notifySelectionChange(int, int) Change-Id: I3a5c79ada69c1530ebbbf386bf271f3a83843815
* Revert "Let's use RADIO_BUTTON and CHECK_BUTTON roles"Stephan Bergmann2014-02-261-6/+4
| | | | | This reverts commit 97d4f7e07462cbc33e8ebf0bc2bbf8a17c232d50, which appears to be incomplete: CHECK_BUTTON is not defined anywhere.
* Let's use RADIO_BUTTON and CHECK_BUTTON rolesJulien Nabet2014-02-261-4/+6
| | | | | | Thank you Caolán and Niklas for your support! Change-Id: Ied978b36cc309b2668889d878a309d1a1400633a
* cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann2014-02-26123-1926/+1926
| | | | Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
* cppcheck: duplicateIfJulien Nabet2014-02-261-7/+9
| | | | | | | | | | - replace duplicate TIB_DROPDOWN test with TIB_DROPDOWN + TIB_DROPDOWNONLY - consider also TIB_RADIOCHECK as AccessibleRole::TOGGLE_BUTTON Change-Id: Id159349ae1515681f47c373ba3d839ed406284ab Reviewed-on: https://gerrit.libreoffice.org/8164 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Remove visual noise from accessibilityAlexander Wilms2014-02-2578-314/+314
| | | | | | | Change-Id: I876e5c4d1991b35c1dc2cea709d8a2ddf781b012 Reviewed-on: https://gerrit.libreoffice.org/8225 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Replace exisiting TriState, AutoState with more generic TriStateStephan Bergmann2014-02-245-14/+14
| | | | Change-Id: Ida05478aae5a379775c671e0c2f2851d820d78be
* Remove unneccessary commentsAlexander Wilms2014-02-2353-1564/+1318
| | | | | | | Change-Id: I939160ae72fecbe3d4a60ce755730bd4c38497fb Reviewed-on: https://gerrit.libreoffice.org/8182 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Remove vcl/settings.hxx from vcl/svapp.hxx and vcl/outdev.hxxTobias Lippert2014-02-2016-3/+16
| | | | | | | | | | | | | Added vcl/settings.hxx to all cxx files which require it. This helps to speed up compilation after changes to the settings. Conflicts: sc/source/ui/dbgui/pvlaydlg.cxx Change-Id: I211a0735c47f72d6879f6f15339355abfe0e3cf4 Reviewed-on: https://gerrit.libreoffice.org/7933 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Related: fdo#74242 hook up a selection and caret change for multiline editsCaolán McNamara2014-02-191-0/+1
| | | | | | | | | | | | | a) split the textengine selection changed broadcast into a text selection changed and text caret change event b) handle both in existing places that handled the text selection event c) listen for them in VclMultiLineEdit and translate to VCLEVENT_EDIT_SELECTIONCHANGED VCLEVENT_EDIT_CARETCHANGED events d) profit from the VCLEVENT_EDIT_SELECTIONCHANGED and VCLEVENT_EDIT_CARETCHANGED handling in accessibility/source/standard/vclxaccessibleedit.cxx for VCLXAccessibleEdits Change-Id: I09187e76ae4eb189ee9469e388374154087faf80
* Related: fdo#74242 #i104470# we now have a seperate CARET_CHANGED eventCaolán McNamara2014-02-191-17/+14
| | | | | | | so cursoring left and right in normal non-multiline edit widgets should give a11y cursor-changed events Change-Id: I91e86e43d049793912b3ffcc2802bb9aa4b3857c
* accessibility: add some missing locks in new codeMichael Stahl2014-02-114-9/+18
| | | | Change-Id: Ic38a07c592a89973ca1df807505f87910e84a17f
* the return of callcatcherCaolán McNamara2014-02-102-15/+0
| | | | Change-Id: I447ebcc61ff061bce15678c6fcfd3d0a4669908f
* std::less is the default hereStephan Bergmann2014-02-071-1/+1
| | | | Change-Id: I57428772187d3487668eca2f553c74e18e4f736e
* auto_ptr -> scoped_ptrStephan Bergmann2014-02-074-15/+9
| | | | Change-Id: I05c461deca118f1a1b75d94b4d00ce797987e8d6
* Avoid unnecessary heap allocationStephan Bergmann2014-02-074-22/+8
| | | | Change-Id: I7bc1a0960cc100c51b90c50838a5ef620f3c763f
* Doxygen warnings correctedChris Sherlock2014-02-052-3/+3
| | | | | | | | | | | | | | | * Doxygen spits out a lot of warnings about not being able to find match function signatures, etc. This is because in some headers we have a using namespace statement, in others it gets confused between ::Window and Window (!). * Wrong use of tags: + Lots of @seealso - should be @see + Wrong usage of @overload - corrected with the right function signature + HTML tags that doxygen doesn't recognize removed Change-Id: I1c2eed941619b8764dbfcfc5ab38027518cdf261
* fix ambiguitiesCaolán McNamara2014-02-042-3/+3
| | | | Change-Id: Ice46cfbe8a21c5e641e09070128b8f412411c9d9
* Related: #i123862# use O*String's isEmpty() method...Herbert Dürr2014-02-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | to check for emptiness in modules starting with 'a' Patch-by: j.nitschke@ok.de Review-by: hdu@apache.org (cherry picked from commit 40b2ffbb1036c2f2db50965525009213d8fd6343) Conflicts: accessibility/source/extended/accessibleiconchoicectrl.cxx accessibility/source/extended/textwindowaccessibility.cxx accessibility/source/standard/vclxaccessiblebox.cxx accessibility/source/standard/vclxaccessibletoolboxitem.cxx automation/source/server/recorder.cxx automation/source/server/statemnt.cxx avmedia/source/framework/mediacontrol.cxx avmedia/source/framework/soundhandler.cxx avmedia/source/gstreamer/gstplayer.cxx avmedia/source/viewer/mediawindow.cxx Change-Id: I8e6d6754282634f6931a2e7f73f08f70c23d452e
* bool improvementsStephan Bergmann2014-01-285-7/+7
| | | | Change-Id: I09ac60e907b7c0a78934dce05343e2ba2b6df49f
* Resolves: #i124008# listbox entry's are selected automatically...Steve Yin2014-01-282-0/+2
| | | | | | | | on mouse over instead of mouse click (cherry picked from commit ae97dcb5b8d2dead81b007c5c71daf608f0c7bc3) Change-Id: Ib1ce0db60785ddc96473ef4ce88b578287956164
* coverity#983594 Uncaught exceptionCaolán McNamara2014-01-282-4/+10
| | | | Change-Id: Ic49f88b44216a24df8a236b755d22b2537245c5f
* coverity#706219 Uncaught exceptionCaolán McNamara2014-01-282-4/+10
| | | | Change-Id: I1c2566337347679453bee46a78a4a0ecd7fdd2fb
* fdo#63154 Removed unused solar.h referenceAlexandre Vicenzi2014-01-231-3/+0
| | | | | | | Change-Id: I6b2f9e04e2232570c8bb9f36252b23a70c5f7525 Reviewed-on: https://gerrit.libreoffice.org/7604 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
* coverity#1132668 : missing break in switchNorbert Thiebaud2014-01-191-84/+81
| | | | Change-Id: Ia759081e7ccc9105898f25025ea567d900d33b13
* coverity#1158122 : Explicit null dereferencedNorbert Thiebaud2014-01-191-2/+4
| | | | Change-Id: I6ffec6e69ca5ddb01998653d2f7637ed062bdc80
* coverity#1132662 Dereference after null checkNorbert Thiebaud2014-01-191-1/+1
| | | | Change-Id: Iaccf8bb7903702d2bed3c4fc2195172e18ead6c7
* coverity#1132661 Dereference after null checkNorbert Thiebaud2014-01-191-13/+10
| | | | Change-Id: Ida147599810b0aff4a7ebd00b90475b1c6cd1366