From 13998901f7831beaaeca6ada4def3e29af4e251d Mon Sep 17 00:00:00 2001 From: Thierry Bastian Date: Thu, 6 Aug 2009 13:43:42 +0200 Subject: QMenuBar: we can't test the action geometries when it is native --- src/gui/widgets/qmenubar.cpp | 3 +++ tests/auto/qmenubar/tst_qmenubar.cpp | 3 +++ 2 files changed, 6 insertions(+) diff --git a/src/gui/widgets/qmenubar.cpp b/src/gui/widgets/qmenubar.cpp index 4a900d6..389b65f 100644 --- a/src/gui/widgets/qmenubar.cpp +++ b/src/gui/widgets/qmenubar.cpp @@ -272,6 +272,9 @@ QRect QMenuBarPrivate::actionRect(QAction *act) const //makes sure the geometries are up-to-date const_cast(this)->updateGeometries(); + if (index >= actionRects.count()) + return QRect(); // that can happen in case of native menubar + QRect ret = actionRects.at(index); return QStyle::visualRect(q->layoutDirection(), q->rect(), ret); } diff --git a/tests/auto/qmenubar/tst_qmenubar.cpp b/tests/auto/qmenubar/tst_qmenubar.cpp index 67e088b..f8f536c 100644 --- a/tests/auto/qmenubar/tst_qmenubar.cpp +++ b/tests/auto/qmenubar/tst_qmenubar.cpp @@ -1586,6 +1586,9 @@ void tst_QMenuBar::menubarSizeHint() } style; QMenuBar mb; + if (mb.isNativeMenuBar()) + QSKIP("we can't check the geometry of native menubars", SkipAll); + mb.setStyle(&style); //this is a list of arbitrary strings so that we check the geometry QStringList list = QStringList() << "trer" << "ezrfgtgvqd" << "sdgzgzerzerzer" << "eerzertz" << "er"; -- cgit v0.12