Closed
Bug 1465372
Opened 6 years ago
Closed 6 years ago
remove reference to old winxp in manifest file
Categories
(Core :: Disability Access APIs, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: jmaher, Assigned: apavel)
References
Details
(Whiteboard: [manifest_cleanup] )
Attachments
(2 files, 2 obsolete files)
2.51 KB,
patch
|
jmaher
:
review-
|
Details | Diff | Splinter Review |
595 bytes,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
there are 4 instances in this file: https://searchfox.org/mozilla-central/source/accessible/tests/browser/browser.ini#14 and 1 instance in this file: https://searchfox.org/mozilla-central/source/accessible/tests/mochitest/jsat/a11y.ini#10
Assignee | ||
Comment 1•6 years ago
|
||
Assignee: nobody → apavel
Attachment #8982138 -
Flags: review?(jmaher)
Assignee | ||
Comment 2•6 years ago
|
||
Attachment #8982139 -
Flags: review?(jmaher)
Reporter | ||
Comment 3•6 years ago
|
||
Comment on attachment 8982138 [details] [diff] [review] removed old win references from browser.ini Review of attachment 8982138 [details] [diff] [review]: ----------------------------------------------------------------- ::: accessible/tests/browser/browser.ini @@ +11,4 @@ > [browser_shutdown_multi_acc_reference_doc.js] > [browser_shutdown_multi_reference.js] > [browser_shutdown_parent_own_reference.js] > +skip-if = !e10s || os == 'win' # e10s specific test for a11y start/shutdown between parent and content. this is skipping now on win7/win10, just leave the !e10s
Attachment #8982138 -
Flags: review?(jmaher) → review-
Reporter | ||
Comment 4•6 years ago
|
||
Comment on attachment 8982139 [details] [diff] [review] removed old win references from a11y.ini Review of attachment 8982139 [details] [diff] [review]: ----------------------------------------------------------------- ::: accessible/tests/mochitest/jsat/a11y.ini @@ +7,4 @@ > doc_content_text.html > !/accessible/tests/mochitest/*.js > !/accessible/tests/mochitest/moz.png > +skip-if = os == 'win' remove the whole line here so we can run on win7/win10
Attachment #8982139 -
Flags: review?(jmaher) → review-
Assignee | ||
Comment 5•6 years ago
|
||
Attachment #8982138 -
Attachment is obsolete: true
Attachment #8982139 -
Attachment is obsolete: true
Attachment #8982188 -
Flags: review?(jmaher)
Assignee | ||
Comment 6•6 years ago
|
||
Attachment #8982189 -
Flags: review?(jmaher)
Reporter | ||
Comment 7•6 years ago
|
||
Comment on attachment 8982188 [details] [diff] [review] updated patch_browser.ini Review of attachment 8982188 [details] [diff] [review]: ----------------------------------------------------------------- the other reference to removing winXP are great. ::: accessible/tests/browser/browser.ini @@ +15,3 @@ > [browser_shutdown_pref.js] > [browser_shutdown_proxy_acc_reference.js] > +skip-if = !e10s # e10s specific test for a11y start/shutdown between parent and content. we need to keep the os == 'win' here @@ +18,2 @@ > [browser_shutdown_proxy_doc_acc_reference.js] > +skip-if = !e10s # e10s specific test for a11y start/shutdown between parent and content. and keep win here as well @@ +20,2 @@ > [browser_shutdown_multi_proxy_acc_reference_doc.js] > +skip-if = !e10s # e10s specific test for a11y start/shutdown between parent and content. and keep win here as well @@ +22,2 @@ > [browser_shutdown_multi_proxy_acc_reference_obj.js] > +skip-if = !e10s # e10s specific test for a11y start/shutdown between parent and content. and keep win here as well
Attachment #8982188 -
Flags: review?(jmaher) → review-
Reporter | ||
Comment 8•6 years ago
|
||
Comment on attachment 8982189 [details] [diff] [review] updated patch_a11y.ini Review of attachment 8982189 [details] [diff] [review]: ----------------------------------------------------------------- great!
Attachment #8982189 -
Flags: review?(jmaher) → review+
Updated•6 years ago
|
Keywords: checkin-needed
Whiteboard: [manifest_cleanup]
Pushed by ebalazs@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c7989875fbb6 remove reference to old winxp in manifest file r=jmaher
Keywords: checkin-needed
Comment 10•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c7989875fbb6
Status: NEW → RESOLVED
Closed: 6 years 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.
Description
•