Issue 71063 - Impress crashes on delete several outlines in outline view in new window
Summary: Impress crashes on delete several outlines in outline view in new window
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: editing (show other issues)
Version: OOo 2.0.4
Hardware: All Windows 2000
: P2 Trivial (vote)
Target Milestone: OOo 2.3
Assignee: christian.guenther
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-01 06:01 UTC by jlcheng
Modified: 2007-09-25 10:47 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description jlcheng 2006-11-01 06:01:47 UTC
Steps to replicate(use 2.0.4rc3):

1) Start impress.
2) Insert 3 pages with layout "Title, Text". And give some title text to all pages.
3) Select 1st page. Select outline view. Use "Window"->"New Window" to open a
new window.
4) Select first two titles and press "delete" key. Then click somewhere, OOo crash.
Comment 1 wolframgarten 2006-11-01 07:29:20 UTC
Reproducible. Reassigned.
Bug is reproducible down to 2.0 final.
Comment 2 wolframgarten 2006-11-01 08:03:25 UTC
Target set.
Comment 3 clippka 2006-12-04 18:18:51 UTC
this issue we have since the beginning of time, not sure yet if I'm able to fix
this without refactoring the complete outline view (month,years...)...
Comment 4 clippka 2007-01-10 10:29:03 UTC
ok, Malte gave me an idea on how to fix this issue. Currently we create one
EditEngine and one EditEngine view for each outliner window. A better sollution
is to only have one EditEngine and use multiple EditEngine views for each
outliner window. The EditEngine already supports multiple views, the crash is
caused by the redundant data in the two open EditEngines so this should solve
the problem.

Never the less, this is a not so small rework, so I prefer not to add it this
short before code freeze of 2.2. Retargeting to 2.2.1 so I can fix it early for
the next bugfix release.
Comment 5 clippka 2007-04-11 14:55:21 UTC
Had no time to fix this yet for 2.2.1

Since this is no regression I shift target to 2.3
Comment 6 liyuan 2007-06-07 04:50:06 UTC
cl: I have sent the patch to Christian to review. 
Comment 7 liyuan 2007-06-08 05:16:49 UTC
I have commited the patch into the CWS impress122.
Comment 8 clippka 2007-06-18 10:59:32 UTC
verified in cws, issue is fixed
Comment 9 clippka 2007-06-18 11:00:49 UTC
qa, please verify
Comment 10 christian.guenther 2007-06-21 14:21:31 UTC
CGU: Verified in cws impress122
Comment 11 christian.guenther 2007-09-25 10:47:08 UTC
CGU: integrated in src680m228 and oog680m5