72060 – new Chart: DocumentHolder::ShowUI() for new chart crashes due to reparenting and empty DockingAreaAcceptor

Issue 72060 - new Chart: DocumentHolder::ShowUI() for new chart crashes due to reparenting and empty DockingAreaAcceptor

Summary: new Chart: DocumentHolder::ShowUI() for new chart crashes due to reparenting ...

Status:	CLOSED FIXED

Alias:	None

Product:	General
Classification:	Code
Component:	ui (show other issues)
Version:	680m193
Hardware:	All All

Importance:	P2 Trivial (vote)
Target Milestone:	OOo 2.3
Assignee:	kla
QA Contact:	issues@framework

URL:
Keywords:	new_implementation

Depends on:
Blocks:

Reported:	2006-11-28 12:05 UTC by bjoern.milcke
Modified:	2007-07-17 16:09 UTC (History)
CC List:	1 user (show)

See Also:
Issue Type:	TASK
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 bjoern.milcke 2006-11-28 12:05:07 UTC

1. New Calc document
2. Enter some numbers and insert a Chart
=> Before the Hatchborder is drawn, the Office crashes in Window::SetParent()

Please fix on CWS chart2mst3.

As a workaround, I changed two lines to avoid the crash. For reproducing the
crash, un-comment the two lines starting with //HOTFIX in
embeddedobj/source/general/docholder.cxx (lines 814 and 899) or switch back to
revision 1.19.10.9.

Stack :

>	vcl680mi.dll!Window::ImplGetFirstOverlapWindow()  Line 1596 + 0x3 bytes	C++
 	vcl680mi.dll!Window::SetParent(Window * pNewParent=0x00000000)  Line 6223 +
0x8 bytes	C++
 	fwk680mi.dll!framework::LayoutManager::implts_reparentChildWindows()  Line
3937	C++
 	fwk680mi.dll!framework::LayoutManager::setDockingAreaAcceptor(const
com::sun::star::uno::Reference<com::sun::star::ui::XDockingAreaAcceptor> &
xDockingAreaAcceptor={...})  Line 3826	C++
 	embobj.dll!DocumentHolder::ShowUI(const
com::sun::star::uno::Reference<com::sun::star::frame::XLayoutManager> &
xContainerLM={...}, const
com::sun::star::uno::Reference<com::sun::star::frame::XDispatchProvider> &
xContainerDP={...}, const rtl::OUString & aContModuleName={...})  Line 814 +
0x3e bytes	C++
 	embobj.dll!OCommonEmbeddedObject::SwitchStateTo_Impl(long nNextState=4)  Line
379 + 0x1a bytes	C++
 	embobj.dll!OCommonEmbeddedObject::changeState(long nNewState=4)  Line 522 +
0xc bytes	C++
 	embobj.dll!OCommonEmbeddedObject::doVerb(long nVerbID=-1)  Line 604 + 0x10
bytes	C++
 	sfx680mi.dll!SfxInPlaceClient::DoVerb()  + 0x245 bytes	C++
 	sc680mi.dll!ScTabViewShell::ActivateObject()  + 0x374 bytes	C++
 	sc680mi.dll!FuInsertChart::FuInsertChart(ScTabViewShell * pViewSh=0x0939bf88,
Window * pWin=0x0939de60, SdrView * pView=0x0938dd10, SdrModel *
pDoc=0x093bd3d8, SfxRequest & rReq={...})  Line 700	C++
 	sc680mi.dll!ScTabViewShell::ExecDrawIns()  + 0x288 bytes	C++

Comment 1 carsten.driesner 2006-11-28 16:00:15 UTC

cd: Accepted. I have to look why reparenting and the new docking area acceptor
changes collides. As I introduced the reparenting about 1 year ago I have to
check the code again to find a good solution.

Comment 2 carsten.driesner 2006-12-05 10:38:33 UTC

cd: Fixed by MBA.

Comment 3 carsten.driesner 2006-12-05 10:39:16 UTC

cd->kla: Please verify.

Comment 4 bjoern.milcke 2006-12-05 12:28:10 UTC

Confirmed that the two hot-fix commented-out lines are uncommented now, and the
crash no longer happens.

Comment 5 IngridvdM 2007-01-18 12:10:28 UTC

changed target to 2.3

Comment 6 IngridvdM 2007-03-01 19:03:15 UTC

set keyword and changed issue type

Comment 7 kla 2007-03-07 14:17:59 UTC

verified

Comment 8 p9w.vu.31122010 2007-07-17 16:09:39 UTC

Verified fixed in m221