Implement evictFrames() to support MSE's coded frame eviction algorithm.

Implement evictFrames() to support MSE's coded frame eviction algorithm.

s/WebKit/Blink/ Implemented WebSourceBufferImpl::evictFrames() to call SourceBuffer::GarbageCollectIfNeeded()

[kjoswiak, 2014-12-09 22:51:07]

Codependent on https://codereview.chromium.org/785343002/

[kjoswiak, 2014-12-09 23:28:22]

kjoswiak@chromium.org changed reviewers:
+ dalecurtis@chromium.org, wolenetz@chromium.org

[philipj_slow, 2014-12-10 09:26:07]

philipj@opera.com changed reviewers:
+ philipj@opera.com

[philipj_slow, 2014-12-10 09:26:08]

https://codereview.chromium.org/789983003/diff/1/media/filters/chunk_demuxer.cc
File media/filters/chunk_demuxer.cc (right):

https://codereview.chromium.org/789983003/diff/1/media/filters/chunk_demuxer....
media/filters/chunk_demuxer.cc:389: for (TextStreamMap::iterator itr =
text_stream_map_.begin();
If C++11 range-based for loops work here, that would be nice.

[philipj_slow, 2014-12-10 09:31:02]

s/WebKit/Blink/ in the title/description. Something more descriptive like
"Implement evictFrames() to support MSE's coded frame eviction algorithm" would
perhaps be better.

[DaleCurtis, 2014-12-10 18:44:04]

dalecurtis@chromium.org changed reviewers:
- dalecurtis@chromium.org

[kjoswiak, 2014-12-10 19:35:28]

https://codereview.chromium.org/789983003/diff/1/media/filters/chunk_demuxer.cc
File media/filters/chunk_demuxer.cc (right):

https://codereview.chromium.org/789983003/diff/1/media/filters/chunk_demuxer....
media/filters/chunk_demuxer.cc:389: for (TextStreamMap::iterator itr =
text_stream_map_.begin();
On 2014/12/10 09:26:08, philipj wrote:
> If C++11 range-based for loops work here, that would be nice.

I did it this way because this is how it's done elsewhere in file (see line 374,
for instance)

[philipj_slow, 2014-12-10 19:39:56]

https://codereview.chromium.org/789983003/diff/1/media/filters/chunk_demuxer.cc
File media/filters/chunk_demuxer.cc (right):

https://codereview.chromium.org/789983003/diff/1/media/filters/chunk_demuxer....
media/filters/chunk_demuxer.cc:389: for (TextStreamMap::iterator itr =
text_stream_map_.begin();
On 2014/12/10 19:35:28, kjoswiak wrote:
> On 2014/12/10 09:26:08, philipj wrote:
> > If C++11 range-based for loops work here, that would be nice.
> 
> I did it this way because this is how it's done elsewhere in file (see line
374,
> for instance)

OK, owners get to decide.

[kjoswiak, 2014-12-11 23:28:26]

Combined with another CL to create https://codereview.chromium.org/794343003/