Issue 105958 - Crash when pasting slides in Impress with active accessibility
Summary: Crash when pasting slides in Impress with active accessibility
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: code (show other issues)
Version: OOo 1.0.0
Hardware: All Solaris
: P2 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: stefan.baltzer
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks: 99999
  Show dependency tree
 
Reported: 2009-10-16 12:27 UTC by groucho266
Modified: 2009-12-07 11:35 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description groucho266 2009-10-16 12:27:24 UTC 
How to reproduce:

1. Create new Impress document.
2. Create one or more shapes on the first slide.
3. In the slide sorter bar copy and paste the slide.
4. In the dialog box that pops up click OK to insert after the current slide.

=> crash on Solaris
Comment 1 groucho266 2009-10-16 12:28:07 UTC 
Note that accessibility support has to be active.
Comment 2 groucho266 2009-10-16 12:35:34 UTC 
The crash is caused by an unexpected exception:

1. The XAccessibleContext::getAccessibleChild method is declared to raise (among
others) the IndexOutOfBoundsException.

2. The implementing methods for the Impress edit view (in classes
AccessibleDocumentViewBase and AccessibleDrawDocumentView) dropped this exception.

3. When a slide is copied and pasted then, for a reason that is not important
here, this very exception is thrown in AccessibleDrawDocumentView.  Because the
exception is not part of the throw part of the declaration it is treated as
unexpected exception and therefore leads to an abortion of  the whole process.


The root cause of all this is of course item 2: dropping the
IndexOutOfBoundsException from the declaration.
Comment 3 groucho266 2009-10-16 16:10:41 UTC 
Fixed by adding IndexOutOfBoundsException to the list of supported exceptions of
getAccessibleChild().
Comment 4 groucho266 2009-10-19 14:30:37 UTC 
@es: Please verify.
Comment 5 eric.savary 2009-10-26 14:12:49 UTC 
Reassigned
Comment 6 stefan.baltzer 2009-10-26 16:12:10 UTC 
Verified in CWS impressaccessiblity6.
Comment 7 stefan.baltzer 2009-10-26 16:13:05 UTC 
Reassigned to me.
Comment 8 stefan.baltzer 2009-12-07 11:35:49 UTC 
OK in OOO320_m7. Closed.