Do not extend the toolbarGroups config object

[Frederico Caldeira Knabben]

Change the appendto.html sample with the following:

var globalConfig = {
	height: 50,
	toolbarGroups : [ { name: 'basicstyles', groups: [ 'basicstyles' ] } ]
};

CKEDITOR.appendTo( 'section1', globalConfig );
CKEDITOR.appendTo( 'section1', globalConfig );
CKEDITOR.appendTo( 'section1', globalConfig );

Check the results :|

It seems that we're touching the original toolbarGroups configuration object internally. This brings troubles when there is a global configuration option in the application, which is reused by several editors.

If we want to use the original toolbarGroups object internally, we should make a depth copy of it instead.

[Piotrek Koszuliński]

Pushed t/9636 on dev and tests.

[Garry Yao]

To be bulletproof on other object based configuration as well, let's have it fixed it in the editor creation instead.

[Piotrek Koszuliński]

Sample 9636b.html shows that t/9636b solves only problem with sharing instance configs - global config is still shared. Thus t/9636 is still required to fix this completely. So we should use both patches. Original one to fix this exact issue and t/9636b to prevent some of the future issue with instance config sharing. But this still won't be enough to ensure 100% safety which can be achieved only by deep cloning of the entire config object.

[Piotrek Koszuliński]

Masterised ​https://github.com/ckeditor/ckeditor-dev/commit/3aeeb21