Issue 102142 - sd: BasicViewFactory leaks windows
Summary: sd: BasicViewFactory leaks windows
Status: CLOSED FIXED
Alias: None
Product: Draw
Classification: Application
Component: code (show other issues)
Version: DEV300 m39
Hardware: All Linux, all
: P3 Trivial (vote)
Target Milestone: 3.4.1
Assignee: groucho266
QA Contact: issues@graphics
URL:
Keywords:
: 111455 (view as issue list)
Depends on:
Blocks: 90439 96090 97906
  Show dependency tree
 
Reported: 2009-05-22 14:22 UTC by caolanm
Modified: 2012-03-01 22:21 UTC (History)
3 users (show)

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

Attachments
proposed fix (1.06 KB, patch)
2009-05-22 14:22 UTC, caolanm 		no flags Details | Diff
replacement for first patch, move the ctor before its last user (1.18 KB, patch)
2009-05-25 11:36 UTC, caolanm 		no flags Details | Diff
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 caolanm 2009-05-22 14:22:21 UTC 
Everytime we start slideshow mode and then stop it, xrestop shows an increase of
two pixmaps. (Those pixmaps are actually the unshown icons for a top-level
window). Looking closer (with some experimental tooling I've cobbled together)
we're leaking the vcl Window from BasicViewFactory.

i.e. attached is a patch to release those resources
Comment 1 caolanm 2009-05-22 14:22:40 UTC 
Created attachment 62441 [details]
proposed fix
Comment 2 wolframgarten 2009-05-25 10:28:04 UTC 
Reassigned. Please handle.
Comment 3 caolanm 2009-05-25 11:36:05 UTC 
Created attachment 62493 [details]
replacement for first patch, move the ctor before its last user
Comment 4 clippka 2009-06-04 14:11:01 UTC 
reasigned
Comment 5 groucho266 2009-10-05 15:19:35 UTC 
Due to time constraints I change the target to OOo 3.3.
Comment 6 groucho266 2010-07-14 09:09:00 UTC 
Changing target.
Comment 7 groucho266 2010-07-14 12:49:06 UTC 
Applied patch.
Comment 8 groucho266 2010-10-15 10:13:23 UTC 
*** Issue 111455 has been marked as a duplicate of this issue. ***
Comment 9 clippka 2011-02-09 11:11:09 UTC 
verified that the patch looks good and is applied to the cws impress195