Ticket #4826: 4826.patch

_source/plugins/contextmenu/plugin.js

@@ -183 +183 @@
                                 {
                                         var domEvent = event.data;
 
-                                        if ( nativeContextMenuOnCtrl
-                                                 && ( domEvent.$.ctrlKey || domEvent.$.metaKey ) )
+                                        if ( nativeContextMenuOnCtrl &&
+                                             // Safari on Windows always show 'ctrlKey' as true in 'contextmenu' event,
+                                                // which make this property unreliable. (#4826)    
+                                             ( safariOnWin ? holdCtrlKey : domEvent.$.ctrlKey || domEvent.$.metaKey ) )
                                                 return;
 
                                         // Cancel the browser context menu.
@@ -201 +203 @@
                                                 0, this );
                                 },
                                 this );
-                },
+
+                        var safariOnWin = CKEDITOR.env.webkit && !CKEDITOR.env.mac;
+                        if( nativeContextMenuOnCtrl && safariOnWin )
+                        {
+                                var holdCtrlKey,
+                                        onKeyDown = function( event )
+                                        {
+                                                holdCtrlKey = event.data.$.ctrlKey || event.data.$.metaKey;
+                                        },
+                                        resetOnKeyUp = function()
+                                        {
+                                                holdCtrlKey = 0;
+                                        };
 
+                                element.on( 'keydown', onKeyDown );
+                                element.on( 'keyup', resetOnKeyUp );
+                                element.on( 'contextmenu', resetOnKeyUp );
+                        }
+                },
+
                 addListener : function( listenerFn )
                 {
                         this._.listeners.push( listenerFn );