Bug 90852

Summary:

REGRESSION(r122166): It made 170 tests crash on 32 bit platforms

Product:

WebKit

Reporter:

Csaba Osztrogonác <ossy>

Component:

JavaScriptCore

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Blocker

CC:

barraclough, fpizlo, ossy, webkit.review.bot

Priority:

Keywords:

Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

79668, 90731

Attachments:

Description	Flags
the patch	none
better patch	none

Csaba Osztrogonác

Reported 2012-07-10 00:04:50 PDT

https://trac.webkit.org/changeset/122166 made 170 tests cras on 32 bit platforms. (at least on Qt and GTK) GTK: http://build.webkit.org/results/GTK%20Linux%2032-bit%20Release/r122167%20%2824889%29/results.html Qt: http://build.webkit.org/results/Qt%20Linux%20Release/r122167%20%2849248%29/results.html

Attachments
the patch (1.54 KB, patch) 2012-07-10 01:21 PDT, Filip Pizlo	no flags	Details Formatted Diff Diff
better patch (1.56 KB, patch) 2012-07-10 01:30 PDT, Filip Pizlo	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Filip Pizlo

Comment 1 2012-07-10 01:21:22 PDT

Created attachment 151416 [details] the patch This might fix it.

Filip Pizlo

Comment 2 2012-07-10 01:30:59 PDT

Created attachment 151417 [details] better patch Really fix it this time.

Zoltan Herczeg

Comment 3 2012-07-10 01:49:12 PDT

Comment on attachment 151417 [details] better patch r=me

Csaba Osztrogonác

Comment 4 2012-07-10 02:36:25 PDT

Comment on attachment 151417 [details] better patch Clearing flags on attachment: 151417 Committed r122208: <http://trac.webkit.org/changeset/122208>

Csaba Osztrogonác

Comment 5 2012-07-10 02:36:32 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.