Issue 92711 - Error: SdrPageView::DrawLayer: Creating temporary SdrPageWindow (ObjectContact), this should never be needed (!)
Summary: Error: SdrPageView::DrawLayer: Creating temporary SdrPageWindow (ObjectContac...
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: ui (show other issues)
Version: DEV300m29
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: christoph.lukasiak
QA Contact: issues@sw
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-12 14:50 UTC by Frank Schönheit
Modified: 2013-08-07 14:43 UTC (History)
2 users (show)

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

Attachments
document to reproduce the bug case (8.37 KB, application/vnd.oasis.opendocument.text)
2008-08-12 15:05 UTC, Frank Schönheit 		no flags Details
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 Frank Schönheit 2008-08-12 14:50:27 UTC 
- open the attached document
- mark the frame
- open the context menu for the frame
- cancel the context menu with ESC
=> Error: SdrPageView::DrawLayer: Creating temporary SdrPageWindow
  (ObjectContact), this should never be needed (!)

From now on, every mark/unmark of the frame will also trigger this assertion (it
won't trigger it until you opened the context menu once).

In some cases, it even crashes - however, I suspect this has to do with the
assertions, since the crash does not happen when you re-route the assertions
into a window instead of a message box.
Comment 1 Frank Schönheit 2008-08-12 14:55:15 UTC 
assertion bug, hindering work with the non-product release => targeting to the
next feasible non-micro release
Comment 2 Frank Schönheit 2008-08-12 15:05:57 UTC 
Created attachment 55722 [details]
document to reproduce the bug case
Comment 3 Armin Le Grand 2008-09-09 12:51:50 UTC 
AW: Good catch, this is indeed an error in SW and should not happen. SW somehow
creates a bitmap representation of the FlyFrame (for what purpose ever) and
exchanges the target device to a local VDev (where a MetaFile seems to be
recorded). The result is only used temporarily, need to ask OD if this is needed
at all...
Fixing involves correctly using DLPrePaint2/DLPostPaint2 as always when using
DrawingLayer in SW. Added and works as expected (using the prepared
SdrPageWindow, patching with a new SdrPaintWindow in SdrPageView::DrawLayer).
Still open: ask OD if this is needed at all anymore...
Comment 4 Armin Le Grand 2008-09-09 12:55:18 UTC 
AW: Commited changes to SW.
Comment 5 Armin Le Grand 2008-09-10 16:39:30 UTC 
AW: OD checked that it's still needed, so done for now.
Comment 6 Armin Le Grand 2008-10-08 12:11:43 UTC 
AW->WG: Unfortunately You will have to install a non-pro version for seeing that
the assertion is no longer there; even a compare non-pro master version to see
that it was there. I have built unxlngi6 and wntmsci12 for that.
Comment 7 wolframgarten 2008-10-15 09:06:47 UTC 
Verified in CWS.
Comment 8 wolframgarten 2009-01-19 13:09:15 UTC 
Reassigned.
Comment 9 larysahitekschool 2009-05-01 19:20:37 UTC 
Verified in OOo-dev 3.2.0 DEV300m44 WinXP. No problem.
Comment 10 natalia_hitekschool 2009-05-02 01:17:03 UTC 
Verified on OOo310m11, Windows, Vista.

- opened the attached document
- selected the frame
- opened the context menu for the frame
- cancelled the context menu with ESC

Expected result: context menu is closed
Actual result: context menu is closed, no messages