[IE8] Red tests on major

[Piotr Jasiun]

2 red tests on major on IE8:

​http://tests.ckeditor.dev:1030/tests/plugins/widget/nestedwidgets

pasted widget is focused

​http://tests.ckeditor.dev:1030/tests/plugins/widget/widgetsintegration

data.focusInited was passed

[Artur Delura]

With reference to ticket:12008:

Very likely we'll not support nested widgets in IE8 for some time.

I'm gonna ignore first test on IE8 ]:->

[Artur Delura]

Changes in ​branch:t/12793. Problem was caused by wrongly set property processedWidgetOnly in IE8.

[Piotrek Koszuliński]

Could you elaborate more about the second problem on IE8? Mainly - what's inside evt.data.dataValue on IE8 and what's on other browsers.

[Artur Delura]

Thats a good question. ​this function call return HTML with new line character at the very beginning. And this value becomes later evt.data.dataValue.

[Piotrek Koszuliński]

So isn't a simple trim() an easier solution?

[Artur Delura]

This value might be not okay as an output from getOuterHtml. Nevethless this value is valid as an input in processDataForInsertion function so should be handled properly.

[Piotrek Koszuliński]

OK, the solution makes sense. We need to improve naming and tests though.

1. Write a test in editable/insertion.js that when insertHtml wrapped content. Check that in #toHtml listener the evt.data.wrappedForWhitespaces is set.
2. Rename options.wrappedForWhitespaces to options.wrappedData. The documentation should say that the insertHtml() function sets this option to true when it wraps content with spans to preserve spaces. Also, you need to document evt.data.wrappedData in the #toHtml event.
3. In editable.js reuse the protect variable instead of creating new.

[Piotrek Koszuliński]

Fixed on major with ​git:9980296.

Perfect tests run:

3928 passed / 0 failed in 10m 50s 695ms - 100%