Have DocumentOrderedMap API deal with AtomicString type instead of StringImpl*

Have DocumentOrderedMap API deal with AtomicString type instead of StringImpl*

Have DocumentOrderedMap API deal with AtomicString type instead of StringImpl*
as it would be unsafe to use if the StringImpls were not atomized.

This basically moves the AtomicString::impl() calls from the DocumentOrderedMap
API call sites to the DocumentOrderedMap implementation. This simplifies the
call sites a bit and makes the DocumentOrderedMap code look safer.

R=morrita@chromium.org, adamk@chromium.org

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

[Hajime Morrita, 2014-08-12 00:47:30]

LGTM.

"Look safer" precisely states the situation :-/
(Not blaming you. Just mumbling.)

[Hajime Morrita, 2014-08-12 00:48:07]

The bot redness seems real btw.

[haraken, 2014-08-12 00:51:12]

LGTM

[Inactive, 2014-08-12 00:54:53]

The CQ bit was checked by ch.dumez@samsung.com

[Inactive, 2014-08-12 00:56:46]

The CQ bit was unchecked by ch.dumez@samsung.com

[Inactive, 2014-08-12 00:57:07]

On 2014/08/12 00:48:07, morrita wrote:
> The bot redness seems real btw.

Right, I have just noticed. I'll fix that, thanks.

[commit-bot: I haz the power, 2014-08-12 00:58:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/459203004/1

[Inactive, 2014-08-12 01:16:54]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-08-12 01:17:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/459203004/40001

[commit-bot: I haz the power, 2014-08-12 02:03:25]

Change committed as 180022