Issue 91928 - Office hangs after pastespecial strangely marked text.
Summary: Office hangs after pastespecial strangely marked text.
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: code (show other issues)
Version: DEV300m24
Hardware: All All
: P2 Trivial (vote)
Target Milestone: ---
Assignee: Unknown
QA Contact: issues@sw
URL:
Keywords:
Depends on:
Blocks: 88888
  Show dependency tree
 
Reported: 2008-07-21 16:57 UTC by fredrik.haegg
Modified: 2013-08-07 14:43 UTC (History)
4 users (show)

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


Attachments
Writer-document (2.68 KB, application/vnd.sun.xml.writer)
2008-07-21 17:00 UTC, fredrik.haegg
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description fredrik.haegg 2008-07-21 16:57:50 UTC
Open attached document. 
Place the marker in the upper left corner. 
Try to select the first line of text with SHIFT + DOWN. (this seems to be done,
even if the text isn't correctly marked) 
Menu: EditCopy
Open a New Writer-Document
Menu: EditPasteSpecial
Choose to paste as Writer.

Result - the office hangs.
Comment 1 fredrik.haegg 2008-07-21 17:00:24 UTC
Created attachment 55257 [details]
Writer-document
Comment 2 fredrik.haegg 2008-07-22 10:26:03 UTC
cc + blocks 88888
Comment 3 Oliver Specht 2008-07-24 09:36:05 UTC
Hi,
->mod: the name SwPostItAnkor sounds a bit strange to me ;-)

->pl: 
It loops in:
>	vclmi.dll!OutputDevice::DrawPolyLine(const Polygon & rPoly={...}, const
LineInfo & rLineInfo={...})  Line 2523	C++
 	swmi.dll!SwPostItAnkor::implDrawGeometry(OutputDevice & rOutputDevice={...},
Color aColor={...}, double fOffX=0.00000000000000000, double
fOffY=0.00000000000000000)  Line 1640 + 0x30 bytes	C++
 	swmi.dll!SwPostItAnkor::drawGeometry(OutputDevice & rOutputDevice={...}) 
Line 1737	C++
 	svxmi.dll!0842110c() 	
...
const Polygon* ImplLineConverter::ImplGetNext() always finds a new Polygon...
Comment 4 philipp.lohmann 2008-07-24 16:51:09 UTC
Actually it does not loop but you're trying to draw the dashed outline of a
Polygon spanning the whole width of the coordinate space:

DrawPolygon( -1073732580 2147483573, -1073732655 -2147483573, -1073732505
-2147483573, -1073732580 2147483573  )

drawing small line pieces 3 pixels apiece around that large a polygon can take a
while ...
Comment 5 max.odendahl 2008-07-24 17:49:35 UTC
The wrong values for drawing the anchor results from wrong return values from
SwPostItHelper::getLayoutInfos, which should return the position inside the
document of the note, but the note is not really the root cause.

I copied text without the note, and then I get a debug assertion "RootSize" and
the page is completly gone (I guess this is the reason why I get wrong info for
getLayoutInfos in the first place) This seems to happen only when copying text
from the first line. Once this is fixed, I'm sure the anchor and drawing of
polygon will be fine as well. 

(I guess we could put in some more safety checkin code for drawing the anchors
for 3.1)
Comment 6 Oliver Specht 2008-07-24 19:37:18 UTC
->od: Please have a look. 
Comment 7 Oliver-Rainer Wittmann 2008-07-25 07:44:40 UTC
First investigation reveals the following:
- Broken in DEV300m18
- Simple test case:
-- new HTML document
-- insert two paragraphs
-- copy first paragraph
-- new text document
-- paste
--> assertion "Rootsize"; tiny page; application no longer usable - no reaction
on menu, etc.
Comment 8 Oliver-Rainer Wittmann 2008-07-25 10:25:08 UTC
fixed in cws sw30bf10 - changed file:
/sw/source/core/unocore/unostyle.cxx, 1.82.90.1

defect cause:
Since fix for issue 80780 the page style set at a paragraph is also copied and
pasted. Unfortunately, the page size attributes of page style "HTML" are
LONG_MAX width and LONG_MAX height, which troubles the Writer.

solution:
Use sensible default values instead of LONG_MAX, when setting the attributes of
a page style to default values.
Comment 9 Oliver-Rainer Wittmann 2008-07-25 14:02:29 UTC
OD->SBA: Checked in internal installation set of cws sw30bf10 - please verify.
Comment 10 stefan.baltzer 2008-07-29 11:06:46 UTC
SBA: Verified in CWS sw30bf10.
Comment 11 stefan.baltzer 2008-07-29 11:18:24 UTC
SBA: Reassigned to FHA.
SBA->FHA: Please re-verify after integration in master and verify the respective
AutoTest running OK now. 
Comment 12 stefan.baltzer 2008-08-01 12:24:57 UTC
Reassigned to FHA.
Comment 13 redflagzhulihua 2008-08-06 07:41:26 UTC
Verified in DEV300m28 with windows XP and Fedora 8
Comment 14 sgautier.ooo 2008-09-09 17:02:02 UTC
Verified in OOO300m5 - Closing - Sophie