Simplify DocumentOrderedMap

Simplify DocumentOrderedMap

This CL simplifies DocumentOrderedMap's internal state. Rather than keeping a
map separately from the duplicate count, we now keep the duplicate count in the
same map.

This CL is a fixed version of r155823, which was reverted because it was causing
an assertion in clusterfuzz.

R=abarth, adamk
BUG=265009, 271131
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164029

[Inactive, 2013-12-17 15:34:50]

https://codereview.chromium.org/117313002/diff/1/Source/core/dom/DocumentOrde...
File Source/core/dom/DocumentOrderedMap.cpp (right):

https://codereview.chromium.org/117313002/diff/1/Source/core/dom/DocumentOrde...
Source/core/dom/DocumentOrderedMap.cpp:91: if (it == m_map.end())
This is the fix for the clusterfuzz crash. We need to keep handling the case
where the key is not in the map.

[esprehn, 2013-12-17 19:05:37]

lgtm (This just looks like a subset of the patch you posted that I've been
reviewing)

[commit-bot: I haz the power, 2013-12-17 19:09:04]

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

[Inactive, 2013-12-17 19:10:13]

On 2013/12/17 19:05:37, esprehn wrote:
> lgtm (This just looks like a subset of the patch you posted that I've been
> reviewing)

Yes, it is.
I split the patch in hope of facilitating review since this is simple
refactoring and the other is mostly about optimizing selector queries.

[commit-bot: I haz the power, 2013-12-17 19:15:40]

Change committed as 164029