Bug 90658

Summary: REGRESSION(r121925): It broke 5 sputnik tests on x86 platforms
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, fpizlo, loki, ossy, pvarga, zherczeg
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 79666, 90347    
Attachments:
Description Flags
the patch zherczeg: review+

Description Csaba Osztrogonác 2012-07-05 21:55:48 PDT
--- /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_218/S7.6_A1.1_T2-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_218/S7.6_A1.1_T2-actual.txt 
@@ -1,6 +1,6 @@
 S7.6_A1.1_T2
 
-FAIL SputnikError: #2118 
+FAIL SputnikError: #2RangeError: 1RangeError: 1RangeError: 8 
 
 TEST COMPLETE
 


--- /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_320/S7.6_A3.1-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_320/S7.6_A3.1-actual.txt 
@@ -1,6 +1,6 @@
 S7.6_A3.1
 
-FAIL SputnikError: #0221 
+FAIL SputnikError: #0000 
 
 TEST COMPLETE
 



--- /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_320/S7.6_A3.2-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_320/S7.6_A3.2-actual.txt 
@@ -1,6 +1,6 @@
 S7.6_A3.2
 
-FAIL SputnikError: #0221 
+FAIL SputnikError: #0000 
 
 TEST COMPLETE
 


--- /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_410/S7.6_A3.1-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_410/S7.6_A3.1-actual.txt 
@@ -1,6 +1,6 @@
 S7.6_A3.1
 
-FAIL SputnikError: #0242-024F 
+FAIL SputnikError: #0000-0000 
 
 TEST COMPLETE
 


--- /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_410/S7.6_A3.2-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/sputnik/Unicode/Unicode_410/S7.6_A3.2-actual.txt 
@@ -1,6 +1,6 @@
 S7.6_A3.2
 
-FAIL SputnikError: #0242-024F 
+FAIL SputnikError: #0000-0000 
 
 TEST COMPLETE
Comment 1 Filip Pizlo 2012-07-05 21:56:43 PDT
Ah darn!  I will look at this asap.
Comment 2 Csaba Osztrogonác 2012-07-05 22:07:13 PDT
It isn't Qt specific bug, but general 32 bit. (these tests fail on GTK 32 bit too.)
Comment 3 Filip Pizlo 2012-07-05 23:55:40 PDT
Oh wow that was a dumb mistake.  Simple patch on the way.
Comment 4 Filip Pizlo 2012-07-05 23:58:38 PDT
Created attachment 151023 [details]
the patch
Comment 5 Zoltan Herczeg 2012-07-06 00:00:45 PDT
Comment on attachment 151023 [details]
the patch

r=me
Comment 6 Filip Pizlo 2012-07-06 00:01:55 PDT
Fix landed in http://trac.webkit.org/changeset/121946