diff options
author | Joerg Bornemann <joerg.bornemann@qt.io> | 2016-05-18 14:55:13 +0200 |
---|---|---|
committer | Liang Qi <liang.qi@qt.io> | 2016-05-19 18:47:23 +0000 |
commit | f4b806ffbc2f11a5a64fab2856dfe68ae7d603c8 (patch) | |
tree | e0549b086cdb024b5c360c1a949b5a9988873049 /tests | |
parent | f2370c0d09998f382364a9a58932cf717c28a764 (diff) |
Fix pasting images to web pages
WebKit expects pasted images to have the MIME type image/png (see code
and comment in DataObjectItem::getAsFile()).
Task-number: QTBUG-53409
Change-Id: I2b0c1244d309687ad190db26c5b00718ed0c4258
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'tests')
3 files changed, 62 insertions, 0 deletions
diff --git a/tests/auto/widgets/qwebenginepage/resources/pasteimage.html b/tests/auto/widgets/qwebenginepage/resources/pasteimage.html new file mode 100644 index 000000000..cc37d9319 --- /dev/null +++ b/tests/auto/widgets/qwebenginepage/resources/pasteimage.html @@ -0,0 +1,30 @@ +<html> + <head> + <meta charset="UTF-8"> + <script> + window.onload = function() { + document.getElementById("pasteArea").focus(); + } + document.onpaste = function(event) { + var items = (event.clipboardData || event.originalEvent.clipboardData).items; + var blob = null; + for (var i = 0; i < items.length; ++i) { + if (items[i].type.indexOf("image/png") !== -1) { + blob = items[i].getAsFile(); + break; + } + } + if (!blob) + return; + var reader = new FileReader(); + reader.onload = function(event) { + window.myImageDataURL = event.target.result; + } + reader.readAsDataURL(blob); + }; + </script> + </head> + <body> + <textarea id="pasteArea" placeholder="paste here"></textarea> + </body> +</html> diff --git a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp index 5d53c0468..f29b24933 100644 --- a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp +++ b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp @@ -31,6 +31,7 @@ #include <QPushButton> #include <QStateMachine> #include <QStyle> +#include <QtGui/QClipboard> #include <QtTest/QtTest> #include <QTextCharFormat> #include <QWebChannel> @@ -120,6 +121,7 @@ private Q_SLOTS: void geolocationRequestJS(); void loadFinished(); void actionStates(); + void pasteImage(); void popupFormSubmission(); void userStyleSheet(); void userStyleSheetFromLocalFileUrl(); @@ -459,6 +461,35 @@ void tst_QWebEnginePage::actionStates() QTRY_VERIFY(!stopAction->isEnabled()); } +static QImage imageWithoutAlpha(const QImage &image) +{ + QImage result = image; + QPainter painter(&result); + painter.fillRect(result.rect(), Qt::green); + painter.drawImage(0, 0, image); + return result; +} + +void tst_QWebEnginePage::pasteImage() +{ + // Pixels with an alpha value of 0 will have different RGB values after the + // test -> clipboard -> webengine -> test roundtrip. + // Clear the alpha channel to make QCOMPARE happy. + const QImage origImage = imageWithoutAlpha(QImage(":/resources/image.png")); + QClipboard *clipboard = QGuiApplication::clipboard(); + clipboard->setImage(origImage); + QWebEnginePage *page = m_view->page(); + page->load(QUrl("qrc:///resources/pasteimage.html")); + QVERIFY(waitForSignal(m_view, SIGNAL(loadFinished(bool)))); + page->triggerAction(QWebEnginePage::Paste); + QTRY_VERIFY(evaluateJavaScriptSync(page, + "window.myImageDataURL ? window.myImageDataURL.length : 0").toInt() > 0); + QByteArray data = evaluateJavaScriptSync(page, "window.myImageDataURL").toByteArray(); + data.remove(0, data.indexOf(";base64,") + 8); + const QImage image = QImage::fromData(QByteArray::fromBase64(data), "PNG"); + QCOMPARE(image, origImage); +} + class ConsolePage : public QWebEnginePage { public: diff --git a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.qrc b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.qrc index c7bffd5bb..91adeb694 100644 --- a/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.qrc +++ b/tests/auto/widgets/qwebenginepage/tst_qwebenginepage.qrc @@ -12,6 +12,7 @@ <file>resources/script.html</file> <file>resources/user.css</file> <file>resources/image.png</file> + <file>resources/pasteimage.html</file> <file>resources/style.css</file> <file>resources/test1.html</file> <file>resources/test2.html</file> |