Closed
Bug 121171
Opened 23 years ago
Closed 23 years ago
nsJSEnvironment::CallEventHandler ignores mScriptsEnabled
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla0.9.9
People
(Reporter: rginda, Assigned: rginda)
References
Details
Attachments
(1 file)
|
785 bytes,
patch
|
brendan
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
I'm trying to call nsIScriptContext::SetScriptsEnabled(NS_FALSE) to disable the debug target while we're stopped. This isn't working because jsJSEnvironment::CallEventHandler ignores the current state of mScriptsEnabled. Patch coming up.
|
Assignee | |
Comment 1•23 years ago
|
||
|
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.9
|
||
Comment 2•23 years ago
|
||
Seems like the right thing to do as long as this doesn't cause chrome event handlers to break (like right click n' such) when scripts are disabled.
|
|
Assignee | |
Comment 3•23 years ago
|
||
With JavaScript disabled I can still bring up context menus over content, everything else looks fine too.
|
|
||
Comment 4•23 years ago
|
||
Comment on attachment 65949 [details] [diff] [review] patch to nsJSEnvironment.cpp sr=jst
Attachment #65949 -
Flags: superreview+
|
|
Assignee | |
Comment 5•23 years ago
|
||
thanks jst, anyone free to r=?
|
||
Comment 6•23 years ago
|
||
Comment on attachment 65949 [details] [diff] [review] patch to nsJSEnvironment.cpp r=brendan@mozilla.org for 0.9.9. /be
Attachment #65949 -
Flags: review+
|
|
Assignee | |
Comment 7•23 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
|
||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•