Ticket #8342: 8342_2.patch

_source/plugins/clipboard/plugin.js

@@ -290 +290 @@
                 return retval;
         }
 
-        var inReadOnly;
         function setToolbarStates()
         {
                 if ( this.mode != 'wysiwyg' )
                         return;
 
-                this.getCommand( 'cut' ).setState( inReadOnly ? CKEDITOR.TRISTATE_DISABLED : stateFromNamedCommand( 'Cut', this ) );
+                this.getCommand( 'cut' ).setState( this.readOnly ? CKEDITOR.TRISTATE_DISABLED : stateFromNamedCommand( 'Cut', this ) );
                 this.getCommand( 'copy' ).setState( stateFromNamedCommand( 'Copy', this ) );
-                var pasteState = inReadOnly ? CKEDITOR.TRISTATE_DISABLED :
+                var pasteState = this.readOnly ? CKEDITOR.TRISTATE_DISABLED :
                                                 CKEDITOR.env.webkit ? CKEDITOR.TRISTATE_OFF : stateFromNamedCommand( 'Paste', this );
                 this.fire( 'pasteState', pasteState );
         }
@@ -411 +410 @@
                                         body.on( 'keyup', setToolbarStates, editor );
                                 });
 
-                                // For improved performance, we're checking the readOnly state on selectionChange instead of hooking a key event for that.
-                                editor.on( 'selectionChange', function( evt )
-                                {
-                                        inReadOnly = evt.data.selection.getRanges()[ 0 ].checkReadOnly();
-                                        setToolbarStates.call( editor );
-                                });
 
                                 // If the "contextmenu" plugin is loaded, register the listeners.
                                 if ( editor.contextMenu )