So we don't disable them by accident.
Created attachment 646037 [details] [diff] [review]
Make sure we can't accidentally turn off mochitests for a directory by reordering the Makefile.in.
Can you go ahead and add the other MOCHITEST_* variables from mochitest.mk as well?
> Can you go ahead and add the other MOCHITEST_* variables from mochitest.mk as well?
Good catch. When I do, I get:
/Users/bzbarsky/mozilla/inbound/mozilla/config/rules.mk:1769: *** Makefile variable 'MOCHITEST_BROWSER_FILES' changed value after including rules.mk. Was '', now browser/head.js browser/crashreport.sjs browser/browser_aboutCrashes.js browser/browser_bug471404.js browser/browser_aboutCrashesResubmit.js .. Stop.
Backed out for:
(rest of push left in)
Sorry s/&onlyunstarred=1// in the previous comment's first URL.
https://hg.mozilla.org/integration/mozilla-inbound/rev/9085598907ef with the fix for that, which was apparently in an earlier patch in my queue.