This has caused problems in the past for Android builds (see bug 1031874) since the define is specifically for the application update component and should not be re-purposed for other uses.

Note: I cleaned up the build system from the toolkit end of things in bug 1152997. When this bug is fixed |#if defined(MOZ_UPDATER) && !defined(MOZ_WIDGET_ANDROID)| etc. that was added in bug 1152997 can be changed to just |if defined(MOZ_UPDATER)| etc.

What usage I see in mobile/ now looks appropriate: https://searchfox.org/mozilla-central/search?q=MOZ_UPDATER&path=mobile.

This has caused breakage when landing in the past as well as compiling code not used by mobile because it has two different meanings in common code due to the re-purposing by mobile. One example https://dxr.mozilla.org/mozilla-central/source/toolkit/xre/nsAppRunner.cpp#27

(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #3) > This has caused breakage when landing in the past as well as compiling code > not used by mobile because it has two different meanings in common code due > to the re-purposing by mobile. One example > https://dxr.mozilla.org/mozilla-central/source/toolkit/xre/nsAppRunner.cpp#27 Thanks for the clarification, Robert. I saw no shady uses in mobile/ but I see now what you mean.