Bug 65366

Summary: [chromium] Fail new fast/js/parseInt.html test
Product: WebKit Reporter: Tony Gentilcore <tonyg>
Component: JavaScriptCoreAssignee: Tony Gentilcore <tonyg>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, shinyak, webkit.review.bot
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 Tony Gentilcore 2011-07-29 04:15:19 PDT
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 Tony Gentilcore 2011-07-29 04:20:32 PDT
Created attachment 102343 [details]
Patch
Comment 2 Tony Gentilcore 2011-07-29 04:21:50 PDT
Committed r91983: <http://trac.webkit.org/changeset/91983>
Comment 3 Tony Gentilcore 2011-07-29 04:22:36 PDT
Leaving open to track the real fix.
Comment 4 Shinya Kawanaka 2011-08-31 07:20:03 PDT
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 Shinya Kawanaka 2011-09-11 23:05:57 PDT
Created attachment 107022 [details]
Patch
Comment 6 WebKit Review Bot 2011-09-11 23:31:54 PDT
Comment on attachment 107022 [details]
Patch

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 Eric Seidel 2011-09-12 15:08:04 PDT
Comment on attachment 107022 [details]
Patch

I take it we haven't rolled in the latest v8?  Is that why the cr-linux bot failed?
Comment 8 WebKit Review Bot 2011-09-12 22:21:41 PDT
Comment on attachment 107022 [details]
Patch

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 Kent Tamura 2011-09-12 22:46:57 PDT
Comment on attachment 107022 [details]
Patch

This needs Chromium r100348 or later?
Let's try again.
Comment 10 WebKit Review Bot 2011-09-12 23:56:46 PDT
Comment on attachment 107022 [details]
Patch

Clearing flags on attachment: 107022

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