I personally doubt that's a good idea even more considering user can disable autofocus. The option to disable autofocus could be removed if we want to disable autofocus when script is disabled.
We shouldn't. Not requiring scripting to be enabled is one of the main points of autofocus.
(In reply to comment #1) > We shouldn't. Not requiring scripting to be enabled is one of the main points > of autofocus. I agree but sicking doesn't, claiming there is a security treat for people using autofocus which is not acceptable if they have disabled script (considering that's often done for security reasons).
I don't want people to feel that we have provided them with a pref to turn of scripting, and then we inject a different way of doing script-like things and which doesn't honor the pref. Not requiring authors to use scripting was one of the goals I agree. Being able to use autofocus for users that have disabled scripts was not a goal as far as I am aware.
If there are features that could cause problems, we should fix those problems rather than adding even more untested pref-governed branches to our code.
Are you suggesting we disable scripts since some people think it's too troublesome?
I think we should try to fix bug 601030 then we will see what could still be a security issue with this feature. If there is no, I think we should just keep it enabled by default and not assume that people who disable scripts want it to be disabled.
Depends on: 601030
Jonas, would you be fine to mark this bug WONTFIX given that bug 601030 seems to be in a good path to be fixed?
I guess I'm not religious about fixing this bug. As long as we have a separate pref for disabling autofocus? (We usually do for *any* new feature, as a safty measure if there is any bugs in the implementation or any security aspects that we've missed)
(In reply to comment #11) > I guess I'm not religious about fixing this bug. As long as we have a separate > pref for disabling autofocus? We do. I'm marking this bug WONTFIX then.
Assignee: nobody → mounir.lamouri
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.