Enable tests "test_seek-1 ~ test_seek-14" on Fennec.

NEW
Assigned to

Status

()

P3
normal
a year ago
a year ago

People

(Reporter: bechen, Assigned: bechen)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
Priority: -- → P3
(Assignee)

Updated

a year ago
Assignee: nobody → bechen
Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8903040 [details]
Bug 1393368 - Enable tests "test_seek-1 ~ test_seek-14" on Fennec.

https://reviewboard.mozilla.org/r/174820/#review179880

::: commit-message-53fd1:5
(Diff revision 1)
> +Bug 1393368 - Enable tests "test_seek-1 ~ test_seek-14" on Fennec. r=alwu
> +
> +MozReview-Commit-ID: DOKtcmPTe99
> +* * *
> +[mq]: ttt

please remove this line and only keep one MozReview-Commit-ID.

::: dom/media/test/mochitest.ini
(Diff revision 1)
>  [test_seek_negative.html]
> -skip-if = toolkit == 'android' # bug 1295443, bug 1306787, android(bug 1232305)
> +tags=seek
>  [test_seek_nosrc.html]
> +tags=seek
>  [test_seek_out_of_range.html]
> -skip-if = toolkit == 'android' # bug 1299382, android(bug 1232305)

add "tags=seek".

::: dom/media/test/mochitest.ini:1058
(Diff revision 1)
>  [test_seekable1.html]
> -skip-if = toolkit == 'android' # android(bug 1232305)
> +tags=seek
>  [test_seekLies.html]
> +tags=seek
>  skip-if = android_version == '15' || android_version == '17' # android(bug 1232305)
>  [test_seekToNextFrame.html]

should we need to add tag for this one?
eg. tags=seektonextframe seek
Attachment #8903040 - Flags: review?(alwu) → review+
You need to log in before you can comment on or make changes to this bug.