871188 - [webvtt] HTMLTrackElement readyState attribute assigning random values

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Marcus Saad (:msaad)]

HTMLTrackElement is assigning random numbers to readyState. Acceptable values are between 0 and 3 included.

Comment 1

[Marcus Saad (:msaad)]

Attachment: Test Case - HTMLTrackElement readyState improver value.

Comment 2

[Marcus Saad (:msaad)]

We should come to a common sense of which value to set by default. I personally think that 0 is the best value, so that it can be updated as needed.

//Possible values for readyState
  enum {
    NONE = 0U,
    LOADING = 1U,
    LOADED = 2U,
    ERROR = 3U
  };

Comment 3

[Caitlin Potter (:caitp)]

I haven't been able to reproduce this with rev 134626:86413e921d5d

The test is passing consistently. Was this fixed somewhere else?

Comment 4

[Rick Eyre (:reyre)]

I think it was fixed while bug 833385 was being reviewed. I'm in the middle of updating 833386 to include tests for this as well so I think we should be covered on it.

Comment 5

[Caitlin Potter (:caitp)]

```
HTMLTrackElement::HTMLTrackElement(already_AddRefed<nsINodeInfo> aNodeInfo)
  : nsGenericHTMLElement(aNodeInfo)
  , mReadyState(NONE)
```

mReadyState is being initiallized now, as of some point. So it should never be random, anyways. WFM, resolved