empty tag should be removed on inline-style format

[yiminghe]

example:

<span style='font-size:28px;'>^测试$</span>

^{indicates start selection position and $ indicates end selection position.}

when apply inline-style format such as change to 48px , now the code generated is (use firebug pls)

{{{
<span style='font-size:48px;'>测试</span>
<span style='font-size:28px;'></span>
}}}

it should be : (no empty style tag)

<span style='font-size:48px;'>测试</span>

i fix it :

in 3.4.2/_source/plugins/styles/plugin.js line 449 ,add following code :

 var validNextSilbing = includedNode.getNext(function(node) {
                                return !node.getAttribute || !node.getAttribute("_cke_bookmark");
                            });
                            while ( !validNextSilbing
                                && ( parentNode = includedNode.getParent(), dtd[ parentNode.getName() ] )
                                && ( parentNode.getPosition( firstNode ) | CKEDITOR.POSITION_FOLLOWING | CKEDITOR.POSITION_IDENTICAL | CKEDITOR.POSITION_IS_CONTAINED ) == ( CKEDITOR.POSITION_FOLLOWING + CKEDITOR.POSITION_IDENTICAL + CKEDITOR.POSITION_IS_CONTAINED )
                                && ( !def.childRule || def.childRule( parentNode ) ) )
                            {
                                includedNode = parentNode;
                            }

in summary ,we should ignore bookmark and include parent node in style range.

[yiminghe]

incorrect code generated

[yiminghe]

'^{' indicates start selection position and '$' indicates end selection position.}

[Garry Yao]

Yet another bookmark sucker, having no impacts on WYSYWIG, mark as low-pri, though requires a fix, thanks yiminghe.

[Garry Yao]

Fixed with [6275].