[Safari] Failing test in core/dom/element and plugins/font due to vendor issue

[Tomasz Jakut]

1 test in this suite is failing:

tests/core/dom/element/element test_isIdentical2
different styles formatting
Expected: true (boolean)
Actual:   false (boolean)

It seems that CKEDITOR.tools.normalizeCssText is returning different result just for Safari.

Safari version: 10.0.1

[Tomasz Jakut]

Ignored in ​git:c0e4dc3.

[kkrzton]

Based on the linked commit, four tests were ignored:

tests/core/dom/element/element.js: test_isIdentical2
tests/plugins/font/font.js: 'test apply font size over another font size (collapsed selection in empty span)'
tests/plugins/embedbase/undo.js: 'test undo and redo after creation';'test undo and redo after creation and edition'

The reason why 2 first tests are failing is visible in this codepen - ​http://codepen.io/f1ames/pen/mRdWbQ. Looks like a native bug, I'll report it soon. It can be workaround using

thisEl.$.setAttribute( 'style', CKEDITOR.tools.normalizeCssText( thisEl.$.style.cssText ) )

instead of

thisEl.$.style.cssText = CKEDITOR.tools.normalizeCssText( thisEl.$.style.cssText )

in ​https://github.com/ckeditor/ckeditor-dev/blob/master/core/dom/element.js#L942. Not sure if it is reasonable, it will be best to wait a little for a vendor answer.

The next 2 failing tests are still under investigation.

[kkrzton]

Reported this issue to the Safari team (​https://bugreport.apple.com/). It seems there is no direct link to the issue so I cannot post it here. The issue id is 29861252. Waiting for the vendor answer.

[kkrzton]

Created separate ticket for tests/plugins/embedbase/undo as the cause of failing tests is different: #16772 -> #14744.

[Marek Lewandowski]

You could report it directly to webkit, it's public repository, and this is the place where bug reports for safari in the end go.

[kkrzton]

Makes sense, I will report it there too.

[kkrzton]

Reported to WebKit bug tracker: ​https://bugs.webkit.org/show_bug.cgi?id=166716