Skip to content

Commit

Permalink
Fix printing/zoomed-document.html for QT
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=79808

Reviewed by Adam Barth.

Fix zoomed document test case to work on QT port and remove it from skip
list.

* platform/qt/Skipped:
* printing/zoomed-document.html:


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@109126 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  • Loading branch information
eae@chromium.org committed Feb 28, 2012
1 parent 886b7c5 commit d1e7d8e
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 8 deletions.
13 changes: 13 additions & 0 deletions LayoutTests/ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,16 @@
2012-02-28 Emil A Eklund <eae@chromium.org>

Fix printing/zoomed-document.html for QT
https://bugs.webkit.org/show_bug.cgi?id=79808

Reviewed by Adam Barth.

Fix zoomed document test case to work on QT port and remove it from skip
list.

* platform/qt/Skipped:
* printing/zoomed-document.html:

2012-02-28 Alok Priyadarshi <alokp@chromium.org>

Heap-use-after-free in WebCore::RenderLayer::addChild
Expand Down
5 changes: 0 additions & 5 deletions LayoutTests/platform/qt/Skipped
Original file line number Diff line number Diff line change
Expand Up @@ -2656,8 +2656,3 @@ fast/ruby/text-decoration-in-descendants-ruby.html
# [Qt] transitions/cancel-transition.html fails
# https://bugs.webkit.org/show_bug.cgi?id=79564
transitions/cancel-transition.html

# Printed font-size should not be dependant on zoom level
# New test introduced in r109073 fails on Qt
# https://bugs.webkit.org/show_bug.cgi?id=79717
printing/zoomed-document.html
16 changes: 13 additions & 3 deletions LayoutTests/printing/zoomed-document.html
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,22 @@

var msg = 'Document zoom should not apply when printing.';
var pass = true;


// Determine minium page size height where text fits on a single page.
var height;
for (var i = 10; i < 100; i += 5) {
if (layoutTestController.numberOfPages(100, i) == 1) {
height = i;
break;
}
}

// Zoom in and verify that page count does not change.
for (var i = 0; i < 10; i++) {
var numberOfPages = layoutTestController.numberOfPages(100, 30);
var numberOfPages = layoutTestController.numberOfPages(100, height);
if (numberOfPages != 1) {
pass = false;
msg += 'Got ' + numberOfPages + ' pages, expected 1.\n';
msg += 'Got ' + numberOfPages + ' pages, expected 1 page.\n';
}
eventSender.zoomPageIn();
}
Expand Down

0 comments on commit d1e7d8e

Please sign in to comment.