Allow linting of js files without including them in the build

Change-Id: I1955c6a95c14396b3c840904472d42daf582cdf7

clear_xpcom_vars.mk

@@ -6,6 +6,7 @@ LOCAL_INSTALLED_XPCOM_IDLS :=
 LOCAL_INSTALLED_XPCOM_IDL_HEADERS :=
 LOCAL_JS_NOTICE :=
 LOCAL_JS_SRC_FILES :=
+LOCAL_JS_LINTONLY_FILES :=
 LOCAL_XPCOM_IDLS :=
 LOCAL_XPCOM_INSTALL_DIR :=
 LOCAL_XPCOM_MODULE_OBJDIR :=

xpcom.mk

@@ -201,6 +201,15 @@ endif
 	$(ACP) $< $@
 endif
 
+JS_LINTONLY_OK_FILES := $(addsuffix .ok, $(addprefix $(LOCAL_XPCOM_MODULE_OBJDIR)/,$(LOCAL_JS_LINTONLY_FILES)))
+
+$(JS_LINTONLY_OK_FILES): $(LOCAL_XPCOM_MODULE_OBJDIR)/%.js.ok: $(LOCAL_PATH)/%.js
+ifdef GJSLINT
+	$(GJSLINT) $<
+endif
+	@mkdir -p $(@D)
+	touch $@
+
 LOCAL_ADDITIONAL_INSTALL_DEPENDENCIES += $(LOCAL_MODULE_PATH)/chrome.manifest
 
 $(LOCAL_MODULE_PATH)/chrome.manifest: $(LOCAL_PATH)/chrome.manifest $(BUILDLIST_PY) $(ACP)
@@ -216,4 +225,4 @@ include $(BUILD_SHARED_LIBRARY)
 
 $(all_objects): $(DEPENDS_ON_GECKO) $(LOCAL_INSTALLED_XPCOM_IDL_HEADERS) $(LOCAL_INSTALLED_REQUIRED_XPCOM_IDL_HEADERS)
 
-$(LOCAL_INSTALLED_MODULE): $(LOCAL_ADDITIONAL_INSTALL_DEPENDENCIES)
+$(LOCAL_INSTALLED_MODULE): $(LOCAL_ADDITIONAL_INSTALL_DEPENDENCIES) $(JS_LINTONLY_OK_FILES)