diff options
author | Michael Stahl <mstahl@redhat.com> | 2012-05-30 08:53:00 +0200 |
---|---|---|
committer | Michael Stahl <mstahl@redhat.com> | 2012-05-30 09:02:31 +0200 |
commit | 8bafe38c569afa2e1055eb647cb7ff161ddd1230 (patch) | |
tree | 69e1c04f9a5c37d16abbc867c9651899098173c8 /vcl/inc/generic | |
parent | 33050f597cef81162eede4360b011b41e10ab69f (diff) |
i#87757: do not create empty Rectangle bounding box for glyphs:
Enforce a minimum width and height of 1 in GlyphMetric::SetSize(),
preventing X11SalGraphics::GetGlyphBoundRect() from returning a
Rectangle that has one side set as RECT_EMPTY.
This problem has surfaced due to the change of the status bar in
ce14342c4292628a641a72d4f63d9c048e030c6a, causing the
AccessibleStatusBarItem to fail in getCharacterBounds() because
the text of the control, "Words: 0", now contains a space, for which an
empty Rectangle bounding box is created, which results in a wrong
return value from getCharacterBounds():
LOG> Character bounds outside component
LOG> Character rect: 43, -566, 0, 0
LOG> Component rect: 91, 2, 71, 18
It turns out that the following test was disabled due to this problem,
so re-enable it now:
toolkit.AccessibleFixedText::com::sun::star::accessibility::XAccessibleText
Diffstat (limited to 'vcl/inc/generic')
-rw-r--r-- | vcl/inc/generic/glyphcache.hxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vcl/inc/generic/glyphcache.hxx b/vcl/inc/generic/glyphcache.hxx index 1ab2a74eda45..8c0f3b17cacd 100644 --- a/vcl/inc/generic/glyphcache.hxx +++ b/vcl/inc/generic/glyphcache.hxx @@ -132,7 +132,7 @@ protected: friend class GlyphData; void SetOffset( int nX, int nY ) { maOffset = Point( nX, nY); } void SetDelta( int nX, int nY ) { maDelta = Point( nX, nY); } - void SetSize( const Size& s ) { maSize = s; } + void SetSize(const Size& s); void SetCharWidth( long nW ) { mnAdvanceWidth = nW; } private: |