Commit 5877982c authored by Nicolas Fella's avatar Nicolas Fella
Browse files

[konqpixmapprovider] Don't inherit from KPixmapProvider

Summary: It only defines the pixmapFor method, which we implement ourselves anyway. It is the last non-obsolete usage of KPixmapProvider, so removing it here will allow us to get rid of it entirely eventually.

Test Plan: builds. Basic browsing works, no obvious icons missing

Reviewers: #konqueror, dfaure

Reviewed By: #konqueror, dfaure

Maniphest Tasks: T12310

Differential Revision: https://phabricator.kde.org/D25818
parent 919076fa
......@@ -43,7 +43,7 @@ KonqPixmapProvider *KonqPixmapProvider::self()
}
KonqPixmapProvider::KonqPixmapProvider()
: KPixmapProvider()
: QObject()
{
}
......
......@@ -22,8 +22,6 @@
#include "konqprivate_export.h"
#include <kpixmapprovider.h>
#include <QMap>
#include <QPixmap>
#include <QUrl>
......@@ -31,13 +29,13 @@
class KConfigGroup;
class KConfig;
class KONQUERORPRIVATE_EXPORT KonqPixmapProvider : public QObject, public KPixmapProvider
class KONQUERORPRIVATE_EXPORT KonqPixmapProvider : public QObject
{
Q_OBJECT
public:
static KonqPixmapProvider *self();
~KonqPixmapProvider() override;
~KonqPixmapProvider();
/**
* Trigger a download of a default favicon
......@@ -51,7 +49,7 @@ public:
/**
* Looks up a pixmap for @p url. Uses a cache for the iconname of url.
*/
QPixmap pixmapFor(const QString &url, int size) override;
QPixmap pixmapFor(const QString &url, int size);
/**
* Loads the cache to @p kc from key @p key.
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment