From edd377245adfecae307b587d937b9873187f5ce7 Mon Sep 17 00:00:00 2001 From: Jason McDonald Date: Thu, 6 Oct 2011 10:57:07 +1000 Subject: Fix unreachable QSKIP in QMenu autotest The QSKIP is only reachable on Mac, so only compile it for Mac. Change-Id: I268e87829e01755051f81cdaf856d936b04b3d49 Reviewed-on: http://codereview.qt-project.org/6091 Reviewed-by: Qt Sanity Bot Reviewed-by: Rohan McGovern --- tests/auto/qmenu/tst_qmenu.cpp | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'tests') diff --git a/tests/auto/qmenu/tst_qmenu.cpp b/tests/auto/qmenu/tst_qmenu.cpp index b20fb1645b..001d863d82 100644 --- a/tests/auto/qmenu/tst_qmenu.cpp +++ b/tests/auto/qmenu/tst_qmenu.cpp @@ -380,15 +380,12 @@ void tst_QMenu::focus() menu.addAction("One"); menu.addAction("Two"); menu.addAction("Three"); - bool fullKeyboardControl = true; #ifdef Q_WS_MAC - fullKeyboardControl = qt_tab_all_widgets; -#endif - - if (!fullKeyboardControl) + if (!qt_tab_all_widgets) QSKIP("Computer is currently set up to NOT tab to all widgets," " this test assumes you can tab to all widgets", SkipAll); +#endif QWidget window; QPushButton button("Push me", &window); -- cgit v1.2.3