Hoist checks for 'disabled' tracks out of textTrackAddCue{,s}

Hoist checks for 'disabled' tracks out of textTrackAddCue{,s}

In most cases the mode has already been checked by calling code, so the
check is redundant.
This is done in preparation of factoring the text track rendering code
out of HTMLMediaElement.

BUG=321654
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189944

[fs, 2015-02-09 16:22:58]

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

[philipj_slow, 2015-02-10 04:14:08]

The bots look sad, but I'm very excited about "factoring the text track
rendering code out of HTMLMediaElement" so hopefully it's easy to fix.

Typo "inf" in description.

[fs, 2015-02-10 11:39:45]

On 2015/02/10 04:14:08, philipj_UTC7 wrote:
> The bots look sad

Apparently I had missed one callsite - hopefully the bots are happier now...

> ..., but I'm very excited about "factoring the text track
> rendering code out of HTMLMediaElement" so hopefully it's easy to fix.

It has been fairly pain-free so far (a few needles, but nothing horrible IMHO.)

[philipj_slow, 2015-02-11 01:46:53]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-02-11 01:46:54]

lgtm

[commit-bot: I haz the power, 2015-02-11 01:47:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/910843002/20001

[commit-bot: I haz the power, 2015-02-11 01:50:26]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189944