Closed Bug 255256 Opened 20 years ago Closed 11 years ago

disallow context menus script setting thing by default

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: asa, Unassigned)

Details

Attachments

(1 file)

We should set the pref by default so sites can't hijack the context menu. If
whoever is in there doing that would like to make me happy, they could also make
that window resizeable.
(and maybe we should also just disable raising and lowering of windows and
moving or resizing existing windows too. all of this should help to prevent
spoofing)
Some more annoyances that should be disabled by default:
Bug 186708 resizing windows that have toolbars
Bug 177838 creating non-resizable windows
I agree. That option should be set by default.
Attached patch PatchSplinter Review
Asa, do you want to drive this in?
Assignee: firefox → gavin.sharp
Status: NEW → ASSIGNED
Attachment #180425 - Flags: review?(mconnor)
Whiteboard: [patch-r?]
Comment on attachment 180425 [details] [diff] [review]
Patch

This'll have unexpected behaviour with the new prefwindow code.  Disable common
annoyances would then do nothing with this pref, then set it back.  Not
acceptable behaviour.

Jesse, you had something written up about disabling some of these by default
and removing that poorly defined checkbox, care to share?
Attachment #180425 - Flags: review?(mconnor) → review-
Attachment #180425 - Attachment is obsolete: true
OS: Windows XP → All
Hardware: PC → All
Whiteboard: [patch-r?]
Version: unspecified → Trunk
(In reply to comment #5)
> This'll have unexpected behaviour with the new prefwindow code.  Disable common
> annoyances would then do nothing with this pref, then set it back.  Not
> acceptable behaviour.

I think it should be noted that the new pref window already does this for some
other prefs (bug 284079). That doesn't make this the right fix, of course, but
it emphasizes the need for a better general solution.
Comment on attachment 180425 [details] [diff] [review]
Patch

This should work now that the prefs UI was redone.
Attachment #180425 - Attachment is obsolete: false
Attachment #180425 - Flags: review- → review?
Attachment #180425 - Flags: review? → review?(mconnor)
See bug 299424 comment 7.
Assignee: gavin.sharp → nobody
Status: ASSIGNED → NEW
So the patch can now be reviewed again?
Many sites now depend on this behavior, for example Google Docs provides their own context menu that is specific to the workflow/actions of the application. Changing this will break those sites.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: