Clean up around text track display state updates
Several minor oddities are changed to be hopefully less odd:
* CueTimeline need not check activeSetChanged due to an earlier
|if (!activeSetChanged) return|.
* Move one-time setup of m_displayTree and m_cueBackgroundBox to where
they are created.
* ASSERT the preconditions guaranteed in VTTCue::updateDisplay() and
getDisplayTree(), guaranteed by TextTrackContainer::updateDisplay.
* Never remove and m_cueBackgroundBox from m_displayTree, just leave it
there and assert that it is so.
* Document where m_cueBackgroundBox is actually populated.
* Remove a bunch of spec comments where the code doesn't resemble the
shape of the spec and FIXMEs that aren't anywhere close to where they
would actually be fixed. A link to http://wkb.ug/79916 remains in
VTTCueBox::applyCSSProperties().
BUG=322434
Review URL: https://codereview.chromium.org/1022523002
git-svn-id: svn://svn.chromium.org/blink/trunk@192113 bbb929c8-8fbe-4397-9dbb-9b2b20218538
3 files changed
