Only swap positions if both characters actually are absolute positioned

Only swap positions if both characters actually are absolute positioned

When re-ordering inline boxes within a "line", character-based absolute
positioning attributes are re-ordered to match. However, when attempting
to re-order/swap a inline box that does have a absolute positioning
attribute (and is of length 1) with one that doesn't, the positioning
attribute is copied rather than swapped - leading to additional chunks
being created and consequently additional adjustments being performed.
However, swapping is not needed/required in this case, since the attribute
is already in the correct (logical) position.
To mitigate this, only perform the attribute swap if both characters
(and consequently inline boxes) have attributes in the first place.

BUG=343486
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=167777

[Stephen Chennney, 2014-02-24 16:42:54]

The test doesn't seem to be passing. I'm not sure if that's some artifact of the
try bots, but please verify before I l g t m.

The code change is fine.

[fs, 2014-02-24 17:10:51]

On 2014/02/24 16:42:54, Stephen Chenney wrote:
> The test doesn't seem to be passing. I'm not sure if that's some artifact of
the
> try bots, but please verify before I l g t m.

I've run LayoutTests/svg/ locally without any issues. (And, yes, all the fails
are due to the 'presubmit' step failing and the 'compile' step being skipped
because of that...)

[Stephen Chennney, 2014-02-24 19:18:09]

lgtm

[fs, 2014-02-25 09:24:11]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2014-02-25 09:24:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fs@opera.com/177263003/1

[commit-bot: I haz the power, 2014-02-25 13:28:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fs@opera.com/177263003/1

[commit-bot: I haz the power, 2014-02-25 15:04:49]

Change committed as 167777