forked from qt-creator/qt-creator
Valgrind plugin: fix for just partial usage of coreplugin/coreconstants.h values
Valgrind plugin CallgrindTool and MemcheckTool modules sometime use coreplugin/coreconstants.h values for icons (e.g. Core::Constants::ICON_REDO), and sometime - an own constant is used like ":core/images/prev.png". This commit is fixing this issue and changing everything to Core::Constants usage. Change-Id: I8d6f473a66a3181b03e2193314d17661df1d4cc1 Reviewed-by: hjk <qthjk@ovi.com>
This commit is contained in:
@@ -743,7 +743,7 @@ QWidget *CallgrindToolPrivate::createWidgets()
|
||||
// go back
|
||||
action = new QAction(this);
|
||||
action->setDisabled(true);
|
||||
action->setIcon(QIcon(QLatin1String(":core/images/prev.png")));
|
||||
action->setIcon(QIcon(QLatin1String(Core::Constants::ICON_PREV)));
|
||||
action->setToolTip(tr("Go back one step in history. This will select the previously selected item."));
|
||||
connect(action, SIGNAL(triggered(bool)), m_stackBrowser, SLOT(goBack()));
|
||||
layout->addWidget(createToolButton(action));
|
||||
@@ -752,7 +752,7 @@ QWidget *CallgrindToolPrivate::createWidgets()
|
||||
// go forward
|
||||
action = new QAction(this);
|
||||
action->setDisabled(true);
|
||||
action->setIcon(QIcon(QLatin1String(":core/images/next.png")));
|
||||
action->setIcon(QIcon(QLatin1String(Core::Constants::ICON_NEXT)));
|
||||
action->setToolTip(tr("Go forward one step in history."));
|
||||
connect(action, SIGNAL(triggered(bool)), m_stackBrowser, SLOT(goNext()));
|
||||
layout->addWidget(createToolButton(action));
|
||||
|
||||
Reference in New Issue
Block a user