Bug 65366 – [chromium] Fail new fast/js/parseInt.html test

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Tony Gentilcore]

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.

[Tony Gentilcore]

Created attachment 102343 [details]
Patch

[Tony Gentilcore]

Committed r91983: <http://trac.webkit.org/changeset/91983>

[Tony Gentilcore]

Leaving open to track the real fix.

[Shinya Kawanaka]

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/

[Shinya Kawanaka]

Created attachment 107022 [details]
Patch

[WebKit Review Bot]

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

[Eric Seidel (no email)]

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?

[WebKit Review Bot]

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

[Kent Tamura]

Comment on attachment 107022 [details]
Patch

This needs Chromium r100348 or later?
Let's try again.

[WebKit Review Bot]

Comment on attachment 107022 [details]
Patch

Clearing flags on attachment: 107022

Committed r95021: <http://trac.webkit.org/changeset/95021>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.