Issue 79578 - ole replacement image is requested much too often (getPreferredVisualRepresentation)
Summary: ole replacement image is requested much too often (getPreferredVisualRepresen...
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: ui (show other issues)
Version: 680m219
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 2.4
Assignee: bjoern.milcke
QA Contact: issues@framework
URL:
Keywords: performance
Depends on:
Blocks:
 
Reported: 2007-07-13 11:43 UTC by IngridvdM
Modified: 2008-01-16 13:43 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description IngridvdM 2007-07-13 11:43:44 UTC
In diverse scenarios the method getPreferredVisualRepresentation is called too
often. As this call can be very expensive for charts with big data or 3D charts,
the superfluous calls must be avoided.

1) Activate an existing chart leads to calls of getPreferredVisualRepresentation:
-in writer 2 times what is superfluous
-in calc and impress zero times as it should be
2) Deactivating an unmodified chart leads to calls of
getPreferredVisualRepresentation:
-one time for writer, calc and impress but there is no need to change the
representation, or?
3) Deactivating a modified chart leads to calls of getPreferredVisualRepresentation:
- 2 times in writer, calc and impress - there should be only one call instead
4) Editing a chart in impress leads to an immediate call of
getPreferredVisualRepresentation - seems to be superfluous also
5) when creating a default chart getPreferredVisualRepresentation is called very
often in writer over 10 times. In calc and impress its called only once which
might be ok, not sure - maybe this calls can also be avoided.
Comment 1 mikhail.voytenko 2007-07-13 12:42:56 UTC
Should be investigated for 2.4
Comment 2 Mathias_Bauer 2007-10-05 14:59:40 UTC
target 3.0
Comment 3 IngridvdM 2007-12-13 16:02:17 UTC
I fixed this in CWS chart15 for OOo 2.4.
Comment 4 IngridvdM 2007-12-13 16:03:51 UTC
fixed
Comment 5 IngridvdM 2007-12-13 16:05:29 UTC
->Björn, please verify in CWS chart15.
Comment 6 bjoern.milcke 2007-12-14 14:02:47 UTC
Looks like no superfluous replacement images would be requested now. There is a
replacement request right after creation of a chart, but I suppose that's ok.

Verified on CWS chart15.
Comment 7 IngridvdM 2008-01-04 10:54:54 UTC
On replacement request was needed to be reintroduced see follow up issue 84750.
Comment 8 bjoern.milcke 2008-01-16 13:42:31 UTC
Verified in OOH680.m3. As noted all occurrences except 2) are fixed.
Comment 9 bjoern.milcke 2008-01-16 13:43:09 UTC
... closing