Closed
Bug 18690
Opened 26 years ago
Closed 26 years ago
Adding image to cart crashes browser
Categories
(Core :: DOM: Core & HTML, defect, P3)
Tracking
()
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.
Comment 2•26 years ago
|
||
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
| Assignee | ||
Comment 4•26 years ago
|
||
In an attempt to get my bug list in order again, marking all the bugs I have
currently as ASSIGNED.
Updated•26 years ago
|
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
Comment 7•26 years ago
|
||
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
Comment 9•26 years ago
|
||
I again tested it with 2000-01-20-09, and looks like its not working for me.
Reopening bug.
Status: RESOLVED → REOPENED
| Assignee | ||
Comment 11•26 years ago
|
||
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 ago → 26 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•