The file testing/mochitest/tests/browser/Makefile.in has some commented-out lines at the end of its test lists, before the final $(NULL)
IIRC, any commented-out lines in the middle of a Makefile's list of mochitests will end the list prematurely, right at that point -- so really we want the list-terminating $(NULL) to be *before* the commented-out lines.
(In this case it doesn't really matter because there aren't any actual tests after the commented-out chunk; still, we might as well fix this in case any tests are added/uncommented later on, and for style/consistency/correctness points).
Created attachment 592431 [details] [diff] [review]
Tagging gavin for r?, since his name's at the top of the file.
Comment on attachment 592431 [details] [diff] [review]
I left it this way because it makes it easier to re-enable those tests (you don't need to move the $(NULL) around), and AFAICT there is no practical impact. I guess people don't do that enough for it to matter.
> AFAICT there is no practical impact
None right now, correct -- but if someone were to add some more tests between the commented-out chunk and the $(NULL) line, e.g. to maintain alphabetical ordering or consistency or something, then it'd be bad - those added tests would never be run. So I think best-practice is to have $(NULL) before the commented-out lines, to make it clear where the list actually ends.
Thanks for the review! Landed: