Merge ~osomon/oxide:fix-zoomfactor-unit-tests into oxide:master

Proposed by Olivier Tilloy
Status: Merged
Approved by: Chris Coulson
Approved revision: 37c375107120452de8a912b41bf7d9dd284872f7
Merged at revision: bfea1eefd0edb9e0b58550831f4b9e117a255c3d
Proposed branch: ~osomon/oxide:fix-zoomfactor-unit-tests
Merge into: oxide:master
Diff against target: 25 lines (+8/-2)
1 file modified
qt/tests/qmltests/api/tst_WebView_zoomFactor.qml (+8/-2)
Reviewer Review Type Date Requested Status
Oxide Developers Pending
Review via email: mp+322627@code.launchpad.net

Commit message

Make zoom factor unit tests more robust.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/qt/tests/qmltests/api/tst_WebView_zoomFactor.qml b/qt/tests/qmltests/api/tst_WebView_zoomFactor.qml
index d4479df..3126331 100644
--- a/qt/tests/qmltests/api/tst_WebView_zoomFactor.qml
+++ b/qt/tests/qmltests/api/tst_WebView_zoomFactor.qml
@@ -32,12 +32,18 @@ TestWebView {
3232
33 function verify_zoomFactor(factor) {33 function verify_zoomFactor(factor) {
34 compare(webView.zoomFactor, factor);34 compare(webView.zoomFactor, factor);
35 function testpixel(i, color) {
36 var image = grabImage(webView);
37 return (image.pixel(i, i) == color);
38 }
39 var zoomedSize = Math.round(originalSize * factor);
40 verify(TestUtils.waitFor(function() { return testpixel(zoomedSize - 1, red); }));
41 verify(TestUtils.waitFor(function() { return testpixel(zoomedSize, white); }));
35 var image = grabImage(webView);42 var image = grabImage(webView);
36 var zoomedSize = originalSize * factor;
37 for (var i = 0; i < zoomedSize; ++i) {43 for (var i = 0; i < zoomedSize; ++i) {
38 compare(image.pixel(i, i), red);44 compare(image.pixel(i, i), red);
39 }45 }
40 compare(image.pixel(zoomedSize + 1, zoomedSize + 1), white);46 compare(image.pixel(zoomedSize, zoomedSize), white);
41 }47 }
4248
43 function test_WebView_zoomFactor() {49 function test_WebView_zoomFactor() {

Subscribers

People subscribed via source and target branches