The fix checked in to the 1.4 branch on 10/28 for bug 249332 is dependent upon XPCNativeWrapper, which doesn't exist. (At least middle click pasting of URLs into the content area and middle clicking to open a link in a new tab is broken as a result. Error message is displayed in the JS console: XPCNativeWrapper is not defined ...) Adding XPCNativeWrapper is minor. Patch coming up.
Created attachment 166404 [details] [diff] [review] proposed fix Add XPCNativeWrapper, and include it from files that include contentAreaClick.js.
Comment on attachment 166404 [details] [diff] [review] proposed fix That's odd - trunk gets away with including it in contentAreaContextOverlay.xul which doesn't itself include contentAreaClick.js
(In reply to comment #2) Including it from contentAreaContextOverlay.xul only seems fine for the browser but XPCNativeWrapper isn't defined for messageWindow or p3pSummary. (It seems this behavior exists in 1.7 and trunk, bug 267378, also a result of bug 249332.)
Comment on attachment 166404 [details] [diff] [review] proposed fix r/a=caillon for 1.4 only
Attachment #166404 - Flags: review?(jag) → review+
Comment on attachment 166404 [details] [diff] [review] proposed fix Trunk seems to have the includes in different files but whatever works...
Attachment #166404 - Flags: superreview?(neil.parkwaycc.co.uk) → superreview+
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.