feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

QmlProfiler: Only show pixmap cache size on cache change events

Browse Source 
... and make the default size 0, rather than -1. Showing cache sizes
for individual pixmaps doesn't make much sense as other pixmaps might
get loaded at the same time.

Task-number: QTCREATORBUG-17424
Change-Id: Iead21c21d87b454ef03961d22119e0f5351d11ea
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Reviewed-by: Milian Wolff <milian.wolff@kdab.com>
This commit is contained in:
Ulf Hermann
2016-12-13 15:58:57 +01:00
parent 577ab21b30
commit 55d6997154
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/plugins/qmlprofiler/pixmapcachemodel.cpp
View File

@@ -117,6 +117,7 @@ QVariantMap PixmapCacheModel::details(int index) const
if (ev->pixmapEventType == PixmapCacheCountChanged) {
result.insert(QLatin1String("displayName"), tr("Image Cached"));
result.insert(tr("Cache Size"), QString::fromLatin1("%1 px").arg(ev->cacheSize));
} else {
result.insert(QLatin1String("displayName"), tr("Image Loaded"));
if (m_pixmaps[ev->urlIndex].sizes[ev->sizeIndex].loadState != Finished)
@@ -124,7 +125,6 @@ QVariantMap PixmapCacheModel::details(int index) const
result.insert(tr("Duration"), Timeline::formatTime(duration(index)));
}
result.insert(tr("Cache Size"), QString::fromLatin1("%1 px").arg(ev->cacheSize));
result.insert(tr("File"), getFilenameOnly(m_pixmaps[ev->urlIndex].url));
result.insert(tr("Width"), QString::fromLatin1("%1 px")
.arg(m_pixmaps[ev->urlIndex].sizes[ev->sizeIndex].size.width()));

2
src/plugins/qmlprofiler/pixmapcachemodel.h
View File

@@ -92,7 +92,7 @@ public:
int urlIndex = -1;
int sizeIndex = -1;
int rowNumberCollapsed = -1;
qint64 cacheSize = -1;
qint64 cacheSize = 0;
};
PixmapCacheModel(QmlProfilerModelManager *manager, QObject *parent = 0);