Closed Bug 1310297 Opened 8 years ago Closed 8 years ago

Remove test annotations using b2g, mulet or gonk

Categories

(Core :: General, defect)

Product:
Core
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla52
Tracking Flags:
Tracking Status
firefox52 --- fixed

People

(Reporter: aryx, Assigned: aryx)

References

(Blocks 1 open bug)

Details

Attachments

(71 files, 9 obsolete files)

Bug 1310297 - Remove test annotations using b2g, mulet or gonk: browser.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: caps.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: chrome.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: devtools.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: docshell.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/animation.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/apps.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/archivereader.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/base.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/bindings.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/broadcastchannel.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/browser-element.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/cache.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/canvas.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/console.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/crypto.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/devicestorage.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/encoding.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/events.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/html.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/imptests.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/indexedDB.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/inputmethod.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/ipc.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/json.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/jsurl.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/media.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/network.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/notification.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/permission.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/plugins.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/power.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/presentation.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/promise.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/push.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/security.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/svg.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/system.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/tests.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/url.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/workers.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xbl.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xhr.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xml.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xul.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: editor.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: embedding.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: extensions.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: gfx.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: image.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: intl.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: ipc.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: js.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/base.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/forms.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/generic.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/inspector.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/style.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/xul.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: modules.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: netwerk.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: parser.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: rdf.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: security.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: services.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: storage.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: toolkit.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: uriloader.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: widget.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpcom.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpfe.
58 bytes, text/x-review-board-request
RyanVM
: review+
Details
B2G related code gets removed from mozilla-central, see bug 1306391.
Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=93fa979b61a0f80ee2dbded31f6566a8a3a4222e&selectedJob=29177611 Only touched the test manifests. Removals of tests and e.g. interfaces implemented for them belong into other bugs, especially if it has to be evaluated if code should be kept because its use in standard interfaces can be foreseen for the upcoming months.
Comment on attachment 8801230 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: browser. https://reviewboard.mozilla.org/r/85944/#review84632
Attachment #8801230 - Flags: review?(ryanvm) → review+
Comment on attachment 8801231 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: caps. https://reviewboard.mozilla.org/r/85946/#review84634
Attachment #8801231 - Flags: review?(ryanvm) → review+
Comment on attachment 8801232 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: chrome. https://reviewboard.mozilla.org/r/85948/#review84636
Attachment #8801232 - Flags: review?(ryanvm) → review+
Comment on attachment 8801233 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: devtools. https://reviewboard.mozilla.org/r/85950/#review84638 ::: devtools/client/performance/test/unit/xpcshell.ini:13 (Diff revision 1) > [test_frame-utils-01.js] > [test_frame-utils-02.js] > [test_marker-blueprint.js] > [test_marker-utils.js] > [test_profiler-categories.js] > [test_jit-graph-data.js] Can you please remove the trailing whitespace from this line while you're here? ::: devtools/server/tests/unit/xpcshell.ini:137 (Diff revision 1) > [test_conditional_breakpoint-01.js] > [test_conditional_breakpoint-02.js] > [test_conditional_breakpoint-03.js] > [test_eventlooplag_actor.js] > -skip-if = toolkit != "gonk" > +skip-if = true > +reason = only ran on B2G Please ping Paul Rouget about whether this test can be removed or not. Looks like bug 962511 where this landed was only ever intended for B2G to begin with. It also may be that there's other code that can go besides just the test at this point. Certainly fine for a follow-up, though. ::: devtools/shared/security/tests/unit/xpcshell.ini:13 (Diff revision 1) > -# Failures on B2G emulator debug, B2G emulator-x86-kk > -# See bug 1234972 and bug 1199472 > -skip-if = toolkit == 'gonk' && (debug || android_version > '15') > [test_oob_cert_auth.js] > -# Failures on B2G emulator debug, B2G emulator-x86-kk and Android opt > -# See bug 1141544, bug 1163052, bug 1166032 and bug 1241831 > +# Failures on Android opt, see bug 1141544 > +skip-if = (toolkit == 'android' && !debug) Do you mind seeing if this still fails on Android opt or not? Maybe we can just re-enable it across the board and close out bug 1141544. I've also ni? myself in that bug to check otherwise, but if you beat me to it, great :)
Attachment #8801233 - Flags: review?(ryanvm) → review+
Comment on attachment 8801234 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: docshell. https://reviewboard.mozilla.org/r/85952/#review84646 ::: docshell/test/chrome/chrome.ini:71 (Diff revision 1) > [test_bug449778.xul] > [test_bug449780.xul] > [test_bug453650.xul] > [test_bug454235.xul] > # bug 684176 > skip-if = toolkit == "gtk2" Another possible cleanup bug we could file: getting rid of GTK2 annotations now that we're on GTK3 across the board AFAIK. ::: docshell/test/iframesandbox/mochitest.ini:24 (Diff revision 1) > [test_sibling_navigation_by_location.html] > tags = openwindow > [test_top_navigation_by_location_exotic.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || android_version == '18' #bug 948948, NS_ERROR_FAILURE from nsWindowWatcher::GetPrompt > +skip-if = android_version == '18' > [test_top_navigation_by_location.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || android_version == '18' #bug 948948, NS_ERROR_FAILURE from nsWindowWatcher::GetPrompt > +skip-if = android_version == '18' Please leave the bug number here as I'm pretty sure it's applicable to Android as well (window.open doesn't work there either). ::: docshell/test/mochitest.ini:62 (Diff revision 1) > support-files = bug530396-noref.sjs bug530396-subframe.html > [test_bug540462.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') #Bug 931116, b2g desktop specific, initial triage > [test_bug551225.html] > [test_bug570341.html] > -skip-if = (toolkit == 'gonk' && debug) || (android_version == '18' && debug) #debug-only failure, android bug 1040769 > +skip-if = (android_version == '18' && debug) #debug-only failure, bug 1040769 Might be worth seeing if this still fails since Android 4.3 tests migrated to Taskcluster. ::: docshell/test/mochitest.ini:66 (Diff revision 1) > -skip-if = (toolkit == 'gonk' && debug) || (android_version == '18' && debug) #debug-only failure, android bug 1040769 > +skip-if = (android_version == '18' && debug) #debug-only failure, bug 1040769 > [test_bug580069.html] > [test_bug590573.html] > -skip-if = buildapp == 'b2g' #bug 823022 # b2g(queryinterfaces into webnavigation, might suffer from something similar as bug 823022) b2g-debug(queryinterfaces into webnavigation, might suffer from something similar as bug 823022) b2g-desktop(queryinterfaces into webnavigation, might suffer from something similar as bug 823022) > [test_bug598895.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || toolkit == 'android' #Bug 931116, b2g desktop specific, initial triage > +skip-if = toolkit == 'android' Older-than time test disabling, might be worth seeing if it still fails or not on Android. ::: docshell/test/mochitest.ini:69 (Diff revision 1) > -skip-if = buildapp == 'b2g' #bug 823022 # b2g(queryinterfaces into webnavigation, might suffer from something similar as bug 823022) b2g-debug(queryinterfaces into webnavigation, might suffer from something similar as bug 823022) b2g-desktop(queryinterfaces into webnavigation, might suffer from something similar as bug 823022) > [test_bug598895.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || toolkit == 'android' #Bug 931116, b2g desktop specific, initial triage > +skip-if = toolkit == 'android' > [test_bug634834.html] > [test_bug637644.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || toolkit == 'android' #Bug 931116, b2g desktop specific, initial triage > +skip-if = toolkit == 'android' Same as previous comment. ::: docshell/test/mochitest.ini:76 (Diff revision 1) > [test_bug653741.html] > [test_bug660404.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') #Bug 931116, b2g desktop specific, initial triage > [test_bug662170.html] > [test_bug668513.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || toolkit == 'android' #Bug 931116, b2g desktop specific, initial triage > +skip-if = toolkit == 'android' And again. ::: docshell/test/mochitest.ini:90 (Diff revision 1) > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') #Bug 931116, b2g desktop specific, initial triage > [test_bug797909.html] > [test_bug1045096.html] > [test_bug1186774.html] > [test_framedhistoryframes.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || toolkit == 'android' #Bug 931116, b2g desktop specific, initial triage, and also bug 784321 > +skip-if = toolkit == 'android' # bug 784321 And again (test disabling going back to 2012, ouch). ::: docshell/test/mochitest.ini:95 (Diff revision 1) > [test_pushState_after_document_open.html] > [test_windowedhistoryframes.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') #Bug 931116, b2g desktop specific, initial triage > [test_bug1121701.html] > -skip-if = (buildapp == 'b2g' || buildapp == 'mulet') > [test_forceinheritprincipal_overrule_owner.html] While you're here, can you please alphabetize these last few tests hanging off the end of the list? ::: docshell/test/navigation/mochitest.ini:41 (Diff revision 1) > file_bug1300461_redirect.html > file_bug1300461_redirect.html^headers^ > file_bug1300461_back.html > > [test_bug13871.html] > -skip-if = buildapp == 'b2g' || buildapp == 'mulet' || toolkit == 'android' #RANDOM # Bug 1136180 disabled on B2G Desktop and Mulet for intermittent failures > +skip-if = toolkit == 'android' #RANDOM Some of these poorly-annotated Android skips might be worth checking on Try too.
Attachment #8801234 - Flags: review?(ryanvm) → review+
Comment on attachment 8801235 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/animation. https://reviewboard.mozilla.org/r/85954/#review84652
Attachment #8801235 - Flags: review?(ryanvm) → review+
Comment on attachment 8801236 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/apps. https://reviewboard.mozilla.org/r/85956/#review84654 ::: dom/apps/tests/chrome.ini:2 (Diff revision 1) > [DEFAULT] > -skip-if = buildapp != 'mulet' > +skip-if = true ### Bug 1255339: blacklist because no more mozApps I'm wondering if these should just be removed at this point. Probably worth pinging gerard-majax about it.
Comment on attachment 8801237 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/archivereader. https://reviewboard.mozilla.org/r/85958/#review84656
Attachment #8801237 - Flags: review?(ryanvm) → review+
Comment on attachment 8801238 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/base. https://reviewboard.mozilla.org/r/85960/#review84660 ::: dom/base/test/chrome.ini:28 (Diff revision 1) > -skip-if = buildapp == 'mulet' > [test_bug945152.html] > [test_bug1008126.html] > [test_sandboxed_blob_uri.html] > [test_websocket_frame.html] > [test_mozbrowser_apis_allowed.html] Please alphabetize these while you're here. ::: dom/base/test/mochitest.ini:279 (Diff revision 1) > [test_bug51034.html] > [test_bug116083.html] > subsuite = clipboard > [test_bug166235.html] > subsuite = clipboard > -skip-if = buildapp == 'b2g' || toolkit == 'android' # b2g(clipboard undefined) b2g-debug(clipboard undefined) b2g-desktop(clipboard undefined) > +skip-if = toolkit == 'android' More craptasically-commented Android disablings worth checking on :( ::: dom/base/test/mochitest.ini:305 (Diff revision 1) > [test_bug333673.html] > [test_bug337631.html] > [test_bug338541.xhtml] > [test_bug338583.html] > -# b2g(https not working, bug 907770) b2g-debug(https not working, bug 907770) b2g-desktop(43 total - bug 901343, specialpowers.wrap issue createsystemxhr) > # e10s - bug 970589, bug 1091934 This line isn't needed anymore, is it? ::: dom/base/test/mochitest.ini:417 (Diff revision 1) > [test_bug459424.html] > [test_bug461555.html] > [test_bug461735.html] > [test_bug465767.html] > [test_bug466080.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' # b2g(bug 901356, also fails on android) b2g-debug(bug 901356, also fails on android) b2g-desktop(bug 901356, also fails on android) > +skip-if = toolkit == 'android' # bug 901356 Another one for the "I wonder if this even fails on Android anymore" list. Especially on a WONTFIXed bug. ::: dom/base/test/mochitest.ini:618 (Diff revision 1) > [test_clearTimeoutIntervalNoArg.html] > [test_constructor-assignment.html] > [test_constructor.html] > [test_copyimage.html] > subsuite = clipboard > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') || (toolkit != 'cocoa' && toolkit != 'gonk' && toolkit != 'gtk2' && toolkit != 'gtk3' && toolkit != 'windows') #b2g-desktop(Bug 931116, b2g desktop specific, initial triage) > +skip-if = (toolkit != 'cocoa' && toolkit != 'gonk' && toolkit != 'gtk2' && toolkit != 'gtk3' && toolkit != 'windows') Couldn't this just be more clearly written as just skip-if = toolkit == 'android' at this point? Plus there's still a gonk reference in that line otherwise. ::: dom/base/test/mochitest.ini:732 (Diff revision 1) > [test_orientation_frame.html] > -skip-if = toolkit != 'gonk' > +skip-if = true # Test only ran on Gonk > [test_orientation_frame_lock.html] > -skip-if = toolkit != 'gonk' > +skip-if = true # Test only ran on Gonk > [test_orientation_sandbox_no_lock.html] > -skip-if = toolkit != 'gonk' > +skip-if = true # Test only ran on Gonk Please file a follow-up bug and ni? wchen to see whether these are needed anymore or not. You can also change the comment to point to that bug instead then. ::: dom/base/test/mochitest.ini:794 (Diff revision 1) > [test_websocket3.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' # TC: Bug 1144079 - Re-enable Mulet mochitests and reftests taskcluster-specific disables. > +skip-if = toolkit == 'android' > [test_websocket4.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' # TC: Bug 1144079 - Re-enable Mulet mochitests and reftests taskcluster-specific disables. > +skip-if = toolkit == 'android' > [test_websocket5.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' # TC: Bug 1144079 - Re-enable Mulet mochitests and reftests taskcluster-specific disables. > +skip-if = toolkit == 'android' Would love to know if these work on Android at this point. ::: dom/base/test/mochitest.ini:806 (Diff revision 1) > [test_window_enumeration.html] > [test_window_extensible.html] > [test_window_indexing.html] > [test_window_named_frame_enumeration.html] > [test_window_orientation.html] > -skip-if = toolkit != 'gonk' > +skip-if = true # Test only ran on Gonk Same as test_orientation* ::: dom/base/test/websocket_hybi/mochitest.ini:13 (Diff revision 1) > [test_receive-blob.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' > +skip-if = toolkit == 'android' > [test_send-arraybuffer.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' > +skip-if = toolkit == 'android' > [test_send-blob.html] > -skip-if = buildapp == 'b2g' || toolkit == 'android' > +skip-if = toolkit == 'android' More older-than-time disablings on Android :(
Attachment #8801238 - Flags: review?(ryanvm) → review+
Comment on attachment 8801239 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/bindings. https://reviewboard.mozilla.org/r/85962/#review84666
Attachment #8801239 - Flags: review?(ryanvm) → review+
Comment on attachment 8801240 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/broadcastchannel. https://reviewboard.mozilla.org/r/85964/#review84668
Attachment #8801240 - Flags: review?(ryanvm) → review+
Comment on attachment 8801241 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/browser-element. https://reviewboard.mozilla.org/r/85966/#review84670
Attachment #8801241 - Flags: review?(ryanvm) → review+
Comment on attachment 8801236 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/apps. https://reviewboard.mozilla.org/r/85956/#review84672
Attachment #8801236 - Flags: review?(ryanvm) → review+
Comment on attachment 8801242 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/cache. https://reviewboard.mozilla.org/r/85968/#review84678 ::: dom/browser-element/mochitest/mochitest-oop.ini:6 (Diff revision 1) > [DEFAULT] > # Both the "inproc" and "oop" versions of OpenMixedProcess open remote frames, > # so we don't run that test on platforms which don't support OOP tests. > # OOP tests don't work on native-fennec (bug 774939). > # Bug 960345 - Disabled on OSX debug for frequent crashes. > -skip-if = os == "android" || (toolkit == "cocoa" && debug) || buildapp == 'mulet' || (buildapp == 'b2g' && (toolkit != 'gonk' || debug)) || e10s > +skip-if = os == "android" || (toolkit == "cocoa" && debug) || e10s I messed up over in bug 960345. I'm going to check soon if these can be re-enabled on OSX debug and will do so if they succeed on Try. ::: dom/browser-element/mochitest/mochitest-oop.ini:89 (Diff revision 1) > [test_browserElement_oop_ReloadPostRequest.html] > disabled = Disabling some OOP tests for WebIDL scope changes > [test_browserElement_oop_RemoveBrowserElement.html] > [test_browserElement_oop_ScrollEvent.html] > [test_browserElement_oop_SecurityChange.html] > -skip-if = toolkit == 'android' || (toolkit == 'gonk' && !debug) #TIMED_OUT, bug 766586 > +skip-if = toolkit == 'android' #TIMED_OUT, bug 766586 Goodness, another one disabled over bug 766586. ::: dom/browser-element/mochitest/mochitest.ini:122 (Diff revision 1) > [test_browserElement_inproc_CopyPaste.html] > subsuite = clipboard > skip-if = (os == "android") # Disabled on Android, see bug 1230421 > [test_browserElement_inproc_DataURI.html] > [test_browserElement_inproc_Download.html] > disabled = bug 1022281 Another one I'd be curious about whether it passes or not at this point. ::: dom/browser-element/mochitest/mochitest.ini:143 (Diff revision 1) > [test_browserElement_inproc_Opensearch.html] > [test_browserElement_inproc_PrivateBrowsing.html] > [test_browserElement_inproc_PromptCheck.html] > [test_browserElement_inproc_PromptConfirm.html] > # Disabled on B2G Emulator because permission cannot be asserted in content process, > # need to fix either bug 1094055 or bug 1020135. Useless comment can go. ::: dom/browser-element/mochitest/mochitest.ini:148 (Diff revision 1) > [test_browserElement_inproc_Proxy.html] > -skip-if = toolkit == 'gonk' || buildapp == 'mulet' > [test_browserElement_inproc_RemoveBrowserElement.html] > [test_browserElement_inproc_ScrollEvent.html] > [test_browserElement_inproc_SecurityChange.html] > -skip-if = toolkit == 'android' || (toolkit == 'gonk' && !debug) # android(TIMED_OUT, bug 766586) androidx86(TIMED_OUT, bug 766586) > +skip-if = toolkit == 'android' # android(TIMED_OUT, bug 766586) androidx86(TIMED_OUT, bug 766586) Wonder if this still fails on Android or not. Either way, please clean up the comment as it's redundantly-written as it stands currently. ::: dom/browser-element/mochitest/mochitest.ini:161 (Diff revision 1) > [test_browserElement_inproc_XFrameOptionsSameOrigin.html] > [test_browserElement_oop_NextPaint.html] > # Disabled due to https://bugzilla.mozilla.org/show_bug.cgi?id=774100 > disabled = temp disabling some OOP tests for WebIDL scope changes > [test_browserElement_inproc_Reload.html] > disabled = bug 774100 The last comments in the referenced bug suggest we could try re-enabling these too. And if they're otherwise broken, we should probably file a new bug or re-open that one (depending on the failure method).
Attachment #8801242 - Flags: review?(ryanvm) → review+
Comment on attachment 8801243 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/canvas. https://reviewboard.mozilla.org/r/85970/#review84694 ::: dom/cache/test/mochitest/mochitest.ini (Diff revision 1) > [test_cache_put.html] > [test_cache_requestCache.html] > [test_cache_delete.html] > [test_cache_put_reorder.html] > [test_cache_https.html] > - skip-if = buildapp == 'b2g' # bug 1162353 Can the referenced bug be closed now? Not sure if there are other annotations for it in the tree too.
Attachment #8801243 - Flags: review?(ryanvm) → review+
Comment on attachment 8801244 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/console. https://reviewboard.mozilla.org/r/85972/#review84696 r- for the accidentally-disabled tests. ::: dom/canvas/test/mochitest.ini:215 (Diff revision 1) > [test_bug1215072.html] > subsuite = gpu > [test_canvas.html] > -skip-if = (toolkit == 'gonk' && debug) || (android_version == '18' && debug) #debug-only crash; bug 933541 #android 4.3 debug bug 1143317 > +skip-if = (android_version == '18' && debug) #android 4.3 debug bug 1143317 > [test_canvas_focusring.html] > -skip-if = (toolkit == 'gonk' && !debug) || os == 'win' #specialpowers.wrap > +skip-if = os == 'win' #specialpowers.wrap I'm pretty sure the comment here applied to the B2G skip. Can you please double-check that? ::: dom/canvas/test/reftest/reftest.list:140 (Diff revision 1) > -fuzzy(5,30000) skip-if(!(Android||B2G)) skip-if(Android) pref(webgl.prefer-16bpp,true) == webgl-color-test.html?16bpp&________&premult&_____ wrapper.html?colors-no-alpha.png > -fuzzy(5,30000) skip-if(!(Android||B2G)) skip-if(Android) pref(webgl.prefer-16bpp,true) pref(webgl.force-layers-readback,true) == webgl-color-test.html?16bpp&readback&premult&_____ wrapper.html?colors-no-alpha.png > -fuzzy(9,40000) skip-if(!(Android||B2G)) skip-if(Android) pref(webgl.prefer-16bpp,true) == webgl-color-test.html?16bpp&________&_______&alpha wrapper.html?colors-non-premult.png > -fuzzy(9,40000) skip-if(!(Android||B2G)) skip-if(Android) pref(webgl.prefer-16bpp,true) pref(webgl.force-layers-readback,true) == webgl-color-test.html?16bpp&readback&_______&alpha wrapper.html?colors-non-premult.png > -fuzzy(9,40000) skip-if(!(Android||B2G)) skip-if(Android) pref(webgl.prefer-16bpp,true) == webgl-color-test.html?16bpp&________&premult&alpha wrapper.html?colors-premult.png > -fuzzy(9,40000) skip-if(!(Android||B2G)) skip-if(Android) pref(webgl.prefer-16bpp,true) pref(webgl.force-layers-readback,true) == webgl-color-test.html?16bpp&readback&premult&alpha wrapper.html?colors-premult.png > +# fuzzy(5,30000) pref(webgl.prefer-16bpp,true) == webgl-color-test.html?16bpp&________&premult&_____ wrapper.html?colors-no-alpha.png > +# fuzzy(5,30000) pref(webgl.prefer-16bpp,true) pref(webgl.force-layers-readback,true) == webgl-color-test.html?16bpp&readback&premult&_____ wrapper.html?colors-no-alpha.png > +# fuzzy(9,40000) pref(webgl.prefer-16bpp,true) == webgl-color-test.html?16bpp&________&_______&alpha wrapper.html?colors-non-premult.png > +# fuzzy(9,40000) pref(webgl.prefer-16bpp,true) pref(webgl.force-layers-readback,true) == webgl-color-test.html?16bpp&readback&_______&alpha wrapper.html?colors-non-premult.png > +# fuzzy(9,40000) pref(webgl.prefer-16bpp,true) == webgl-color-test.html?16bpp&________&premult&alpha wrapper.html?colors-premult.png > +# fuzzy(9,40000) pref(webgl.prefer-16bpp,true) pref(webgl.force-layers-readback,true) == webgl-color-test.html?16bpp&readback&premult&alpha wrapper.html?colors-premult.png Did you mean to comment out these test entries? ::: dom/canvas/test/reftest/reftest.list:160 (Diff revision 1) > # Bug 815648 > == stroketext-shadow.html stroketext-shadow-ref.html > > # focus rings > -pref(canvas.focusring.enabled,true) skip-if(B2G) skip-if(cocoaWidget) skip-if(winWidget) needs-focus == drawFocusIfNeeded.html drawFocusIfNeeded-ref.html > -pref(canvas.customfocusring.enabled,true) skip-if(B2G) skip-if(cocoaWidget) skip-if(Android) skip-if(winWidget) fuzzy-if(gtkWidget,64,410) needs-focus == drawCustomFocusRing.html drawCustomFocusRing-ref.html > +pref(canvas.focusring.enabled,true) skip-if(cocoaWidget) skip-if(winWidget) needs-focus == drawFocusIfNeeded.html drawFocusIfNeeded-ref.html > +pref(canvas.customfocusring.enabled,true) skip-if(cocoaWidget) skip-if(Android) skip-if(winWidget) fuzzy-if(gtkWidget,64,410) needs-focus == drawCustomFocusRing.html drawCustomFocusRing-ref.html While you're here, can you please consolidate the skip-ifs? There's no reason to leave them separate like that. ::: dom/canvas/test/webgl-mochitest/mochitest.ini:3 (Diff revision 1) > [DEFAULT] > subsuite = webgl > -skip-if = ((os == 'linux') && (buildapp == 'b2g')) > +skip-if = (os == 'linux') You're disabling these tests on Linux when they weren't previously. The original condition as given was for Linux B2G Desktop builds.
Attachment #8801244 - Flags: review?(ryanvm) → review-
Comment on attachment 8801245 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/crypto. https://reviewboard.mozilla.org/r/85974/#review84698
Attachment #8801245 - Flags: review?(ryanvm) → review+
Comment on attachment 8801246 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/devicestorage. https://reviewboard.mozilla.org/r/85976/#review84700 ::: dom/contacts/tests/chrome.ini:18 (Diff revision 1) > file_contacts_substringmatchingVE.html > file_contacts_substringmatchingCL.html > test_migration_chrome.js > file_migration.html > > # renaming with "_a_" to execure before others, since we hardcode open of Remove the trailing whitespace while you're here please.
Attachment #8801246 - Flags: review?(ryanvm) → review+
Comment on attachment 8801247 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/encoding. https://reviewboard.mozilla.org/r/85978/#review84702
Attachment #8801247 - Flags: review?(ryanvm) → review+
Comment on attachment 8801248 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/events. https://reviewboard.mozilla.org/r/85980/#review84704 ::: dom/devicestorage/test/mochitest.ini:28 (Diff revision 1) > [test_sanity.html] > [test_usedSpace.html] > [test_watch.html] > [test_watchOther.html] > [test_storageAreaListener.html] > -skip-if = (toolkit != 'gonk') > +skip-if = true # Test only ran on Gonk Check with Dave Hylands if we need this test anymore. Also, please fix up the alphabetization while you're here.
Attachment #8801248 - Flags: review?(ryanvm) → review+
Comment on attachment 8801249 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/html. https://reviewboard.mozilla.org/r/85982/#review84706 This looks OK to me as it is, but I'd rather just remove these tests at this point if they're not useful outside of B2G. Maybe Fabrice can rubberstamp the total removal of this directory?
Attachment #8801249 - Flags: review?(ryanvm) → review+
Comment on attachment 8801250 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/imptests. https://reviewboard.mozilla.org/r/85984/#review84708 ::: dom/encoding/test/mochitest.ini:27 (Diff revision 1) > [test_TextEncoder.html] > [test_stringencoding.html] > -skip-if = buildapp == 'b2g' # b2g(Test timed out on b2g board) b2g-debug(Test timed out on b2g board) b2g-desktop(Test timed out on b2g board) > [test_submit_euckr.html] > [test_TLD.html] > [test_utf16_files.html] Clean up the alphabetizing while you're here. ::: dom/encoding/test/unit/xpcshell.ini:13 (Diff revision 1) > [test_big5.js] > [test_euc-jp.js] > [test_euc-kr.js] > [test_gbk.js] > [test_iso-2022-jp.js] > [test_shift_jis.js] Alphebetization and extra newline. The harness runs them alphabetically anyway.
Attachment #8801250 - Flags: review?(ryanvm) → review+
Comment on attachment 8801251 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/indexedDB. https://reviewboard.mozilla.org/r/85986/#review84710 ::: dom/events/test/mochitest.ini:42 (Diff revision 1) > -# Sometimes fails to finish after tests pass on 'B2G ICS Emulator'. > -skip-if = (os == 'b2g') > [test_bug422132.html] > -skip-if = buildapp == 'b2g' # b2g(2 failures out of 8, mousewheel test) b2g-debug(2 failures out of 8, mousewheel test) b2g-desktop(2 failures out of 8, mousewheel test) > [test_bug426082.html] > -skip-if = buildapp == 'mulet' || buildapp == 'b2g' || os == "win" || toolkit == 'android' # b2g(1 failure out of 6, Moving the mouse down from the label should have unpressed the button) b2g-debug(1 failure out of 6, Moving the mouse down from the label should have unpressed the button) b2g-desktop(1 failure out of 6, Moving the mouse down from the label should have unpressed the button) > +skip-if = os == "win" || toolkit == 'android' Looks like the Windows annotation goes back to bug 921693. Not sure about Android. Would be interesting to see if the test passes these days, though. Especially given that it was recently revamped for e10s. ::: dom/events/test/mochitest.ini:44 (Diff revision 1) > [test_bug422132.html] > -skip-if = buildapp == 'b2g' # b2g(2 failures out of 8, mousewheel test) b2g-debug(2 failures out of 8, mousewheel test) b2g-desktop(2 failures out of 8, mousewheel test) > [test_bug426082.html] > -skip-if = buildapp == 'mulet' || buildapp == 'b2g' || os == "win" || toolkit == 'android' # b2g(1 failure out of 6, Moving the mouse down from the label should have unpressed the button) b2g-debug(1 failure out of 6, Moving the mouse down from the label should have unpressed the button) b2g-desktop(1 failure out of 6, Moving the mouse down from the label should have unpressed the button) > +skip-if = os == "win" || toolkit == 'android' > [test_bug427537.html] > [test_bug1003432.html] WHY? Of all the random places to put a test... ::: dom/events/test/mochitest.ini:61 (Diff revision 1) > -skip-if = buildapp == 'mulet' || (buildapp == 'b2g' && toolkit != 'gonk') #Bug 931116, b2g desktop specific, initial triage > [test_bug502818.html] > skip-if = toolkit == 'android' #CRASH_DUMP, RANDOM > [test_bug508479.html] > -skip-if = buildapp == 'mulet' || buildapp == 'b2g' || toolkit == 'android' #CRASH_DUMP, RANDOM # b2g(drag event, also fails on Android) b2g-debug(drag event, also fails on Android) b2g-desktop(drag event, also fails on Android) > +skip-if = toolkit == 'android' #CRASH_DUMP, RANDOM # drag event fails > [test_bug822898.html] *sigh* ::: dom/events/test/pointerevents/mochitest.ini:2 (Diff revision 1) > [DEFAULT] > -skip-if = (toolkit == 'gonk') || (os == 'android') # Bug 1178701 - Issue on 'B2G ICS Emulator' and 'Android' > +skip-if = os == 'android' # Bug 1178701 Ugh, pointing to a closed bug. We should see if these still fail on Android and file a proper bug for fixing and re-enabling them if they don't. Especially since there are people in Taipei actively working on pointer events now.
Attachment #8801251 - Flags: review?(ryanvm) → review+
Comment on attachment 8801252 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/inputmethod. https://reviewboard.mozilla.org/r/85988/#review84714 ::: dom/html/test/chrome.ini:11 (Diff revision 1) > [test_anchor_ping.html] > -skip-if = buildapp == 'b2g' || os == 'android' > +skip-if = os == 'android' > [test_audio_wakelock.html] > -skip-if = buildapp == 'mulet' # TC: Bug 1144079 - Re-enable Mulet mochitests and reftests taskcluster-specific disables. > [test_video_wakelock.html] > -skip-if = toolkit == 'android' || (toolkit == 'gonk' && debug) #bug 871015, bug 881443 > +skip-if = toolkit == 'android' #bug 871015 Wonder if we still need this annotation.
Attachment #8801252 - Flags: review?(ryanvm) → review+
Comment on attachment 8801253 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/ipc. https://reviewboard.mozilla.org/r/85990/#review84750 ::: dom/imptests/html/mochitest.ini:16 (Diff revision 1) > subsuite = gpu > -skip-if = (toolkit == 'android') || (buildapp == 'b2g') #android(WebGL) > +skip-if = toolkit == 'android' #android(WebGL) > [webgl/test_compressedTexSubImage2D.html] > subsuite = gpu > skip-if = true # Bug 1226336 > -#skip-if = (toolkit == 'android') || (buildapp == 'b2g') #android(WebGL) > +#skip-if = toolkit == 'android' #android(WebGL) This is pretty silly. Let's just remove the commented-out line.
Attachment #8801253 - Flags: review?(ryanvm) → review+
Comment on attachment 8801254 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/json. https://reviewboard.mozilla.org/r/85992/#review84752 r- due to the accidentally disabled test. ::: dom/indexedDB/test/mochitest.ini:206 (Diff revision 1) > [test_invalid_version.html] > -skip-if = (buildapp == 'b2g' && toolkit != 'gonk') # Bug 931116 > [test_invalidate.html] > # disabled for the moment > skip-if = true > -# skip-if = (buildapp == 'b2g' && toolkit != 'gonk') # Bug 931116 > +# [test_key_requirements.html] This looks wrong.
Attachment #8801254 - Flags: review?(ryanvm) → review-
Comment on attachment 8801255 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/jsurl. https://reviewboard.mozilla.org/r/85994/#review84754 Looks fine, but this is another one I'd like to see about removing outright if possible. Check with Fabrice.
Attachment #8801255 - Flags: review?(ryanvm) → review+
Comment on attachment 8801256 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/media. https://reviewboard.mozilla.org/r/85996/#review84756 Looks fine, but doesn't appear to have any value outside of B2G. Check with James Cheng if we can remove this outright.
Attachment #8801256 - Flags: review?(ryanvm) → review+
Comment on attachment 8801257 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/network. https://reviewboard.mozilla.org/r/85998/#review84758
Attachment #8801257 - Flags: review?(ryanvm) → review+
Comment on attachment 8801258 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/notification. https://reviewboard.mozilla.org/r/86000/#review84760 Please alphabetize the tests while you're here.
Attachment #8801258 - Flags: review?(ryanvm) → review+
Comment on attachment 8801259 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/permission. https://reviewboard.mozilla.org/r/86002/#review84762
Attachment #8801259 - Flags: review?(ryanvm) → review+
Comment on attachment 8801260 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/plugins. https://reviewboard.mozilla.org/r/86004/#review84764 ::: dom/media/mediasource/test/mochitest.ini:62 (Diff revision 1) > [test_DurationChange.html] > [test_DurationUpdated.html] > [test_DurationUpdated_mp4.html] > skip-if = ((os == "win" && os_version == "5.1") || (toolkit == 'android')) # Not supported on xp and android 2.3 > [test_EndOfStream.html] > -skip-if = (true || toolkit == 'android' || buildapp == 'mulet') #timeout android/mulet only bug 1101187 and bug 1182946 > +skip-if = (true || toolkit == 'android') #timeout android only bug 1101187 and bug 1182946 This is...interesting. ::: dom/media/mediasource/test/mochitest.ini:64 (Diff revision 1) > [test_DurationUpdated_mp4.html] > skip-if = ((os == "win" && os_version == "5.1") || (toolkit == 'android')) # Not supported on xp and android 2.3 > [test_EndOfStream.html] > -skip-if = (true || toolkit == 'android' || buildapp == 'mulet') #timeout android/mulet only bug 1101187 and bug 1182946 > +skip-if = (true || toolkit == 'android') #timeout android only bug 1101187 and bug 1182946 > [test_EndOfStream_mp4.html] > -skip-if = ((os == "win" && os_version == "5.1") || (toolkit == 'android' || buildapp == 'mulet')) # Not supported on xp and android 2.3 > +skip-if = ((os == "win" && os_version == "5.1") || (toolkit == 'android')) # Not supported on xp and android 2.3 I wonder how many of these tests would pass on Android if indeed it's an issue with 2.3 specifically. ::: dom/media/test/crashtests/crashtests.list:81 (Diff revision 1) > load 1045650.html > load 1080986.html > load 1122218.html > load 1127188.html > load 1157994.html > -skip-if(!B2G) load 1158427.html > +skip-if(true) load 1158427.html # Test only ran on B2G I think this test can be removed. Paul Adenot can probably say for sure. ::: dom/media/test/crashtests/crashtests.list:99 (Diff revision 1) > load offline-buffer-source-ended-1.html > load oscillator-ended-1.html > load oscillator-ended-2.html > load video-replay-after-audio-end.html > # This needs to run at the end to avoid leaking busted state into other tests. > -skip-if(B2G) load 691096-1.html # bug 852821 > +load 691096-1.html # bug 852821 Comment can be removed. ::: dom/media/test/mochitest.ini:26 (Diff revision 1) > # test first check canPlayType for the type, and if it's not supported, just > # do ok(true, "Type not supported") and stop the test. > > [DEFAULT] > subsuite = media > -skip-if = buildapp == 'mulet' || android_version == '18' > +skip-if = android_version == '18' I believe this is because we're running them on real devices now via autophone, but can we please add a comment to that effect while we're here? ::: dom/media/test/mochitest.ini:642 (Diff revision 1) > [test_bug448534.html] > -skip-if = buildapp == 'mulet' > [test_bug463162.xhtml] > [test_bug465498.html] > [test_bug495145.html] > skip-if = (os == 'mac' && os_version == '10.6') # bug 1021174 Definitely want a follow-up bug for getting rid of 10.6 annotations too. ::: dom/media/test/mochitest.ini:781 (Diff revision 1) > tags=msg > [test_mediarecorder_webm_support.html] > skip-if = os == 'android' || arch == 'arm' || arch == 'arm64' > tags=msg > [test_mediarecorder_mp4_support.html] > -skip-if = toolkit != 'gonk' || android_version < '17' # Android/Gonk before SDK version 17 does not have the OMX Encoder API. > +skip-if = true # Test only ran on Gonk Check with Randell Jesup if we need this test at all anymore.
Attachment #8801260 - Flags: review?(ryanvm) → review+
Comment on attachment 8801261 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/power. https://reviewboard.mozilla.org/r/86006/#review84766 Check with Fabrice if these are needed at all anymore.
Attachment #8801261 - Flags: review?(ryanvm) → review+
Comment on attachment 8801262 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/presentation. https://reviewboard.mozilla.org/r/86008/#review84768 Check with Fabrice if these can go.
Attachment #8801262 - Flags: review?(ryanvm) → review+
Comment on attachment 8801263 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/promise. https://reviewboard.mozilla.org/r/86010/#review84770 ::: dom/network/tests/mochitest.ini:18 (Diff revision 1) > [test_networkstats_disabled.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk > [test_networkstats_enabled_no_perm.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk > [test_networkstats_enabled_perm.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk Check with Fabrice if the networkstats tests can be removed outright.
Attachment #8801263 - Flags: review?(ryanvm) → review+
Comment on attachment 8801264 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/push. https://reviewboard.mozilla.org/r/86012/#review84772
Attachment #8801264 - Flags: review?(ryanvm) → review+
Comment on attachment 8801265 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/security. https://reviewboard.mozilla.org/r/86014/#review84774
Attachment #8801265 - Flags: review?(ryanvm) → review+
Comment on attachment 8801266 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/svg. https://reviewboard.mozilla.org/r/86016/#review84776
Attachment #8801266 - Flags: review?(ryanvm) → review+
Comment on attachment 8801267 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/system. https://reviewboard.mozilla.org/r/86018/#review84778 ::: dom/plugins/test/reftest/reftest.list:10 (Diff revision 1) > fails-if(!haveTestPlugin) fuzzy-if(skiaContent&&haveTestPlugin,1,164000) == plugin-alpha-opacity.html div-alpha-opacity.html > random-if(/^Windows\x20NT\x206\.1/.test(http.oscpu)) fails-if(!haveTestPlugin) == windowless-clipping-1.html windowless-clipping-1-ref.html # bug 631832 > # fuzzy because of anti-aliasing in dashed border > fuzzy(16,256) random-if(/^Windows\x20NT\x206\.1/.test(http.oscpu)) fails-if(!haveTestPlugin) == border-padding-1.html border-padding-1-ref.html # bug 629430 > fuzzy(16,256) random-if(/^Windows\x20NT\x206\.1/.test(http.oscpu)) fails-if(!haveTestPlugin) == border-padding-2.html border-padding-2-ref.html # bug 629430 > -fuzzy(16,256) random-if(/^Windows\x20NT\x206\.1/.test(http.oscpu)) skip-if(!haveTestPlugin) skip-if(Android||B2G) == border-padding-3.html border-padding-3-ref.html # bug 629430 # bug 773482 > +fuzzy(16,256) random-if(/^Windows\x20NT\x206\.1/.test(http.oscpu)) skip-if(!haveTestPlugin) skip-if(Android) == border-padding-3.html border-padding-3-ref.html # bug 629430 # bug 773482 Please consolidate the skip-ifs into one condition.
Attachment #8801267 - Flags: review?(ryanvm) → review+
Comment on attachment 8801268 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/tests. https://reviewboard.mozilla.org/r/86020/#review84780 ::: dom/power/test/chrome.ini:9 (Diff revision 1) > [test_power_set_screen_brightness.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk > [test_power_set_screen_enabled.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk > [test_power_set_key_light_enabled.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk Check with Fabrice if these can be removed outright.
Attachment #8801268 - Flags: review?(ryanvm) → review+
Comment on attachment 8801269 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/url. https://reviewboard.mozilla.org/r/86022/#review84784
Attachment #8801269 - Flags: review?(ryanvm) → review+
Comment on attachment 8801270 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/workers. https://reviewboard.mozilla.org/r/86024/#review84786
Attachment #8801270 - Flags: review?(ryanvm) → review+
Comment on attachment 8801271 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xbl. https://reviewboard.mozilla.org/r/86026/#review84788
Attachment #8801271 - Flags: review?(ryanvm) → review+
Comment on attachment 8801272 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xhr. https://reviewboard.mozilla.org/r/86028/#review84790 Looks like another directory ripe for removal. Check with Fabrice.
Attachment #8801272 - Flags: review?(ryanvm) → review+
Comment on attachment 8801273 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xml. https://reviewboard.mozilla.org/r/86030/#review84792
Attachment #8801273 - Flags: review?(ryanvm) → review+
Comment on attachment 8801274 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xul. https://reviewboard.mozilla.org/r/86032/#review84794 More to check with Fabrice about. Please fix up the ordering too.
Attachment #8801274 - Flags: review?(ryanvm) → review+
Comment on attachment 8801275 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: editor. https://reviewboard.mozilla.org/r/86034/#review84796
Attachment #8801275 - Flags: review?(ryanvm) → review+
Comment on attachment 8801276 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: embedding. https://reviewboard.mozilla.org/r/86036/#review84798 ::: dom/system/tests/mochitest.ini:8 (Diff revision 1) > preload-SystemUpdateManager-jsm.js > file_bug1197901.html > > [test_system_update_enabled.html] > -skip-if = buildapp != 'b2g' > +skip-if = true # Tests only ran on B2G > [test_bug1197901.html] Please alphabetize.
Attachment #8801276 - Flags: review?(ryanvm) → review+
Comment on attachment 8801277 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: extensions. https://reviewboard.mozilla.org/r/86038/#review84800 ::: dom/tests/browser/browser.ini:16 (Diff revision 1) > !/dom/tests/mochitest/geolocation/network_geolocation.sjs > > [browser_test_toolbars_visibility.js] > support-files = > test_new_window_from_content_child.html > [browser_bug1008941_dismissGeolocationHanger.js] Please alphabetize. ::: dom/tests/mochitest/ajax/offline/mochitest.ini:90 (Diff revision 1) > [test_obsolete.html] > -skip-if = buildapp == 'b2g' > [test_offlineIFrame.html] > -skip-if = buildapp == 'b2g' > # [test_offlineMode.html] > # disabled due to bug 656943 Can we please turn this into a proper skip? ::: dom/tests/mochitest/ajax/prototype/mochitest.ini:67 (Diff revision 1) > test/unit/tmp/selector_test.html > test/unit/tmp/string_test.html > test/unit/tmp/unit_test.html > > [test_Prototype.html] > -skip-if = (toolkit == 'gonk' && debug) || toolkit == 'android' #debug-only timeout > +skip-if = toolkit == 'android' #debug-only timeout Comment doesn't appear to match the skip-if condition. ::: dom/tests/mochitest/ajax/scriptaculous/mochitest.ini:34 (Diff revision 1) > test/unit/sortable_test.html > test/unit/string_test.html > test/unit/unit_test.html > > [test_Scriptaculous.html] > -skip-if = (toolkit == 'gonk' && debug) || toolkit == 'android' #debug-only failure > +skip-if = toolkit == 'android' #debug-only failure Same as previous. ::: dom/tests/mochitest/general/chrome.ini:8 (Diff revision 1) > [test_innerScreen.xul] > [test_offsets.css] > [test_offsets.js] > [test_offsets.xul] > -skip-if = buildapp == 'mulet' > [test_idleapi_permissions.html] Please alphabetize. ::: dom/tests/mochitest/general/mochitest.ini:125 (Diff revision 1) > [test_selectevents.html] > -skip-if = buildapp == 'b2g' || buildapp == 'mulet' || toolkit == 'android' # Mouse doesn't select in the same way > +skip-if = toolkit == 'android' # bug 1230232 - Mouse doesn't select in the same way > -# Disabled on Android, see bug 1230232 > [test_WebKitCSSMatrix.html] > [test_resource_timing_frameset.html] > [test_bug1208217.html] Please alphabetize.
Attachment #8801277 - Flags: review?(ryanvm) → review+
Comment on attachment 8801278 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: gfx. https://reviewboard.mozilla.org/r/86040/#review84802 ::: dom/url/tests/chrome.ini:12 (Diff revision 1) > jsm_url_worker.js > !/dom/workers/test/dom_worker_helper.js > > [test_url.xul] > [test_worker_url.xul] > [test_bug883784.xul] Alphabetize please :)
Attachment #8801278 - Flags: review?(ryanvm) → review+
Comment on attachment 8801279 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: image. https://reviewboard.mozilla.org/r/86042/#review84804
Attachment #8801279 - Flags: review?(ryanvm) → review+
Comment on attachment 8801280 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: intl. https://reviewboard.mozilla.org/r/86044/#review84806 ::: dom/xbl/test/mochitest.ini:42 (Diff revision 1) > [test_bug821850.html] > -skip-if = buildapp == 'mulet' > [test_bug844783.html] > [test_bug872273.xhtml] > [test_bug946815.html] > -skip-if = toolkit != "gonk" > +skip-if = true # Test only ran on Gonk. Check with bholley if this can be removed outright.
Attachment #8801280 - Flags: review?(ryanvm) → review+
Comment on attachment 8801281 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: ipc. https://reviewboard.mozilla.org/r/86046/#review84808 ::: dom/xhr/tests/mochitest.ini:101 (Diff revision 1) > -skip-if = buildapp == 'b2g' > [test_worker_xhr_responseURL.html] > [test_worker_xhr_system.html] > [test_worker_xhr_timeout.html] > skip-if = (os == "win") || (os == "mac") || toolkit == 'android' #bug 798220 > [test_relativeLoad.html] More painful test ordering :(
Attachment #8801281 - Flags: review?(ryanvm) → review+
Comment on attachment 8801282 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: js. https://reviewboard.mozilla.org/r/86048/#review84810
Attachment #8801282 - Flags: review?(ryanvm) → review+
Comment on attachment 8801283 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/base. https://reviewboard.mozilla.org/r/86050/#review84812
Attachment #8801283 - Flags: review?(ryanvm) → review+
Comment on attachment 8801284 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/forms. https://reviewboard.mozilla.org/r/86052/#review84814
Attachment #8801284 - Flags: review?(ryanvm) → review+
Comment on attachment 8801285 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/generic. https://reviewboard.mozilla.org/r/86054/#review84816
Attachment #8801285 - Flags: review?(ryanvm) → review+
Comment on attachment 8801286 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/inspector. https://reviewboard.mozilla.org/r/86056/#review84818
Attachment #8801286 - Flags: review?(ryanvm) → review+
Comment on attachment 8801287 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/style. https://reviewboard.mozilla.org/r/86058/#review84820 ::: gfx/layers/apz/test/mochitest/mochitest.ini:42 (Diff revision 1) > - skip-if = (os == 'android') || (os == 'b2g') || (buildapp == 'mulet') # wheel events not supported on mobile; see bug 1164274 for mulet > + skip-if = (os == 'android') # wheel events not supported on mobile > [test_wheel_scroll.html] > - skip-if = (os == 'android') || (os == 'b2g') || (buildapp == 'mulet') # wheel events not supported on mobile; see bug 1164274 for mulet > + skip-if = (os == 'android') # wheel events not supported on mobile > [test_wheel_transactions.html] > - skip-if = (os == 'android') || (os == 'b2g') || (buildapp == 'mulet') # wheel events not supported on mobile; see bug 1164274 for mulet > + skip-if = (os == 'android') # wheel events not supported on mobile > [test_bug1151667.html] Please alphabetize.
Attachment #8801287 - Flags: review?(ryanvm) → review+
Comment on attachment 8801288 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: layout/xul. https://reviewboard.mozilla.org/r/86060/#review84822 ::: image/test/mochitest/mochitest.ini:92 (Diff revision 1) > # [test_bug435296.html] > # disabled - See bug 578591 > [test_bug466586.html] > [test_bug468160.html] > # [test_bug478398.html] > # disabled - See bug 579139 Please make this and the one above it proper skip-ifs while you're here. ::: image/test/mochitest/mochitest.ini:120 (Diff revision 1) > -skip-if = toolkit == "gonk" #Bug 997034 - canvas.toDataURL() often causes lost connection to device. > [test_error_events.html] > [test_short_gif_header.html] > [test_image_buffer_limit.html] > -#skip-if = toolkit != "gonk" #Image buffer limit is only set for Firefox OS currently. > +#skip-if = true #Image buffer limit was only set for Firefox OS. > disabled = bug 1060869 Please see what Timothy Nikkel thinks about whether we should keep this test or not. ::: image/test/mochitest/mochitest.ini:156 (Diff revision 1) > [test_undisplayed_iframe.html] > -skip-if = buildapp == 'b2g' || os == 'android' > +skip-if = os == 'android' > [test_xultree_animation.xhtml] > -skip-if = buildapp == 'b2g' || os == 'android' > +skip-if = os == 'android' > [test_bug1132427.html] > -skip-if = buildapp == 'b2g' || os == 'android' > +skip-if = os == 'android' Please alphabetize. ::: image/test/reftest/gif/reftest.list:48 (Diff revision 1) > # --BOUNDARYOMG\r\n > # Content-Type: image/gif\r\n > # \r\n > # <contents of blue.gif> (no newline) > # --BOUNDARYOMG--\r\n > # Please kill this trailing whitespace while you're here.
Attachment #8801288 - Flags: review?(ryanvm) → review+
Comment on attachment 8801289 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: modules. https://reviewboard.mozilla.org/r/86062/#review84824
Attachment #8801289 - Flags: review?(ryanvm) → review+
Comment on attachment 8801290 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: netwerk. https://reviewboard.mozilla.org/r/86064/#review84826
Attachment #8801290 - Flags: review?(ryanvm) → review+
Comment on attachment 8801291 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: parser. https://reviewboard.mozilla.org/r/86066/#review84828 ::: js/xpconnect/tests/chrome/chrome.ini:88 (Diff revision 1) > [test_bug1074863.html] > [test_bug1092477.xul] > [test_bug1124898.html] > [test_bug1126911.html] > [test_bug1281071.xul] > [test_xrayToJS.xul] Please alphabetize.
Attachment #8801291 - Flags: review?(ryanvm) → review+
Comment on attachment 8801292 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: rdf. https://reviewboard.mozilla.org/r/86068/#review84830 ::: layout/base/tests/mochitest.ini:85 (Diff revision 1) > [test_bug404209.xhtml] > [test_bug416896.html] > [test_bug423523.html] > [test_bug449781.html] > [test_bug450930.xhtml] > -skip-if = buildapp == 'b2g' || true # bug 934301 # b2g(times out) b2g-debug(times out) b2g-desktop(times out) > +skip-if = true # bug 934301 Might be worth seeing if this passes these days.
Attachment #8801292 - Flags: review?(ryanvm) → review+
Comment on attachment 8801293 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: security. https://reviewboard.mozilla.org/r/86070/#review84832 ::: layout/forms/test/mochitest.ini:63 (Diff revision 1) > [test_bug961363.html] > skip-if = toolkit == 'android' # Bug 1021644 - Fails when pushed into a different chunk on Android > [test_bug1111995.html] > -skip-if = buildapp == 'mulet' || buildapp == 'b2g' > [test_select_vertical.html] > -skip-if = e10s || buildapp == 'mulet' || buildapp == 'b2g' || toolkit == 'android' # Bug 1170129 - vertical <select> popup not implemented for e10s # <select> elements don't use an in-page popup on B2G/Android > +skip-if = e10s || toolkit == 'android' # Bug 1170129 - vertical <select> popup not implemented for e10s # <select> elements don't use an in-page popup on Android Please alphabetize.
Attachment #8801293 - Flags: review?(ryanvm) → review+
Comment on attachment 8801294 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: services. https://reviewboard.mozilla.org/r/86072/#review84834 ::: layout/generic/test/chrome.ini:16 (Diff revision 1) > [test_bug469613.xul] > [test_bug469774.xul] > [test_bug508115.xul] > [test_bug514732-2.xul] > [test_bug632379.xul] > -skip-if = buildapp == 'mulet' || os == 'linux' # Bug 1207914 for Linux > +skip-if = os == 'linux' # Bug 1207914 for Linux You can drop the "for Linux" part from this
Attachment #8801294 - Flags: review?(ryanvm) → review+
Comment on attachment 8801295 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: storage. https://reviewboard.mozilla.org/r/86074/#review84836
Attachment #8801295 - Flags: review?(ryanvm) → review+
Comment on attachment 8801296 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: toolkit. https://reviewboard.mozilla.org/r/86076/#review84838
Attachment #8801296 - Flags: review?(ryanvm) → review+
Comment on attachment 8801297 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: uriloader. https://reviewboard.mozilla.org/r/86078/#review84840 ::: layout/xul/test/mochitest.ini:11 (Diff revision 1) > skip-if = toolkit == 'android' #bug 798806 > [test_bug563416.html] > [test_resizer_incontent.xul] > [test_splitter.xul] > skip-if = toolkit == 'android' # no XUL theme > [test_bug1197913.xul] Please alphabetize.
Comment on attachment 8801297 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: uriloader. https://reviewboard.mozilla.org/r/86078/#review84842
Attachment #8801297 - Flags: review?(ryanvm) → review+
Comment on attachment 8801298 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: widget. https://reviewboard.mozilla.org/r/86080/#review84844
Attachment #8801298 - Flags: review?(ryanvm) → review+
Comment on attachment 8801299 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpcom. https://reviewboard.mozilla.org/r/86082/#review84846 ::: netwerk/test/mochitests/mochitest.ini:24 (Diff revision 1) > [test_user_agent_overrides.html] > [test_user_agent_updates.html] > [test_user_agent_updates_reset.html] > [test_viewsource_unlinkable.html] > [test_xhr_method_case.html] > [test_loadinfo_redirectchain.html] Please alphabetize.
Attachment #8801299 - Flags: review?(ryanvm) → review+
Comment on attachment 8801299 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpcom. https://reviewboard.mozilla.org/r/86082/#review84848 ::: netwerk/test/unit/xpcshell_b2g.ini:4 (Diff revision 1) > [DEFAULT] > head = head_channels.js head_cache.js head_cache2.js > tail = > -skip-if = buildapp != 'b2g' > +skip-if = true This can probably go. Check with Fabrice.
Comment on attachment 8801300 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpfe. https://reviewboard.mozilla.org/r/86084/#review84850
Attachment #8801300 - Flags: review?(ryanvm) → review+
Comment on attachment 8801301 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpcom. https://reviewboard.mozilla.org/r/86086/#review84852
Attachment #8801301 - Flags: review?(ryanvm) → review+
Comment on attachment 8801302 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpfe. https://reviewboard.mozilla.org/r/86088/#review84854
Attachment #8801302 - Flags: review?(ryanvm) → review+
Comment on attachment 8801303 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpfe. https://reviewboard.mozilla.org/r/86090/#review84856 ::: services/fxaccounts/tests/xpcshell/xpcshell.ini:16 (Diff revision 1) > [test_credentials.js] > [test_loginmgr_storage.js] > -skip-if = appname == 'b2g' # login manager storage only used on desktop. > [test_manager.js] > -skip-if = appname != 'b2g' > -reason = FxAccountsManager is only available for B2G for now > +skip-if = true > +reason = FxAccountsManager was only available for B2G Check with markh as to whether this test can be dropped or not.
Attachment #8801303 - Flags: review?(ryanvm) → review+
Comment on attachment 8801304 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: storage. https://reviewboard.mozilla.org/r/86092/#review84858
Attachment #8801304 - Flags: review?(ryanvm) → review+
Comment on attachment 8801305 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: toolkit. https://reviewboard.mozilla.org/r/86094/#review84860 ::: toolkit/components/osfile/tests/xpcshell/xpcshell.ini:17 (Diff revision 1) > [test_osfile_async_append.js] > [test_osfile_async_bytes.js] > [test_osfile_async_copy.js] > [test_osfile_async_flush.js] > [test_osfile_async_largefiles.js] > [test_osfile_async_setDates.js] Please alphabetize. I'm a bit nervous about removing the requesttimeoutfactor line as well (mostly for Android), but we'll see how it goes. ::: toolkit/identity/tests/unit/xpcshell.ini:4 (Diff revision 1) > [DEFAULT] > head = head_identity.js > tail = tail_identity.js > -skip-if = (appname != "b2g" || toolkit == 'gonk') > +skip-if = true # Tests ran only on non-Gonk B2G I think this can go now. Maybe markh would know for sure. ::: toolkit/mozapps/update/tests/unit_aus_update/xpcshell.ini:26 (Diff revision 1) > [downloadResumeForSameAppVersion.js] > [downloadCompleteAfterPartialFailure.js] > -skip-if = buildapp == "mulet" || toolkit == 'gonk' > -reason = custom nsIUpdatePrompt > [downloadFileTooBig_gonk.js] > -skip-if = toolkit != 'gonk' > +skip-if = true This can probably go. Check with Fabrice.
Attachment #8801305 - Flags: review?(ryanvm) → review+
Comment on attachment 8801306 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: uriloader. https://reviewboard.mozilla.org/r/86096/#review84862
Attachment #8801306 - Flags: review?(ryanvm) → review+
Comment on attachment 8801307 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: widget. https://reviewboard.mozilla.org/r/86098/#review84864
Attachment #8801307 - Flags: review?(ryanvm) → review+
Comment on attachment 8801308 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpcom. https://reviewboard.mozilla.org/r/86100/#review84866
Attachment #8801308 - Flags: review?(ryanvm) → review+
Comment on attachment 8801309 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpfe. https://reviewboard.mozilla.org/r/86102/#review84868
Attachment #8801309 - Flags: review?(ryanvm) → review+
Comment on attachment 8801244 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/console. https://reviewboard.mozilla.org/r/85972/#review84696 > I'm pretty sure the comment here applied to the B2G skip. Can you please double-check that? Yes, it was. Fixed. > Did you mean to comment out these test entries? Yes, the test don't run on any platform since https://hg.mozilla.org/mozilla-central/rev/bef797e6c129 landed.
Comment on attachment 8801244 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/console. https://reviewboard.mozilla.org/r/85972/#review84696 > Yes, the test don't run on any platform since https://hg.mozilla.org/mozilla-central/rev/bef797e6c129 landed. I feel like skip-if(true) better clarifies the intent here in that case.
Comment on attachment 8801299 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpcom. https://reviewboard.mozilla.org/r/86082/#review84848 > This can probably go. Check with Fabrice. This got removed in bug 1310721 / https://hg.mozilla.org/integration/mozilla-inbound/rev/df41cedc3884
Comment on attachment 8801260 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/plugins. https://reviewboard.mozilla.org/r/86004/#review84764 > This is...interesting. Needinfo jya https://bugzilla.mozilla.org/show_bug.cgi?id=1182946#c36 > Definitely want a follow-up bug for getting rid of 10.6 annotations too. Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1312519
Comment on attachment 8801256 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/media. https://reviewboard.mozilla.org/r/85996/#review84756 inputport is designed for TV product on B2G. It is no longer required. Please remove it, Thanks.
Comment on attachment 8801233 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: devtools. https://reviewboard.mozilla.org/r/85950/#review84638 > Do you mind seeing if this still fails on Android opt or not? Maybe we can just re-enable it across the board and close out bug 1141544. I've also ni? myself in that bug to check otherwise, but if you beat me to it, great :) This passed https://treeherder.mozilla.org/#/jobs?repo=try&revision=ad4d80865d4662879dd72a6bb39b732fae75d3e4 , removed the annotation.
Comment on attachment 8801234 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: docshell. https://reviewboard.mozilla.org/r/85952/#review84646 > Another possible cleanup bug we could file: getting rid of GTK2 annotations now that we're on GTK3 across the board AFAIK. Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1312400 > Some of these poorly-annotated Android skips might be worth checking on Try too. Test run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=840db0870cd4928e868f8cf11a42aa5de8c3482f From the tests I enabled, only test_bug570341.html from docshell/test/mochitest.ini and test_bug13871.html and test_bug270414.html both from docshell/test/navigation/mochitest.ini didn't fail instantly.
Comment on attachment 8801238 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/base. https://reviewboard.mozilla.org/r/85960/#review84660 test_bug466080.html is the only enabled test which didn't fail immediately on Android. Test run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7df28a2d5182c64311cbb7851d2feefbfec2d4a1&selectedJob=29782110 > More craptasically-commented Android disablings worth checking on :( Still failing https://treeherder.mozilla.org/logviewer.html#?job_id=29782110&repo=try > Please file a follow-up bug and ni? wchen to see whether these are needed anymore or not. You can also change the comment to point to that bug instead then. Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1312417 > Would love to know if these work on Android at this point. Still fail: https://treeherder.mozilla.org/logviewer.html#?job_id=29790629&repo=try > More older-than-time disablings on Android :( And still failing: https://treeherder.mozilla.org/logviewer.html#?job_id=29790629&repo=try
Comment on attachment 8801251 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/indexedDB. https://reviewboard.mozilla.org/r/85986/#review84710 > Looks like the Windows annotation goes back to bug 921693. Not sure about Android. Would be interesting to see if the test passes these days, though. Especially given that it was recently revamped for e10s. Didn't fail on the try push https://treeherder.mozilla.org/#/jobs?repo=try&revision=91b37db0a6563f048587cb555a4d6e6d5d0fec5c , removed both annotations. > Ugh, pointing to a closed bug. We should see if these still fail on Android and file a proper bug for fixing and re-enabling them if they don't. Especially since there are people in Taipei actively working on pointer events now. Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1312791
Comment on attachment 8801292 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: rdf. https://reviewboard.mozilla.org/r/86068/#review84830 > Might be worth seeing if this passes these days. Didn't see it fail in https://treeherder.mozilla.org/#/jobs?repo=try&revision=91b37db0a6563f048587cb555a4d6e6d5d0fec5c , so enabled it.
Comment on attachment 8801256 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/media. https://reviewboard.mozilla.org/r/85996/#review84756 Created bug 1312800 for the removal and dropped this patch for this bug.
Comment on attachment 8801233 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: devtools. https://reviewboard.mozilla.org/r/85950/#review87420 ::: devtools/server/tests/unit/xpcshell.ini:137 (Diff revision 1) > [test_conditional_breakpoint-01.js] > [test_conditional_breakpoint-02.js] > [test_conditional_breakpoint-03.js] > [test_eventlooplag_actor.js] > -skip-if = toolkit != "gonk" > +skip-if = true > +reason = only ran on B2G Can be removed, filed bug 1312807 because also the frontend has to be dropped.
Comment on attachment 8801303 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: xpfe. https://reviewboard.mozilla.org/r/86090/#review84856 > Check with markh as to whether this test can be dropped or not. Removed and filed bug 1313062 for the FxAccountsManager removal.
Comment on attachment 8801305 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: toolkit. https://reviewboard.mozilla.org/r/86094/#review84860 > I think this can go now. Maybe markh would know for sure. Created bug 1313045 and dropped this.
Comment on attachment 8801234 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: docshell. https://reviewboard.mozilla.org/r/85952/#review84646
Comment on attachment 8801272 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xhr. https://reviewboard.mozilla.org/r/86028/#review84790 Already removed in bug 1312591.
Comment on attachment 8801305 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: toolkit. https://reviewboard.mozilla.org/r/86094/#review84860 > This can probably go. Check with Fabrice. Removed.
Comment on attachment 8801274 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/xul. https://reviewboard.mozilla.org/r/86032/#review84794 Filed bug 1313155.
Comment on attachment 8801268 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/tests. https://reviewboard.mozilla.org/r/86020/#review84780 > Check with Fabrice if these can be removed outright. Removed them.
Comment on attachment 8801263 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/promise. https://reviewboard.mozilla.org/r/86010/#review84770 > Check with Fabrice if the networkstats tests can be removed outright. Removed them.
Comment on attachment 8801261 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/power. https://reviewboard.mozilla.org/r/86006/#review84766 Yes, can go, filed bug 1313159.
Comment on attachment 8801249 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/html. https://reviewboard.mozilla.org/r/85982/#review84706 Filed bug 1313167.
Comment on attachment 8801255 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/jsurl. https://reviewboard.mozilla.org/r/85994/#review84754 Filed bug 1313169.
Comment on attachment 8801238 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/base. https://reviewboard.mozilla.org/r/85960/#review84660 > Another one for the "I wonder if this even fails on Android anymore" list. Especially on a WONTFIXed bug. Didn't see it fail in the Try push, so enabled it.
Comment on attachment 8801242 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/cache. https://reviewboard.mozilla.org/r/85968/#review84678 > Another one I'd be curious about whether it passes or not at this point. Fails: https://treeherder.mozilla.org/logviewer.html#?job_id=29826727&repo=try > Wonder if this still fails on Android or not. Either way, please clean up the comment as it's redundantly-written as it stands currently. Hasn't failed in https://treeherder.mozilla.org/#/jobs?repo=try&revision=91b37db0a6563f048587cb555a4d6e6d5d0fec5c&selectedJob=29825601 > The last comments in the referenced bug suggest we could try re-enabling these too. And if they're otherwise broken, we should probably file a new bug or re-open that one (depending on the failure method). Times out: https://treeherder.mozilla.org/logviewer.html#?job_id=29828866&repo=try The Makefile.in linked in the bug is gone.
Comment on attachment 8801260 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/plugins. https://reviewboard.mozilla.org/r/86004/#review84764 > I wonder how many of these tests would pass on Android if indeed it's an issue with 2.3 specifically. There are intermittent failures at least for these tests: test_DrainOnMissingData_mp4.html test_FrameSelection_mp4.html test_SeekToEnd_mp4.html test_SeekTwice_mp4.html test_Threshold_mp4.html test_WaitingOnMissingDataEnded_mp4.html Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=91b37db0a6563f048587cb555a4d6e6d5d0fec5c > I think this test can be removed. Paul Adenot can probably say for sure. Enabled it globally on padenot's suggestion.
Comment on attachment 8801244 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/console. https://reviewboard.mozilla.org/r/85972/#review88712
Attachment #8801244 - Flags: review?(ryanvm) → review+
Comment on attachment 8801254 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/json. https://reviewboard.mozilla.org/r/85992/#review88714 ::: dom/inputmethod/mochitest/chrome.ini:3 (Diff revision 3) > [DEFAULT] > -# dom/inputmethod only makes sense on B2G > -run-if = (buildapp == 'b2g' && toolkit != 'gonk') || buildapp == 'mulet' > +# dom/inputmethod only made sense on B2G > +skip-if = true I'm wondering if these can go at this point.
Attachment #8801254 - Flags: review?(ryanvm) → review+
(In reply to Ryan VanderMeulen [:RyanVM] from comment #133) > ::: dom/xbl/test/mochitest.ini:42 > > [test_bug946815.html] > > -skip-if = toolkit != "gonk" > > +skip-if = true # Test only ran on Gonk. > > Check with bholley if this can be removed outright. Removed it (can't find the comment on review board, opening older revisions shows other patches because some patches got removed). (In reply to Ryan VanderMeulen [:RyanVM] from comment #113) > ::: dom/media/test/crashtests/crashtests.list:81 > > load 1157994.html > > -skip-if(!B2G) load 1158427.html > > +skip-if(true) load 1158427.html # Test only ran on B2G > > I think this test can be removed. Paul Adenot can probably say for sure. Enabled it globally. > > [test_mediarecorder_mp4_support.html] > > -skip-if = toolkit != 'gonk' || android_version < '17' # Android/Gonk before SDK version 17 does not have the OMX Encoder API. > > +skip-if = true # Test only ran on Gonk > > Check with Randell Jesup if we need this test at all anymore. Removed it.
Comment on attachment 8801260 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/plugins. https://reviewboard.mozilla.org/r/86004/#review84764 > Needinfo jya https://bugzilla.mozilla.org/show_bug.cgi?id=1182946#c36 jya handled this in bug 1314212
Comment on attachment 8801254 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/json. https://reviewboard.mozilla.org/r/85992/#review88714 > I'm wondering if these can go at this point. Filed bug 1313169 for dropping dom/inputmethod.
Comment on attachment 8801236 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/apps. Alexandre, this patch disables the dom/apps chrome tests. Should they be kept in tree? If I remember, there were more app-related tests in different directories, but they seem to have already been removed.
Attachment #8801236 - Flags: feedback?(lissyx+mozillians)
Well there is ongoing work to put a bullet into dom/apps completely, so I would say just remove them, no value in keeping. Now technically I am no peer of this, but I don't think it would be a problem in this case :)
Comment on attachment 8801236 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/apps. https://reviewboard.mozilla.org/r/85956/#review90578
Attachment #8801236 - Flags: review+
Attachment #8801236 - Flags: feedback?(lissyx+mozillians)
Comment on attachment 8801236 [details] Bug 1310297 - Remove test annotations using b2g, mulet or gonk: dom/apps. https://reviewboard.mozilla.org/r/85956/#review84654 > I'm wondering if these should just be removed at this point. Probably worth pinging gerard-majax about it. Removed the chrome tests.
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/7ef66a5e57ac Remove test annotations using b2g, mulet or gonk: browser. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/c190459f9e03 Remove test annotations using b2g, mulet or gonk: caps. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/2b55ec912ac5 Remove test annotations using b2g, mulet or gonk: chrome. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/89738aaf87c1 Remove test annotations using b2g, mulet or gonk: devtools. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/82ab05df35f7 Remove test annotations using b2g, mulet or gonk: docshell. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/1e3310aa319d Remove test annotations using b2g, mulet or gonk: dom/animation. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/365227cc8c6b Remove test annotations using b2g, mulet or gonk: dom/apps. r=gerard-majax,RyanVM https://hg.mozilla.org/integration/autoland/rev/9d136b70985b Remove test annotations using b2g, mulet or gonk: dom/archivereader. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/6809bf6acf2e Remove test annotations using b2g, mulet or gonk: dom/base. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/63e0fce78144 Remove test annotations using b2g, mulet or gonk: dom/bindings. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/3cf5abd5dede Remove test annotations using b2g, mulet or gonk: dom/broadcastchannel. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/8a5ca86cb5d7 Remove test annotations using b2g, mulet or gonk: dom/browser-element. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/a6711dd19615 Remove test annotations using b2g, mulet or gonk: dom/cache. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/72f599526c3e Remove test annotations using b2g, mulet or gonk: dom/canvas. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/8b48318b227c Remove test annotations using b2g, mulet or gonk: dom/console. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/c94607d8ac87 Remove test annotations using b2g, mulet or gonk: dom/crypto. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/05a67089ffd2 Remove test annotations using b2g, mulet or gonk: dom/devicestorage. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/3a97d4394764 Remove test annotations using b2g, mulet or gonk: dom/encoding. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/f462dcc8b14d Remove test annotations using b2g, mulet or gonk: dom/events. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/fe49064ccca0 Remove test annotations using b2g, mulet or gonk: dom/html. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/888033db1e2e Remove test annotations using b2g, mulet or gonk: dom/imptests. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/bf145f0505bd Remove test annotations using b2g, mulet or gonk: dom/indexedDB. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/e077cd6f207c Remove test annotations using b2g, mulet or gonk: dom/inputmethod. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/eccf63d3738e Remove test annotations using b2g, mulet or gonk: dom/ipc. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/a7019a80d007 Remove test annotations using b2g, mulet or gonk: dom/json. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/0c132727ac31 Remove test annotations using b2g, mulet or gonk: dom/jsurl. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/3b3a7ce28a42 Remove test annotations using b2g, mulet or gonk: dom/media. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/66c24d153af8 Remove test annotations using b2g, mulet or gonk: dom/network. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/c2798e814069 Remove test annotations using b2g, mulet or gonk: dom/notification. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/d33f4926f6cf Remove test annotations using b2g, mulet or gonk: dom/permission. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/8649b3243a13 Remove test annotations using b2g, mulet or gonk: dom/plugins. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/bc97752afd9c Remove test annotations using b2g, mulet or gonk: dom/power. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/057a4086031c Remove test annotations using b2g, mulet or gonk: dom/presentation. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/756a94ac4754 Remove test annotations using b2g, mulet or gonk: dom/promise. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/656892c00905 Remove test annotations using b2g, mulet or gonk: dom/push. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/77b8d2222315 Remove test annotations using b2g, mulet or gonk: dom/security. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/4e9bd4063b15 Remove test annotations using b2g, mulet or gonk: dom/svg. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/3d25e5ad5459 Remove test annotations using b2g, mulet or gonk: dom/system. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/648493535386 Remove test annotations using b2g, mulet or gonk: dom/tests. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/8ada21f8ae7b Remove test annotations using b2g, mulet or gonk: dom/url. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/460ebcbf27cd Remove test annotations using b2g, mulet or gonk: dom/workers. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/17bfe7005aa7 Remove test annotations using b2g, mulet or gonk: dom/xbl. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/2b3b14097348 Remove test annotations using b2g, mulet or gonk: dom/xhr. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/abc467f5968e Remove test annotations using b2g, mulet or gonk: dom/xml. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/4a9b93ec7422 Remove test annotations using b2g, mulet or gonk: dom/xul. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/1d609a2887cd Remove test annotations using b2g, mulet or gonk: editor. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/563383b1a546 Remove test annotations using b2g, mulet or gonk: embedding. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/b68d70646ba1 Remove test annotations using b2g, mulet or gonk: extensions. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/3a43dbeb1c09 Remove test annotations using b2g, mulet or gonk: gfx. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/73f2d5cb1ebd Remove test annotations using b2g, mulet or gonk: image. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/76a3e514d699 Remove test annotations using b2g, mulet or gonk: intl. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/e337b15543f4 Remove test annotations using b2g, mulet or gonk: ipc. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/bf711dd4d4da Remove test annotations using b2g, mulet or gonk: js. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/05156801c3c1 Remove test annotations using b2g, mulet or gonk: layout/base. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/dd75620065a9 Remove test annotations using b2g, mulet or gonk: layout/forms. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/4dbb0f3fcb90 Remove test annotations using b2g, mulet or gonk: layout/generic. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/b62631078e5a Remove test annotations using b2g, mulet or gonk: layout/inspector. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/3b561c7b5e4f Remove test annotations using b2g, mulet or gonk: layout/style. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/a0244d81e456 Remove test annotations using b2g, mulet or gonk: layout/xul. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/60c84b2e1a0e Remove test annotations using b2g, mulet or gonk: modules. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/7f4ebf66b98e Remove test annotations using b2g, mulet or gonk: netwerk. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/18b940b7127b Remove test annotations using b2g, mulet or gonk: parser. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/1ebca799fd9c Remove test annotations using b2g, mulet or gonk: rdf. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/d5f6ee4455be Remove test annotations using b2g, mulet or gonk: security. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/47326eec64a8 Remove test annotations using b2g, mulet or gonk: services. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/c92eb49b6d37 Remove test annotations using b2g, mulet or gonk: storage. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/f82afee9057c Remove test annotations using b2g, mulet or gonk: toolkit. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/fc19521feaa5 Remove test annotations using b2g, mulet or gonk: uriloader. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/bfab724f8ed8 Remove test annotations using b2g, mulet or gonk: widget. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/7d0ab5434a3c Remove test annotations using b2g, mulet or gonk: xpcom. r=RyanVM https://hg.mozilla.org/integration/autoland/rev/f943477f8ad2 Remove test annotations using b2g, mulet or gonk: xpfe. r=RyanVM
Thank you for doing this!
https://hg.mozilla.org/mozilla-central/rev/7ef66a5e57ac https://hg.mozilla.org/mozilla-central/rev/c190459f9e03 https://hg.mozilla.org/mozilla-central/rev/2b55ec912ac5 https://hg.mozilla.org/mozilla-central/rev/89738aaf87c1 https://hg.mozilla.org/mozilla-central/rev/82ab05df35f7 https://hg.mozilla.org/mozilla-central/rev/1e3310aa319d https://hg.mozilla.org/mozilla-central/rev/365227cc8c6b https://hg.mozilla.org/mozilla-central/rev/9d136b70985b https://hg.mozilla.org/mozilla-central/rev/6809bf6acf2e https://hg.mozilla.org/mozilla-central/rev/63e0fce78144 https://hg.mozilla.org/mozilla-central/rev/3cf5abd5dede https://hg.mozilla.org/mozilla-central/rev/8a5ca86cb5d7 https://hg.mozilla.org/mozilla-central/rev/a6711dd19615 https://hg.mozilla.org/mozilla-central/rev/72f599526c3e https://hg.mozilla.org/mozilla-central/rev/8b48318b227c https://hg.mozilla.org/mozilla-central/rev/c94607d8ac87 https://hg.mozilla.org/mozilla-central/rev/05a67089ffd2 https://hg.mozilla.org/mozilla-central/rev/3a97d4394764 https://hg.mozilla.org/mozilla-central/rev/f462dcc8b14d https://hg.mozilla.org/mozilla-central/rev/fe49064ccca0 https://hg.mozilla.org/mozilla-central/rev/888033db1e2e https://hg.mozilla.org/mozilla-central/rev/bf145f0505bd https://hg.mozilla.org/mozilla-central/rev/e077cd6f207c https://hg.mozilla.org/mozilla-central/rev/eccf63d3738e https://hg.mozilla.org/mozilla-central/rev/a7019a80d007 https://hg.mozilla.org/mozilla-central/rev/0c132727ac31 https://hg.mozilla.org/mozilla-central/rev/3b3a7ce28a42 https://hg.mozilla.org/mozilla-central/rev/66c24d153af8 https://hg.mozilla.org/mozilla-central/rev/c2798e814069 https://hg.mozilla.org/mozilla-central/rev/d33f4926f6cf https://hg.mozilla.org/mozilla-central/rev/8649b3243a13 https://hg.mozilla.org/mozilla-central/rev/bc97752afd9c https://hg.mozilla.org/mozilla-central/rev/057a4086031c https://hg.mozilla.org/mozilla-central/rev/756a94ac4754 https://hg.mozilla.org/mozilla-central/rev/656892c00905 https://hg.mozilla.org/mozilla-central/rev/77b8d2222315 https://hg.mozilla.org/mozilla-central/rev/4e9bd4063b15 https://hg.mozilla.org/mozilla-central/rev/3d25e5ad5459 https://hg.mozilla.org/mozilla-central/rev/648493535386 https://hg.mozilla.org/mozilla-central/rev/8ada21f8ae7b https://hg.mozilla.org/mozilla-central/rev/460ebcbf27cd https://hg.mozilla.org/mozilla-central/rev/17bfe7005aa7 https://hg.mozilla.org/mozilla-central/rev/2b3b14097348 https://hg.mozilla.org/mozilla-central/rev/abc467f5968e https://hg.mozilla.org/mozilla-central/rev/4a9b93ec7422 https://hg.mozilla.org/mozilla-central/rev/1d609a2887cd https://hg.mozilla.org/mozilla-central/rev/563383b1a546 https://hg.mozilla.org/mozilla-central/rev/b68d70646ba1 https://hg.mozilla.org/mozilla-central/rev/3a43dbeb1c09 https://hg.mozilla.org/mozilla-central/rev/73f2d5cb1ebd https://hg.mozilla.org/mozilla-central/rev/76a3e514d699 https://hg.mozilla.org/mozilla-central/rev/e337b15543f4 https://hg.mozilla.org/mozilla-central/rev/bf711dd4d4da https://hg.mozilla.org/mozilla-central/rev/05156801c3c1 https://hg.mozilla.org/mozilla-central/rev/dd75620065a9 https://hg.mozilla.org/mozilla-central/rev/4dbb0f3fcb90 https://hg.mozilla.org/mozilla-central/rev/b62631078e5a https://hg.mozilla.org/mozilla-central/rev/3b561c7b5e4f https://hg.mozilla.org/mozilla-central/rev/a0244d81e456 https://hg.mozilla.org/mozilla-central/rev/60c84b2e1a0e https://hg.mozilla.org/mozilla-central/rev/7f4ebf66b98e https://hg.mozilla.org/mozilla-central/rev/18b940b7127b https://hg.mozilla.org/mozilla-central/rev/1ebca799fd9c https://hg.mozilla.org/mozilla-central/rev/d5f6ee4455be https://hg.mozilla.org/mozilla-central/rev/47326eec64a8 https://hg.mozilla.org/mozilla-central/rev/c92eb49b6d37 https://hg.mozilla.org/mozilla-central/rev/f82afee9057c https://hg.mozilla.org/mozilla-central/rev/fc19521feaa5 https://hg.mozilla.org/mozilla-central/rev/bfab724f8ed8 https://hg.mozilla.org/mozilla-central/rev/7d0ab5434a3c https://hg.mozilla.org/mozilla-central/rev/f943477f8ad2
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Epic. Thanks for doing this!
No longer blocks: 1369194
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: