Issue 35524 - Make view shell creation synchronous
Summary: Make view shell creation synchronous
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: code (show other issues)
Version: 680m56
Hardware: All All
: P2 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: groucho266
QA Contact: issues@graphics
URL:
Keywords:
: 36665 (view as issue list)
Depends on: 33773 34448
Blocks: 31283 32314 32926
  Show dependency tree
 
Reported: 2004-10-14 12:07 UTC by groucho266
Modified: 2006-01-13 14:27 UTC (History)
1 user (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 2004-10-14 12:07:17 UTC 
When a new document and the associated ViewShellBase object are created the
stacked sub view shells are created asynchronously.  This leads to several
problems (see issue 31283, issue 32314, issue 32926, issue 29592, issue 34941.)
One problem is that the controller is not available directly after creation of
the ViewShellBase object.
Comment 1 groucho266 2004-10-14 12:13:21 UTC 
Fixed by making various changes:
1. The PaneManager::LateInit() creates the view shell of the center pane
synchronously.  The shells of the other panes are still created asynchronously.
2. Creation of the controller and registration at the frame has been moved from
the PaneManager to the Init() methods of the view shells.
3. Reorganization of the PaneManager.  It now uses a configuration for each
shell type that can be displayed in the center pane.  A configuration stores
information which view shell to display in which pane and the visibility of the
panes themself.
Comment 2 groucho266 2004-10-14 15:21:58 UTC 
Issue 34448 removes the necessity to have a controller ready in the constructor
of the ViewShell class (which is not possible.)
Comment 3 groucho266 2004-10-14 15:23:43 UTC 
Issue 33773 fixes a bug with the slot state cache that prevents the synchronous
creation of view shell.
Comment 4 bjoern.milcke 2004-10-22 16:26:54 UTC 
source code changes look good and the mentioned issues that needed this fix seem
to be fixed => verified
Comment 5 groucho266 2004-11-04 16:26:51 UTC 
*** Issue 36665 has been marked as a duplicate of this issue. ***
Comment 6 groucho266 2006-01-13 14:27:32 UTC 
Closing.