Closed Bug 18690 Opened 26 years ago Closed 26 years ago

Adding image to cart crashes browser

Categories

(Core :: DOM: Core & HTML, defect, P3)

x86
Windows 98
defect

Tracking

()

VERIFIED DUPLICATE of bug 21686

People

(Reporter: johnt, Assigned: vidur)

References

()

Details

(Whiteboard: [MAKINGTEST] pglendinning@cen.com ; no crash on Linux ; simplification/analysis needed of other problem)

Summary explains STEPS: 1) Go to above URL 2) Click Add to Cart button RESULTS: Browser hangs.
Assignee: chofmann → vidur
URL: http://store.corbis.com/corbis_recomm...http://store.corbis.com/corbis_recomm...
Component: other → DOM Level 0
Whiteboard: no crash on Linux ; simplification/analysis needed of other problem
I could not reproduce the crash on Linux. Could someone test on Windows? However, pressing the button does nothing. Moving to DOM Level 0 since it's a javascript: URL that's not doing anything. This needs a testcase.
Interesting.. On winnt (this am's build) I don't crash, but I get a server error: Microsoft VBScript runtime error '800a005e' Invalid use of Null: 'CLng' /inc/shop/xt_orderform_additem.asp, line 122
QA Contact: leger → desale
Updating QA Contact.
In an attempt to get my bug list in order again, marking all the bugs I have currently as ASSIGNED.
I get the same server error on today's mac build
No crash on win98 (ID: 1999122023), only server error
Whiteboard: no crash on Linux ; simplification/analysis needed of other problem → [MAKINGTEST] pglendinning@cen.com ; no crash on Linux ; simplification/analysis needed of other problem
I tried this one with Win-98 as well as Win-95, but I don't see crash at all. Well "Add to Shopping Cart" is not working though.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I am able to add to the cart now
I again tested it with 2000-01-20-09, and looks like its not working for me. Reopening bug.
Status: RESOLVED → REOPENED
Clearing reolution
Resolution: FIXED → ---
The Add To Cart button doesn't work because we aren't cannonicalizing the protocol name of a URI to be all lower case. Since it uses JavaScript: instead of javascript: we don't fire the handler. Fixing the URI code to deal with this makes it all work right - there is no crash as the original bug reported. I'm DUPing this bug with 21686, which has an attached fix for the problem described above. *** This bug has been marked as a duplicate of 21686 ***
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → DUPLICATE
Dup. Verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.