Closed Bug 4332 Opened 26 years ago Closed 26 years ago

NECKO: [4.xP]Second link does not work

Categories

(Core :: Networking, defect, P3)

All
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: christinehoff4, Assigned: gagan)

References

()

Details

Using 3/26 build on Win 95, Win NT, Win 98, Mac8.5 and Linux. This bug came isolating a section of Yahoo's page (http://www.yahoo.com). On the page is a table at the right titled 'In the News'. There are several links to click on. These links work in Comm 4.51 and IE 4.01 but they do not work in Gecko. In the sample test case provided, I included one link to mirror the URL address format used on the Yahoo page, and one link (to Netscape) using a standard absolute URL address format. The Netscape link works in Gecko, the other one doesn't. Both links work in IE 4.01 and Comm 4.51 I wasn't sure what the 'Component' category would be so I guessed at layout.
Assignee: troy → gagan
Component: Layout → Networking Library
The reason the link doesn't work is that the layout code is making a call to NS_MakeAbsoluteURL() which is failing. The call looks like this: NS_MakeAbsoluteURL(aBaseURL, empty, aURLSpec, absURLSpec); and the values are as follows: aBaseURL: mSpec "http://www.yahoo.com/" aURLSpec: "/homer/?http://headlines.yahoo.com/FC/US/Assisted_Suicide/" Here's the stack trace: NS_MakeAbsoluteURL(nsIURL * 0x014b7f00, const nsString & {""}, const nsString & {"/homer/?http://headlines.yahoo.com/FC/US/Assisted_Suicide/"}, nsString & {""}) line 1134 nsGenericElement::TriggerLink(nsIPresContext & {...}, nsLinkVerb eLinkVerb_Replace, nsIURL * 0x014b7f00, const nsString & {"/homer/?http://headlines.yahoo.com/FC/US/Assisted_Suicide/"}, const nsString & {""}, int 0) line 1153 + 22 bytes nsHTMLAnchorElement::HandleDOMEvent(nsHTMLAnchorElement * const 0x014b692c, nsIPresContext & {...}, nsEvent * 0x0012f8d4, nsIDOMEvent * * 0x0012f81c, unsigned int 2, nsEventStatus & nsEventStatus_eIgnore) line 338 nsGenericDOMDataNode::HandleDOMEvent(nsIPresContext & {...}, nsEvent * 0x0012f8d4, nsIDOMEvent * * 0x0012f81c, unsigned int 1, nsEventStatus & nsEventStatus_eIgnore) line 671 + 39 bytes nsTextNode::HandleDOMEvent(nsTextNode * const 0x014b68bc, nsIPresContext & {...}, nsEvent * 0x0012f8d4, nsIDOMEvent * * 0x00000000, unsigned int 1, nsEventStatus & nsEventStatus_eIgnore) line 199 nsEventStateManager::GenerateMouseEnterExit(nsIPresContext & {...}, nsGUIEvent * 0x0012fbbc) line 304 nsEventStateManager::PreHandleEvent(nsEventStateManager * const 0x014b6240, nsIPresContext & {...}, nsGUIEvent * 0x0012fbbc, nsIFrame * 0x014b5030, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 107 PresShell::HandleEvent(PresShell * const 0x014a92a4, nsIView * 0x014b2df0, nsGUIEvent * 0x0012fbbc, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 1960 + 34 bytes nsView::HandleEvent(nsView * const 0x014b2df0, nsGUIEvent * 0x0012fbbc, unsigned int 8, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 825 nsView::HandleEvent(nsView * const 0x014b7730, nsGUIEvent * 0x0012fbbc, unsigned int 8, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 808 nsView::HandleEvent(nsView * const 0x014b77d0, nsGUIEvent * 0x0012fbbc, unsigned int 8, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 808 nsScrollingView::HandleEvent(nsScrollingView * const 0x014b77d0, nsGUIEvent * 0x0012fbbc, unsigned int 8, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 840 nsView::HandleEvent(nsView * const 0x014a81f0, nsGUIEvent * 0x0012fbbc, unsigned int 28, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 808 nsViewManager::DispatchEvent(nsViewManager * const 0x014a8410, nsGUIEvent * 0x0012fbbc, nsEventStatus & nsEventStatus_eConsumeDoDefault) line 1710 HandleEvent(nsGUIEvent * 0x0012fbbc) line 64 nsWindow::DispatchEvent(nsWindow * const 0x014b7630, nsGUIEvent * 0x0012fbbc, nsEventStatus & nsEventStatus_eIgnore) line 416 + 10 bytes nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012fbbc) line 432 nsWindow::DispatchMouseEvent(unsigned int 300, nsPoint * 0x00000000 {x=??? y=???}) line 2466 + 15 bytes ChildWindow::DispatchMouseEvent(unsigned int 300, nsPoint * 0x00000000 {x=??? y=???}) line 2615 nsWindow::ProcessMessage(unsigned int 512, unsigned int 0, long 2883645, long * 0x0012fef4) line 1813 + 24 bytes nsWindow::WindowProc(HWND__ * 0x00080514, unsigned int 512, unsigned int 0, long 2883645) line 475 + 27 bytes USER32! 77e71250() RAPTORGFXWIN! ??_C@_08JKC@crtdll?4c?$AA@ + 2525 bytes
QA Contact: 4144 → 3819
Status: NEW → ASSIGNED
Another MakeAbsURL bug... I have duplicate somewhere.
*** Bug 4457 has been marked as a duplicate of this bug. ***
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
dp marked this fixed today....paulmac or chrisd, can you verify with April 9 build?
Status: RESOLVED → REOPENED
The second link still doesn't go anywhere. Re-opening bug.
BTW, used 040818 builds on Windows 95. The first link works, the second doesn't. Works fine in Nav4.51.
Resolution: FIXED → ---
Deferring till Necko lands...
Per DP's suggestion marking these till M8. Though Necko lands with M7, we will be able to verify it for M8.
I'm moving this to target M9, Necko will be enabled somewhere during late M8 or early M9. We will need to get on this and it cannot be postponed past the M9 milestone.
Changing all Networking Library/Browser bugs to Networking-Core component for Browser. Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do this in a bulk change. If this happens, I will fix. ;-)
Summary: [4.xP]Second link does not work → NECKO: [4.xP]Second link does not work
Should be fixed with Necko. Pl. verify.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
necko rules, verified fixed with 7/31 builds
Bulk move of all Networking-Core (to be deleted component) bugs to new Networking component.
You need to log in before you can comment on or make changes to this bug.