commit 80ef82107b8f9c2cd1bc707d86bc0fd8bc8cd338
author fs <fs@opera.com> Mon May 16 09:05:07 2016
committer Commit bot <commit-bot@chromium.org> Mon May 16 09:06:43 2016
tree d8dc21d2d09f42891fd2cc3877941d6b89efc4a1
parent b1880493383ee11122cc9be9fe0287855be6a75d

Clean up HTMLTrackElement.kind invalid/missing value default handling

Get rid of the isValidKind(...) and invalidValueDefaultKind() virtual
methods on TrackBase and do any required checking "up front" instead
as required. This should present less surprises and work in a less
side-effectful way.

Also start setting the 'kind' directly in the constructor rather than
invoking setKind() in (all) the constructor body (bodies).
Drop some redundant parenthesis and fix some obviously "wrong" names
in TextTrack.cpp.

BUG=608772

Review-Url: https://codereview.chromium.org/1973343002
Cr-Commit-Position: refs/heads/master@{#393812}