[Safari] format combo don't hide when editor blur

[Garry Yao]

Reproducing Procedures

1. Open the 'replace by class' sample page in Safari;
2. Open the 'Format' Combo, then click outside editor chrome;
   • Actual Result: combo panel won't able to hide from now on;

[Frederico Caldeira Knabben]

It's possible to close the combo by clicking on it again, at least.

[Garry Yao]

The problem happens to 'Format' Combo specifically, modify summary.

[Garry Yao]

Tidied description a bit.

[Garry Yao]

Even worse, 'Format' combo will not open at all on first click (without focus editor).

[Minh Nguyen]

With Safari and Chorme the 'elementPath' allways miss 'lastElement' object

[Alfonso Martínez de Lizarrondo]

First of all, I've tried to reproduce the bug with Safari on windows but it behaved correctly.

The patch addresses the current ticket, but why isn't lastElement available in elementPath? other code might face this same problem so it might be worth to look at the problem at its root.

Also, I understand that this problem wasn't generating an error in the console, so if there's a try catch in the code that calls this, maybe it should be disabled (at least in _source mode) to find more easily this problems, or use the catch an ignore only expected failures.

[Minh Nguyen]

When we click directly in 'Format' Combo (not focus editor), we will have an error. This bug happens (in Chorme and Safari) because the line code selection.getStartElement() ​http://dev.fckeditor.net/browser/CKEditor/trunk/_source/plugins/stylescombo/plugin.js#L157 returns null. This also happens in IE but we have already fixed by setting editor focus before getting selection.

[Minh Nguyen]

Fixed with [5318].