diff options
| author | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2021-03-25 10:20:44 +0100 |
|---|---|---|
| committer | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2021-03-29 15:59:40 +0200 |
| commit | c8f6f8a2220ce8fd628c969f19dfae6f38d7311c (patch) | |
| tree | f15d6d21bd491a40a888679b6d008eba6113e2f8 /tests/auto/widgets/widgets/qlabel | |
| parent | db1c238a66e3b7ae16768a4e01ce46a51a51eba3 (diff) | |
Rename QUrlResourceProvider to QTextDocumentResourceProvider
While the class name is now a bit more than a mouthful, it's purpose
is very narrowly tied to QTextDocument, so don't use a very generic
name for it. That resources are provided based on a URL is to some
degree an implementation detail, and URLs are resource locators so
we don't need that in the class name.
Address code review comment for 6.1. Add documentation and links to
existing APIs with a similar purpose.
Task-number: QTBUG-90211
Task-number: QTBUG-92208
Pick-to: 6.1
Change-Id: I4f09057cc2f53a5595513c1c9422e6ccaad6ca13
Reviewed-by: Kai Koehne <kai.koehne@qt.io>
Diffstat (limited to 'tests/auto/widgets/widgets/qlabel')
| -rw-r--r-- | tests/auto/widgets/widgets/qlabel/tst_qlabel.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/auto/widgets/widgets/qlabel/tst_qlabel.cpp b/tests/auto/widgets/widgets/qlabel/tst_qlabel.cpp index 5eed823a34..e4e9d12fb4 100644 --- a/tests/auto/widgets/widgets/qlabel/tst_qlabel.cpp +++ b/tests/auto/widgets/widgets/qlabel/tst_qlabel.cpp @@ -40,7 +40,7 @@ #include <qmessagebox.h> #include <qfontmetrics.h> #include <qmath.h> -#include <qurlresourceprovider.h> +#include <qtextdocumentresourceprovider.h> #include <private/qlabel_p.h> class Widget : public QWidget @@ -599,7 +599,7 @@ void tst_QLabel::taskQTBUG_48157_dprMovie() QCOMPARE(label.sizeHint(), movie.currentPixmap().size() / movie.currentPixmap().devicePixelRatio()); } -class UrlResourceProvider : public QUrlResourceProvider +class UrlResourceProvider : public QTextDocumentResourceProvider { public: QVariant resource(const QUrl &url) override |