Bug 65366

Summary: [chromium] Fail new fast/js/parseInt.html test
Product: WebKit Reporter: Tony Gentilcore <tonyg@chromium.org>
Component: JavaScriptCoreAssignee: Tony Gentilcore <tonyg@chromium.org>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov@chromium.org, shinyak@google.com, webkit.review.bot@gmail.com
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 65325, 65348    
Attachments:
Description Flags
Patch
none
Patch none

Description From 2011-07-29 04:15:19 PST
fast/js/parseInt.html was added by http://trac.webkit.org/changeset/91966.

JSC passes, but V8 fails one of the assertions:

FAIL state = null; try { parseInt(throwingString, throwingRadix); } catch (e) {} state; should be throwingString. Was throwingRadix.

I'm going to skip the new test for now.
------- Comment #1 From 2011-07-29 04:20:32 PST -------
Created an attachment (id=102343) [details]
Patch
------- Comment #2 From 2011-07-29 04:21:50 PST -------
Committed r91983: <http://trac.webkit.org/changeset/91983>
------- Comment #3 From 2011-07-29 04:22:36 PST -------
Leaving open to track the real fix.
------- Comment #4 From 2011-08-31 07:20:03 PST -------
I've filed the bug in v8.
http://code.google.com/p/v8/issues/detail?id=1649

Also, I've uploaded the CL
http://codereview.chromium.org/7740080/
------- Comment #5 From 2011-09-11 23:05:57 PST -------
Created an attachment (id=107022) [details]
Patch
------- Comment #6 From 2011-09-11 23:31:54 PST -------
(From update of attachment 107022 [details])
Attachment 107022 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/9634876

New failing tests:
fast/js/parseInt.html
------- Comment #7 From 2011-09-12 15:08:04 PST -------
(From update of attachment 107022 [details])
I take it we haven't rolled in the latest v8?  Is that why the cr-linux bot failed?
------- Comment #8 From 2011-09-12 22:21:41 PST -------
(From update of attachment 107022 [details])
Rejecting attachment 107022 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
mismatch : (1)
  svg/custom/svg-fonts-word-spacing.html = IMAGE+TEXT

Regressions: Unexpected image mismatch : (5)
  fast/text/atsui-multiple-renderers.html = IMAGE
  fast/text/international/danda-space.html = IMAGE
  fast/text/international/thai-baht-space.html = IMAGE
  fast/text/international/thai-line-breaks.html = IMAGE
  platform/chromium-linux/fast/text/international/complex-joining-using-gpos.html = IMAGE

Regressions: Unexpected text diff mismatch : (1)
  fast/js/parseInt.html = TEXT



Full output: http://queues.webkit.org/results/9647353
------- Comment #9 From 2011-09-12 22:46:57 PST -------
(From update of attachment 107022 [details])
This needs Chromium r100348 or later?
Let's try again.
------- Comment #10 From 2011-09-12 23:56:46 PST -------
(From update of attachment 107022 [details])
Clearing flags on attachment: 107022

Committed r95021: <http://trac.webkit.org/changeset/95021>
------- Comment #11 From 2011-09-12 23:56:52 PST -------
All reviewed patches have been landed.  Closing bug.