Ticket #5395: 5395.patch

_source/core/dom/domobject.js

@@ -245 +245 @@
                 return this.$._cke_expando || ( this.$._cke_expando = CKEDITOR.tools.getNextNumber() );
         };
 
+        /**
+         * Feature detecting the availability of the specified event on this document/element.
+         * @param {String} eventName
+         * @return {Boolean} true if event is supported
+         */
+        domObjectProto.supports = (function()
+        {
+                var cache = {};
+                function isEventSupported( eventName )
+                {
+                        var type = this.$.nodeType;
+                        if ( !( type == CKEDITOR.NODE_DOCUMENT ||
+                                        type == CKEDITOR.NODE_DOCUMENT ) )
+                                return false;
+
+                        eventName = 'on' + eventName;
+
+                        var element = this.$,
+                                nodeName = element.nodeName;
+
+                        // return cached result.
+                        if ( nodeName in cache )
+                                return cache[ nodeName ];
+
+                        // When using `setAttribute`, IE skips "unload",
+                        // WebKit skips "unload" and "resize", whereas `in` "catches" those
+                        var isSupported = ( eventName in element );
+
+                        if ( !isSupported )
+                        {
+                                // if it has no `setAttribute` (i.e. doesn't implement Node interface), try generic element
+                                if ( !element.setAttribute )
+                                        element = document.createElement( 'div' );
+                                if ( element.setAttribute && element.removeAttribute )
+                                {
+                                        element.setAttribute( eventName, '' );
+                                        isSupported = typeof element[ eventName ] == 'function';
+
+                                        // if property was created, "remove it" (by setting value to `undefined`)
+                                        if ( typeof element[ eventName ] != 'undefined' )
+                                                element[ eventName ] = undefined;
+                                        element.removeAttribute( eventName );
+                                }
+                        }
+
+                        element = null;
+                        return ( cache[ nodeName ] = isSupported );
+                }
+
+                return isEventSupported;
+        })()
+
         // Implement CKEDITOR.event.
         CKEDITOR.event.implementOn( domObjectProto );
 

_source/plugins/contextmenu/plugin.js

@@ -147 +147 @@
         {
                 addTarget : function( element, nativeContextMenuOnCtrl )
                 {
-                        // Opera doesn't support 'contextmenu' event, we have duo approaches employed here:
-                        // 1. Inherit the 'button override' hack we introduced in v2 (#4530), while this require the Opera browser
-                        //  option 'Allow script to detect context menu/right click events' to be always turned on.
+                        // For browsers (Opera <=10a) that doesn't  support 'contextmenu' event, we have duo approaches employed here:
+                        // 1. Inherit the 'button override' hack we introduced in v2 (#4530) (In Opera browser, this require the
+                        //  option 'Allow script to detect context menu/right click events' to be always turned on).
                         // 2. Considering the fact that ctrl/meta key is not been occupied
                         //  for multiple range selecting (like Gecko), we use this key
                         //  combination as a fallback for triggering context-menu. (#4530)
-                        if ( CKEDITOR.env.opera )
+                        if ( !element.supports( 'contextmenu' ) )
                         {
                                 var contextMenuOverrideButton;
                                 element.on( 'mousedown', function( evt )