pastefromword should not depend on configuration from other styles

[Garry Yao]

Paste from word read the configuration on editor styles (block and inline) to properly migrate the elements, but it should still work without those plugins.

[Garry Yao]

Opened t/9709 for review.

[Piotrek Koszuliński]

I moved branch from old cksource remote to Github: ​git:eb9371d.

[Piotrek Koszuliński]

Pastefromword depends also on basicstyles, font and colobutton.

​https://github.com/ckeditor/ckeditor-dev/blob/master/plugins/pastefromword/filter/default.js#L916-L933

HTML to verify format dependency:

<h1 class="MsoNormal">xxx</h1>

HTML to verify basicstyles dependency:

<p class="MsoNormal">b<b>b</b>b</p>

<p>xxx</p>

HTML to verify font dependency (config.pasteFromWordRemoveFontStyles has to be set to false):

<p class="MsoNormal">x<span style="font-family:Arial">x</span>x</p>

<p>xxx</p>

HTML to verify colorbutton dependency (config.pasteFromWordRemoveFontStyles has to be set to false):

<p class="MsoNormal">x<span style="color:red">x</span>x</p>

<p>xxx</p>

I was copy&pasting these on Chrome. Other browsers may not preserve styles.

[Piotrek Koszuliński]

I forgot to mention why I R-ed this branch :D.

The idea proposed in ​​git:eb9371d is good and would help if there hasn't been an error. There's an old typo in variable name - styleDefiniton.

Also, I think that this issue deserves tests. Testing pasteFromWord is pretty simple having code samples I attached in previous comment and assertPasteEvent function with improvements for async execution from #9456.

Please, also try to avoid too much conflicts with tests for #8893.

[Garry Yao]

Forced update the typo fixes with ​a624914

Opened t/9709 for style related tests.

[Piotrek Koszuliński]

​http://ckeditor4.t/dt/plugins/pastefromword/keep_styles.html is failing on IE9 because of spaces inside style attributes.

I pushed untracked file that was added by #9456.

[Piotrek Koszuliński]

I decided that for now we can only use compatHtml - it makes comparison a little bit senseless, because it fixes a lot of issues, but we don't have better (highly configurable) parser&writer duet.

Pushed few fixes to t/9709@tests.

[Piotrek Koszuliński]

Masterised ​git:cac827d.