Hoist cue "deactivation" out of TextTrackCueList::remove
Hoist the calls to setIsActive and removeDisplayTree into the callers.
This brings us closer to avoiding side-effects in fairly low-level code.
In TextTrackCueList::updateCueIndex open-code the remove()-call and
eliminate the redundant call to contains(). Also eliminate contains()
itself because it no longer has any uses.
BUG=321654
Review URL: https://codereview.chromium.org/941983003
git-svn-id: svn://svn.chromium.org/blink/trunk@190574 bbb929c8-8fbe-4397-9dbb-9b2b20218538
3 files changed