remove old osx 10.6 references in manifest files

RESOLVED FIXED in Firefox 62

Status

()

P5
normal
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: jmaher, Assigned: apavel)

Tracking

unspecified
mozilla62
Points:
---

Firefox Tracking Flags

(firefox62 fixed)

Details

(Whiteboard: [manifest_cleanup])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

10 months ago
we run our automation on osx 10.10, we can remove old references to older osx versions.

there are 4 references in:
https://searchfox.org/mozilla-central/source/testing/web-platform/meta/html/semantics/embedded-content/media-elements/event_timeupdate.html.ini#3

lets remove those

Updated

10 months ago
Priority: -- → P5
(Assignee)

Comment 1

10 months ago
Posted patch removed OSX references (obsolete) — Splinter Review
Assignee: nobody → apavel
Attachment #8980099 - Flags: review?(jmaher)
(Reporter)

Comment 2

10 months ago
Comment on attachment 8980099 [details] [diff] [review]
removed OSX references

Review of attachment 8980099 [details] [diff] [review]:
-----------------------------------------------------------------

::: testing/web-platform/meta/html/semantics/embedded-content/media-elements/event_timeupdate.html.ini
@@ +3,2 @@
>      if debug and not e10s and (os == "linux") and (version == "Ubuntu 16.04") and (processor == "x86_64") and (bits == 64): CRASH
>    [setting src attribute on a sufficiently long autoplay audio should trigger timeupdate event]

since there are no conditions under this subtest, please remove the subtest:
[setting src attribute on a sufficiently long autoplay audio should trigger timeupdate event]
Attachment #8980099 - Flags: review?(jmaher) → review-
(Reporter)

Comment 3

10 months ago
Noemi, would you like to pick up the one last change here?
Flags: needinfo?(nerli)
Flags: needinfo?(nerli) → needinfo?(jmaher)
Attachment #8980231 - Flags: review?(jmaher)
(Reporter)

Updated

10 months ago
Attachment #8980099 - Attachment is obsolete: true
Flags: needinfo?(jmaher)
(Reporter)

Updated

10 months ago
Attachment #8980231 - Flags: review?(jmaher) → review+
(Reporter)

Updated

10 months ago
Keywords: checkin-needed

Comment 5

10 months ago
Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dabad83b031e
Removed OSX references. r=jmaher
Keywords: checkin-needed

Comment 6

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/dabad83b031e
Status: NEW → RESOLVED
Last Resolved: 10 months ago
status-firefox62: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.