Bug 56556 - [GTK] fast/viewport/viewport-131.html fails
Summary: [GTK] fast/viewport/viewport-131.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-17 03:59 PDT by Philippe Normand
Modified: 2011-03-17 09:28 PDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 2011-03-17 03:59:50 PDT 
on both GTK Debug builds:

--- /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/fast/viewport/viewport-131-expected.txt	2011-03-17 03:16:30.481410508 -0700
+++ /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/fast/viewport/viewport-131-actual.txt	2011-03-17 03:16:30.481410508 -0700
@@ -1,2 +1,2 @@
-viewport size 320x352 scale 1.000000 with limits [1.000000, 5.000000] and userScalable 0.000000
+viewport size 320x352 scale 1.000000 with limits [1.000000, 1.000000] and userScalable 0.000000
 

Will skip for now.
Comment 1 Philippe Normand 2011-03-17 04:04:02 PDT 
Test was introduced in Bug 56527
Comment 2 Joseph Pecoraro 2011-03-17 09:15:51 PDT 
No need to skip. Those results are correct, I made the expected
results by hand (since I didn't have a machine to test on) and the path
landed via the commit-queue, so I was asleep at the time.

I'll fix this up.
Comment 3 Joseph Pecoraro 2011-03-17 09:27:38 PDT 
Landed in r81356:
http://trac.webkit.org/changeset/81356

Unreviewed since this was just bad expected results. Changes:

  • updated expected results
  • unskipped the test on GTK

Sorry about that.
Comment 4 Philippe Normand 2011-03-17 09:28:52 PDT 
Thanks Joseph!