sc/CppunitTest_sc_anchor_test.mk | 4 -
sc/qa/extras/anchor.cxx | 17 +++++
sc/qa/unit/data/ods/tdf129552.fods | 123 +++++++++++++++++++++++++++++++++++++
sc/source/ui/view/gridwin4.cxx | 7 +-
4 files changed, 147 insertions(+), 4 deletions(-)
New commits:
commit 8728eddf938c9c843ab72929cfd3947735ca8da2
Author: Miklos Vajna <vmik...@collabora.com>
AuthorDate: Mon Jan 6 10:31:50 2020 +0100
Commit: Caolán McNamara <caol...@redhat.com>
CommitDate: Tue Jan 7 16:50:58 2020 +0100
tdf#129552 sc: avoid infinite invalidation loop when the print range is
empty
Commit caeb7b141280a65e60525f11a7e6514b76e12e11 (tdf#124983 In calc make
printable page borders also initially visible, 2019-07-10) added the
ability to paint page breaks right after opening a document. The
implementation calls ScPrintFunc::UpdatePages() whenever there are no
calculated page breaks. The problem is that this is not only true when
they are not calculated, but also happens when the print range is empty.
This means that ScGridWindow::Paint() resulted in a
vcl::Window::Invalidate() for the same window, and this happened again
and again as the idle handler was invoked, resulting in flickering form
controls.
Fix the problem by only calculating page breaks when
ScPrintFunc::HasPrintRange() confirms something will be calculated. This
works because HasPrintRange() return false when ScTable::aPrintRanges is
empty, and UpdatePages() also does nothing in that case.
(cherry picked from commit e7e01efc56f7061d0a2e5142b4bae84dd403cc50)
Change-Id: I14d8a728f4672e647e9009a3dc3e0dd5fb40c023
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86274
Tested-by: Jenkins
Tested-by: Xisco Faulí <xiscofa...@libreoffice.org>
Reviewed-by: Caolán McNamara <caol...@redhat.com>
diff --git a/sc/CppunitTest_sc_anchor_test.mk b/sc/CppunitTest_sc_anchor_test.mk
index d8ae6e69fb8d..da3807004573 100644
--- a/sc/CppunitTest_sc_anchor_test.mk
+++ b/sc/CppunitTest_sc_anchor_test.mk
@@ -70,9 +70,7 @@ $(eval $(call gb_CppunitTest_use_api,sc_anchor_test,\
$(eval $(call gb_CppunitTest_use_ure,sc_anchor_test))
$(eval $(call gb_CppunitTest_use_vcl,sc_anchor_test))
-$(eval $(call gb_CppunitTest_use_components,sc_anchor_test,\
- $(sc_unoapi_common_components) \
-))
+$(eval $(call gb_CppunitTest_use_rdb,sc_anchor_test,services))
$(eval $(call gb_CppunitTest_use_configuration,sc_anchor_test))
diff --git a/sc/qa/extras/anchor.cxx b/sc/qa/extras/anchor.cxx
index c003c09d8084..83553c802291 100644
--- a/sc/qa/extras/anchor.cxx
+++ b/sc/qa/extras/anchor.cxx
@@ -20,6 +20,7 @@
#include <drwlayer.hxx>
#include <scitems.hxx>
#include <svx/svdocirc.hxx>
+#include <vcl/scheduler.hxx>
#include <tabvwsh.hxx>
#include <sc.hrc>
@@ -40,6 +41,7 @@ public:
void testODFAnchorTypes();
void testCopyColumnWithImages();
void testCutWithImages();
+ void testTdf129552();
CPPUNIT_TEST_SUITE(ScAnchorTest);
CPPUNIT_TEST(testUndoAnchor);
@@ -47,6 +49,7 @@ public:
CPPUNIT_TEST(testODFAnchorTypes);
CPPUNIT_TEST(testCopyColumnWithImages);
CPPUNIT_TEST(testCutWithImages);
+ CPPUNIT_TEST(testTdf129552);
CPPUNIT_TEST_SUITE_END();
private:
@@ -366,6 +369,20 @@ void ScAnchorTest::testCutWithImages()
pDocSh->DoClose();
}
+void ScAnchorTest::testTdf129552()
+{
+ OUString aFileURL;
+ createFileURL("tdf129552.fods", aFileURL);
+ uno::Reference<css::lang::XComponent> xComponent =
loadFromDesktop(aFileURL);
+ CPPUNIT_ASSERT(xComponent.is());
+
+ // Without the accompanying fix in place, this test would have never
returned due to an infinite
+ // invalidation loop, where ScGridWindow::Paint() invalidated itself.
+ Scheduler::ProcessEventsToIdle();
+
+ xComponent->dispose();
+}
+
void ScAnchorTest::tearDown()
{
if (mxComponent.is())
diff --git a/sc/qa/unit/data/ods/tdf129552.fods
b/sc/qa/unit/data/ods/tdf129552.fods
new file mode 100644
index 000000000000..b5bb83b76cd6
--- /dev/null
+++ b/sc/qa/unit/data/ods/tdf129552.fods
@@ -0,0 +1,123 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<office:document
xmlns:presentation="urn:oasis:names:tc:opendocument:xmlns:presentation:1.0"
xmlns:grddl="http://www.w3.org/2003/g/data-view#";
xmlns:xhtml="http://www.w3.org/1999/xhtml";
xmlns:css3t="http://www.w3.org/TR/css3-text/";
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
xmlns:xsd="http://www.w3.org/2001/XMLSchema";
xmlns:formx="urn:openoffice:names:experimental:ooxml-odf-interop:xmlns:form:1.0"
xmlns:xforms="http://www.w3.org/2002/xforms";
xmlns:dom="http://www.w3.org/2001/xml-events";
xmlns:script="urn:oasis:names:tc:opendocument:xmlns:script:1.0"
xmlns:form="urn:oasis:names:tc:opendocument:xmlns:form:1.0"
xmlns:math="http://www.w3.org/1998/Math/MathML";
xmlns:field="urn:openoffice:names:experimental:ooo-ms-interop:xmlns:field:1.0"
xmlns:of="urn:oasis:names:tc:opendocument:xmlns:of:1.2"
xmlns:oooc="http://openoffice.org/2004/calc";
xmlns:ooow="http://openoffice.org/2004/writer";
xmlns:config="urn:oasis:names:tc:opendocument:xmlns:config:1.0"
xmlns:tableooo="http://openoffi
ce.org/2009/table" xmlns:text="urn:oasis:names:tc:opendocument:xmlns:text:1.0"
xmlns:calcext="urn:org:documentfoundation:names:experimental:calc:xmlns:calcext:1.0"
xmlns:fo="urn:oasis:names:tc:opendocument:xmlns:xsl-fo-compatible:1.0"
xmlns:dc="http://purl.org/dc/elements/1.1/";
xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0"
xmlns:loext="urn:org:documentfoundation:names:experimental:office:xmlns:loext:1.0"
xmlns:xlink="http://www.w3.org/1999/xlink";
xmlns:ooo="http://openoffice.org/2004/office";
xmlns:table="urn:oasis:names:tc:opendocument:xmlns:table:1.0"
xmlns:meta="urn:oasis:names:tc:opendocument:xmlns:meta:1.0"
xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0"
xmlns:drawooo="http://openoffice.org/2010/draw";
xmlns:draw="urn:oasis:names:tc:opendocument:xmlns:drawing:1.0"
xmlns:dr3d="urn:oasis:names:tc:opendocument:xmlns:dr3d:1.0"
xmlns:svg="urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0"
xmlns:chart="urn:oasis:names:tc:opendocument:xmlns:chart:1
.0" xmlns:rpt="http://openoffice.org/2005/report";
xmlns:number="urn:oasis:names:tc:opendocument:xmlns:datastyle:1.0"
office:version="1.2"
office:mimetype="application/vnd.oasis.opendocument.spreadsheet">
+ <office:font-face-decls>
+ <style:font-face style:name="Liberation Sans"
svg:font-family="'Liberation Sans'" style:font-family-generic="swiss"
style:font-pitch="variable"/>
+ </office:font-face-decls>
+ <office:styles>
+ <style:default-style style:family="table-cell">
+ <style:table-cell-properties style:decimal-places="2"/>
+ <style:paragraph-properties style:tab-stop-distance="1.389in"/>
+ <style:text-properties style:font-name="Albany" fo:language="de"
fo:country="DE" style:font-name-asian="Andale Sans UI"
style:language-asian="zxx" style:country-asian="none"
style:font-name-complex="Tahoma" style:language-complex="zxx"
style:country-complex="none"/>
+ </style:default-style>
+ <style:style style:name="Default" style:family="table-cell">
+ <style:table-cell-properties style:cell-protect="protected
formula-hidden" style:print-content="true"/>
+ <style:text-properties style:font-name="Arial" fo:font-family="Arial"
style:font-family-generic="swiss" style:font-pitch="variable"/>
+ </style:style>
+ </office:styles>
+ <office:automatic-styles>
+ <style:style style:name="co1" style:family="table-column">
+ <style:table-column-properties fo:break-before="auto"
style:column-width="0.1965in"/>
+ </style:style>
+ <style:style style:name="ro1" style:family="table-row">
+ <style:table-row-properties style:row-height="0.1965in"
fo:break-before="auto" style:use-optimal-row-height="false"/>
+ </style:style>
+ <style:style style:name="ta1" style:family="table"
style:master-page-name="Default">
+ <style:table-properties table:display="true" style:writing-mode="lr-tb"/>
+ </style:style>
+ <style:page-layout style:name="pm1">
+ <style:page-layout-properties fo:margin-top="0.5902in"
fo:margin-bottom="0.7083in" fo:margin-left="0.8661in"
fo:margin-right="0.3937in" style:scale-to-pages="1"/>
+ <style:header-style>
+ <style:header-footer-properties fo:min-height="0.1965in"
fo:margin-left="0in" fo:margin-right="0in" fo:margin-bottom="0in"/>
+ </style:header-style>
+ <style:footer-style>
+ <style:header-footer-properties fo:min-height="0.2953in"
fo:margin-left="0in" fo:margin-right="0in" fo:margin-top="0.0984in"/>
+ </style:footer-style>
+ </style:page-layout>
+ </office:automatic-styles>
+ <office:master-styles>
+ <style:master-page style:name="Default" style:page-layout-name="pm1">
+ </style:master-page>
+ </office:master-styles>
+ <office:body>
+ <office:spreadsheet>
+ <table:calculation-settings table:case-sensitive="false"
table:search-criteria-must-apply-to-whole-cell="false" table:null-year="1919">
+ <table:iteration table:steps="10" table:maximum-difference="1"/>
+ </table:calculation-settings>
+ <table:table table:name="test" table:style-name="ta1"
table:print="false">
+ <office:forms form:automatic-focus="false"
form:apply-design-mode="false">
+ <form:form form:name="Standard" form:apply-filter="true"
form:command-type="table"
form:control-implementation="ooo:com.sun.star.form.component.Form"
office:target-frame="">
+ <form:radio form:name="form-name"
form:control-implementation="ooo:com.sun.star.form.component.RadioButton"
xml:id="control1" form:id="control1" form:current-selected="true"
form:value="0" form:input-required="false" form:image-position="center">
+ <form:properties>
+ <form:property form:property-name="ControlTypeinMSO"
office:value-type="float" office:value="0"/>
+ <form:property form:property-name="DefaultControl"
office:value-type="string"
office:string-value="com.sun.star.form.control.RadioButton"/>
+ <form:property form:property-name="ObjIDinMSO"
office:value-type="float" office:value="0"/>
+ <form:property form:property-name="SecondaryRefValue"
office:value-type="string" office:string-value=""/>
+ </form:properties>
+ </form:radio>
+ </form:form>
+ </office:forms>
+ <table:table-column table:style-name="co1"
table:default-cell-style-name="Default"/>
+ <table:table-column table:style-name="co2"
table:default-cell-style-name="Default"/>
+ <table:table-column table:style-name="co1"
table:default-cell-style-name="Default"/>
+ <table:table-column table:style-name="co3"
table:default-cell-style-name="Default"/>
+ <table:table-column table:style-name="co1"
table:default-cell-style-name="Default"/>
+ <table:table-column table:style-name="co4"
table:default-cell-style-name="Default"/>
+ <table:table-column table:style-name="co5"
table:number-columns-repeated="58" table:default-cell-style-name="Default"/>
+ <table:table-row table:style-name="ro3">
+ <table:table-cell table:style-name="ce1"/>
+ <table:table-cell table:number-columns-repeated="2"/>
+ <table:table-cell>
+ <draw:control table:end-cell-address="test.D8"
table:end-x="3.8406in" table:end-y="0.2772in" draw:z-index="1"
draw:style-name="gr2" draw:text-style-name="P2" svg:width="1.5748in"
svg:height="0.2559in" svg:x="2.2657in" svg:y="0.0213in"
draw:control="control1"/>
+ </table:table-cell>
+ <table:table-cell table:style-name="ce111"/>
+ <table:table-cell table:style-name="ce1"/>
+ <table:table-cell table:style-name="ce135"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="57"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro2" table:number-rows-repeated="5">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="6"/>
+ <table:table-cell table:style-name="ce135"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="57"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro6">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="6"/>
+ <table:table-cell table:style-name="ce135"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="57"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro2"
table:number-rows-repeated="137">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="6"/>
+ <table:table-cell table:style-name="ce135"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="57"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro7" table:visibility="collapse">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="6"/>
+ <table:table-cell table:style-name="ce135"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="57"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro8" table:visibility="collapse">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="64"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro8" table:visibility="collapse">
+ <table:table-cell table:style-name="ce1"/>
+ <table:table-cell table:style-name="ce45"/>
+ <table:table-cell table:style-name="ce63"/>
+ <table:table-cell table:style-name="ce94"
table:number-columns-repeated="2"/>
+ <table:table-cell table:style-name="ce124"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="58"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro8" table:visibility="collapse">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="64"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro2" table:visibility="collapse"
table:number-rows-repeated="31538">
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="6"/>
+ <table:table-cell table:style-name="ce135"/>
+ <table:table-cell table:style-name="ce1"
table:number-columns-repeated="57"/>
+ </table:table-row>
+ <table:table-row table:style-name="ro2" table:visibility="collapse"
table:number-rows-repeated="1016529">
+ <table:table-cell table:number-columns-repeated="64"/>
+ </table:table-row>
+ </table:table>
+ <table:named-expressions/>
+ </office:spreadsheet>
+ </office:body>
+</office:document>
diff --git a/sc/source/ui/view/gridwin4.cxx b/sc/source/ui/view/gridwin4.cxx
index 823fa87d7d4e..72926fd372a1 100644
--- a/sc/source/ui/view/gridwin4.cxx
+++ b/sc/source/ui/view/gridwin4.cxx
@@ -575,7 +575,12 @@ void ScGridWindow::DrawContent(OutputDevice &rDevice,
const ScTableInfo& rTableI
{
ScDocShell* pDocSh = pViewData->GetDocShell();
ScPrintFunc aPrintFunc(pDocSh, pDocSh->GetPrinter(), nTab);
- aPrintFunc.UpdatePages();
+ if (aPrintFunc.HasPrintRange())
+ {
+ // We have a non-empty print range, so we can assume that
calling UpdatePages() will
+ // result in non-empty col/row breaks next time we get here.
+ aPrintFunc.UpdatePages();
+ }
}
}
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
