Bug 16889 - REGRESSION (r29425): Canvas-based graphing calculator fails to run
Summary: REGRESSION (r29425): Canvas-based graphing calculator fails to run
Status: RESOLVED DUPLICATE of bug 17164
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P1 Normal
Assignee: Nobody
URL: http://www.calc5.com/
Keywords: InRadar, NeedsReduction, Regression
Depends on: 17164
Blocks:
  Show dependency tree / graph
 
Reported: 2008-01-16 00:30 PST by Oliver Hunt
Modified: 2008-02-04 22:29 PST (History)
6 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Hunt 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.
Comment 1 Oliver Hunt 2008-01-16 00:37:13 PST 
Regression occurs between r29414 and r29438
Comment 2 Oliver Hunt 2008-01-16 00:40:28 PST 
Erk, this includes the activation patch, which could easily be the cause of the badness :(
Comment 3 Cameron Zwarich (cpst) 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.
Comment 4 Oliver Hunt 2008-01-16 09:25:45 PST 
yay google code! /me curses obfuscation tools
Comment 5 Cameron Zwarich (cpst) 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.
Comment 6 Cameron Zwarich (cpst) 2008-01-18 16:26:34 PST 
This is not fixed by Maciej's fix for bug 16909.
Comment 7 Oliver Hunt 2008-01-18 17:01:49 PST 
That makes me sad :(
Comment 8 David Kilzer (:ddkilzer) 2008-01-19 03:05:26 PST 
<rdar://problem/5696255>
Comment 9 David Kilzer (:ddkilzer) 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.
Comment 10 Oliver Hunt 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
Comment 11 Cameron Zwarich (cpst) 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.
Comment 12 Cameron Zwarich (cpst) 2008-02-04 02:09:25 PST 
This bug is fixed by the patch for bug 17164.
Comment 13 Oliver Hunt 2008-02-04 22:29:51 PST 

*** This bug has been marked as a duplicate of 17164 ***