Check for duplicate id strings in ChunkDemuxer::AddId()

Check for duplicate id strings in ChunkDemuxer::AddId()

WebKit's MediaSource object keeps a private member variable, int m_nextSourceBufferId, for adding source ids since ids are no longer part of the spec.  It is possible that this int could wrap around if several source buffers are added.  To ensure this doesn't cause problems, ChunkDemuxer::AddId() should check the id string to make sure it is not a duplicate and, if so, throw a ReachedIdLimit error.

BUG=140861
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150459

[annacc, 2012-08-06 19:36:32]

PTAL!

[acolwell GONE FROM CHROMIUM, 2012-08-06 20:05:31]

LGTM

[commit-bot: I haz the power, 2012-08-06 20:10:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/annacc@chromium.org/10830185/1

[commit-bot: I haz the power, 2012-08-06 20:38:06]

Try job failure for 10830185-1 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2012-08-06 22:45:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/annacc@chromium.org/10830185/1

[commit-bot: I haz the power, 2012-08-06 23:14:50]

Try job failure for 10830185-1 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2012-08-07 04:28:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/annacc@chromium.org/10830185/1

[commit-bot: I haz the power, 2012-08-07 07:10:31]

Try job failure for 10830185-1 (retry) on win_rel for step "net_unittests"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-08-07 22:15:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/annacc@chromium.org/10830185/1

[commit-bot: I haz the power, 2012-08-07 23:53:37]

Change committed as 150459