RESOLVED DUPLICATE of bug 17164 16889
REGRESSION (r29425): Canvas-based graphing calculator fails to run 
https://bugs.webkit.org/show_bug.cgi?id=16889
Summary REGRESSION (r29425): Canvas-based graphing calculator fails to run
Oliver Hunt
Reported 2008-01-16 00:30:18 PST
The graphing calcualtor at calc5.com now completely fails to load or run, it appears to produce "undefined value" exception when it did not used to.
Attachments
Add attachment proposed patch, testcase, etc.
Oliver Hunt
Comment 1 2008-01-16 00:37:13 PST
Regression occurs between r29414 and r29438
Oliver Hunt
Comment 2 2008-01-16 00:40:28 PST
Erk, this includes the activation patch, which could easily be the cause of the badness :(
Cameron Zwarich (cpst)
Comment 3 2008-01-16 05:46:53 PST
I can confirm that it doesn't work. It also doesn't work with my latest patch for bug 16868. Once that gets landed, I'll fix this. It doesn't look like it'll be too bad. The JS isn't obfuscated in any way like the Google stuff.
Oliver Hunt
Comment 4 2008-01-16 09:25:45 PST
yay google code! /me curses obfuscation tools
Cameron Zwarich (cpst)
Comment 5 2008-01-16 17:17:35 PST
I can confirm our suspicions that the bug is caused by r29425 (the ActivationImp tear-off patch). I patched a clean r29414 tree with it and it no longer worked.
Cameron Zwarich (cpst)
Comment 6 2008-01-18 16:26:34 PST
This is not fixed by Maciej's fix for bug 16909.
Oliver Hunt
Comment 7 2008-01-18 17:01:49 PST
That makes me sad :(
David Kilzer (:ddkilzer)
Comment 8 2008-01-19 03:05:26 PST
<rdar://problem/5696255>
David Kilzer (:ddkilzer)
Comment 9 2008-01-22 18:02:10 PST
(In reply to comment #5) > I can confirm our suspicions that the bug is caused by r29425 (the > ActivationImp tear-off patch). I patched a clean r29414 tree with it and it no > longer worked. The ActivationImp issue is now resolved, but the results for the calculator don't display properly (unlike on Firefox). I think this bug should be kept open, and have its focus change to resolve the issue with displaying results.
Oliver Hunt
Comment 10 2008-01-22 19:47:32 PST
Dave, what makes you think that the activation imp issue is resolved? afaict the activation patch introduced the undefined variable exception, and that is still present in ToT -- the various activation related fixes were to stop crashes in GC, not fix correctness bugs -- none even touched correctness
Cameron Zwarich (cpst)
Comment 11 2008-02-03 01:43:47 PST
(In reply to comment #9) > (In reply to comment #5) > > I can confirm our suspicions that the bug is caused by r29425 (the > > ActivationImp tear-off patch). I patched a clean r29414 tree with it and it no > > longer worked. > > The ActivationImp issue is now resolved, but the results for the calculator > don't display properly (unlike on Firefox). I think this bug should be kept > open, and have its focus change to resolve the issue with displaying results. Like Oliver said, the GC marking issues were resolved, but this bug didn't change one bit. It is such a random error that it must be caused by some strange behaviour introduced by the ActivationImp tear-off. There are a few of these bugs now, but my previous attempts to reduce them failed. I will try again with the calculator now that I have more free time because I really want to see this resolved.
Cameron Zwarich (cpst)
Comment 12 2008-02-04 02:09:25 PST
This bug is fixed by the patch for bug 17164.
Oliver Hunt
Comment 13 2008-02-04 22:29:51 PST
*** This bug has been marked as a duplicate of 17164 ***
Note You need to log in before you can comment on or make changes to this bug.