Report phishing Website opens with middle click even when disabled

RESOLVED FIXED in Firefox 2 beta2

Status

()

Firefox
Menus
--
minor
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Aiing, Assigned: Gavin)

Tracking

({fixed1.8.1})

2.0 Branch
Firefox 2 beta2
fixed1.8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060710 BonEcho/2.0b1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060710 BonEcho/2.0b1

On a blank tab Middle mouse click on Help --> Report Phishing Website menu item opens the report phishing website page.

 

Reproducible: Always

Steps to Reproduce:
1. Open a blank tab
2. Open Help Menu
3. Middle click "Report Phishing Website" (greyed out)

Actual Results:  
Report phishing website page opens

Expected Results:  
Nothing happens

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060710 BonEcho/2.0b1 ID:2006071003

Comment 1

12 years ago
Confirmed.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: unspecified → 2.0 Branch

Updated

12 years ago
Blocks: 337484
This isn't really a regression from bug 337484, checkForMiddleClick just doesn't check whether the node it's calling oncommand for is disabled. You can see the same bug if you middle click on a disabled back button, though it only throws so it's not nearly as noticeable.
Severity: normal → minor
OS: Windows XP → All
Hardware: PC → All
Summary: Report phishing Website opens with middle click even when greyed out → Report phishing Website opens with middle click even when disabled
Created attachment 228782 [details] [diff] [review]
patch

checkForMiddleClick is currently used on the following elements:
menuitem, menu, toolbarbutton, popup, menupopup

Of these, the disabled attribute only has an effect on the first three, but checking it for =="true" shouldn't hurt in the [menu]popup case.
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Attachment #228782 - Flags: review?(bugs.mano)
Comment on attachment 228782 [details] [diff] [review]
patch

r=mano if you file a bug on the menuitem binding only caliming to implement nsIDOMXULSelectControlElement
(I'm not sure if this is the right interface to implement, but we sure need the disabled property implemented when the attribute affects the element behavior).

Add a comment above the check referring to that bug.
Attachment #228782 - Flags: review?(bugs.mano) → review+
Whiteboard: [checkin needed]
mozilla/browser/base/content/utilityOverlay.js 	1.39
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed] → [need-a]
Target Milestone: --- → Firefox 2 beta2
Attachment #228782 - Flags: approval1.8.1?

Updated

12 years ago
Attachment #228782 - Flags: approval1.8.1? → approval1.8.1+
mozilla/browser/base/content/utilityOverlay.js 	1.32.2.9 
Keywords: fixed1.8.1
Whiteboard: [need-a]
You need to log in before you can comment on or make changes to this bug.