87524 - No right click menu (back, forward, ....)

Status: RESOLVED DUPLICATE of bug 87289

(SeaMonkey :: General, defect)

Description

[Sirron Davis]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.1+) Gecko/20010624
BuildID:    2001062405

When you go this (and several other) site, no right click menu will appear. If
you want to go back (to your previous page) you must click the back button.

Reproducible: Always
Steps to Reproduce:
1. Go to any web page
2. Go to http://www.sfgate.com
3. Right click on the page.

Actual Results:  No menu will appear

Expected Results:  context menu should appear.

If there is an advertisement on the page, you can right click on the
advertisement, and display the context menu.

Sometimes if you right click on a link, you will get an empty window.

Comment 1

[Francisco León]

There are LOTS of pages that disable right-click, specially on internet explorer
So this is probably not a bug, however, most commercial and serious webpages do
not disable it, so it is a little suspicious, so i checked in with other browser
and indeed it works with konqueror and netscape so i am confirming this as a bug

Comment 2

[Jason Johnston]

This shows up in JS console on right-click:

Error: baseURL is not defined
Source File: chrome://communicator/content/nsContextMenu.js
Line: 682

Error: contextMenu has no properties
Source File: chrome://cookie/content/cookieContextOverlay.xul
Line: 94

Comment 3

[Jason Johnston]

nsContextMenu.js Line 682 is in this function:

makeURLAbsolute : function ( base, url ) {
        // Construct nsIURL.
        var ioService = Components.classes["@mozilla.org/network/io-service;1"]
                      .getService(Components.interfaces.nsIIOService);
        var uriToAdd  = ioService.newURI(base, null);
        // Resolve
<ln.682>var result = baseURL.resolve( url );
        return result;
    }

Changing baseURL to uriToAdd fixes the problem for me.

Comment 4

[Chris Hiner]

Dup of bug 87289?

Comment 5

[Jason Johnston]

Yes, most definitely a dupe of bug 87289 -- makeURLAbsolute() is called when
activated over an element with a background image. It's also called on <input
type="image">, list-style-image, and some other element types.

Looking back through the nsContextMenu.js from 0.9.1, makeURLAbsolute() was much
different at that time; it used a different syntax for getting the service and
used baseURL as its main object. It looks like sometime recently the syntax was
updated and different variables were used, but baseURL was left on that one line
so that's what's causing the problems.

Comment 6

[Jason Johnston]

Found the culprit-- checkin by dougt on 06/21/2001 15:02 as part of the url
parsing branch, bug 73845

Comment 7

[Tom Mraz]

Marking duplicate. More progress is on the older bug.


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