Consolidate adding common java properties

Use a method to add the properties that are present on all java modules.

Bug: 153485543
Test: m checkbuild
Change-Id: I7803b15eb0de810c8ab8d4b9acf2511935a26fb6

java/aar.go

@@ -506,11 +506,8 @@ func (a *AndroidLibrary) GenerateAndroidBuildActions(ctx android.ModuleContext)
 func AndroidLibraryFactory() android.Module {
 	module := &AndroidLibrary{}
 
+	module.Module.addHostAndDeviceProperties()
 	module.AddProperties(
-		&module.Module.properties,
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
 		&module.aaptProperties,
 		&module.androidLibraryProperties)
 

java/app.go

@@ -963,11 +963,8 @@ func AndroidAppFactory() android.Module {
 	module.Module.properties.Instrument = true
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 
+	module.addHostAndDeviceProperties()
 	module.AddProperties(
-		&module.Module.properties,
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
 		&module.aaptProperties,
 		&module.appProperties,
 		&module.overridableAppProperties,
@@ -1083,11 +1080,8 @@ func AndroidTestFactory() android.Module {
 	module.appProperties.AlwaysPackageNativeLibs = true
 	module.Module.dexpreopter.isTest = true
 
+	module.addHostAndDeviceProperties()
 	module.AddProperties(
-		&module.Module.properties,
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
 		&module.aaptProperties,
 		&module.appProperties,
 		&module.appTestProperties,
@@ -1135,11 +1129,8 @@ func AndroidTestHelperAppFactory() android.Module {
 	module.appProperties.AlwaysPackageNativeLibs = true
 	module.Module.dexpreopter.isTest = true
 
+	module.addHostAndDeviceProperties()
 	module.AddProperties(
-		&module.Module.properties,
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
 		&module.aaptProperties,
 		&module.appProperties,
 		&module.appTestHelperAppProperties,

java/java.go

@@ -481,6 +481,21 @@ type Module struct {
 	modulePaths []string
 }
 
+func (j *Module) addHostProperties() {
+	j.AddProperties(
+		&j.properties,
+		&j.protoProperties,
+	)
+}
+
+func (j *Module) addHostAndDeviceProperties() {
+	j.addHostProperties()
+	j.AddProperties(
+		&j.deviceProperties,
+		&j.dexpreoptProperties,
+	)
+}
+
 func (j *Module) OutputFiles(tag string) (android.Paths, error) {
 	switch tag {
 	case "":
@@ -2044,12 +2059,8 @@ var javaHeaderLibsSdkMemberType android.SdkMemberType = &librarySdkMemberType{
 func LibraryFactory() android.Module {
 	module := &Library{}
 
-	module.AddProperties(
+	module.addHostAndDeviceProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.libraryProperties)
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
-		&module.libraryProperties)
 
 	module.initModuleAndImport(&module.ModuleBase)
 
@@ -2071,9 +2082,7 @@ func LibraryStaticFactory() android.Module {
 func LibraryHostFactory() android.Module {
 	module := &Library{}
 
-	module.AddProperties(
+	module.addHostProperties()
-		&module.Module.properties,
-		&module.Module.protoProperties)
 
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 
@@ -2241,12 +2250,8 @@ func (p *testSdkMemberProperties) AddToPropertySet(ctx android.SdkMemberContext,
 func TestFactory() android.Module {
 	module := &Test{}
 
-	module.AddProperties(
+	module.addHostAndDeviceProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.testProperties)
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
-		&module.testProperties)
 
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 	module.Module.dexpreopter.isTest = true
@@ -2259,12 +2264,8 @@ func TestFactory() android.Module {
 func TestHelperLibraryFactory() android.Module {
 	module := &TestHelperLibrary{}
 
-	module.AddProperties(
+	module.addHostAndDeviceProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.testHelperLibraryProperties)
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
-		&module.testHelperLibraryProperties)
 
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 	module.Module.dexpreopter.isTest = true
@@ -2305,10 +2306,8 @@ func JavaTestImportFactory() android.Module {
 func TestHostFactory() android.Module {
 	module := &Test{}
 
-	module.AddProperties(
+	module.addHostProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.testProperties)
-		&module.Module.protoProperties,
-		&module.testProperties)
 
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 
@@ -2392,12 +2391,8 @@ func (j *Binary) DepsMutator(ctx android.BottomUpMutatorContext) {
 func BinaryFactory() android.Module {
 	module := &Binary{}
 
-	module.AddProperties(
+	module.addHostAndDeviceProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.binaryProperties)
-		&module.Module.deviceProperties,
-		&module.Module.dexpreoptProperties,
-		&module.Module.protoProperties,
-		&module.binaryProperties)
 
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 
@@ -2413,10 +2408,8 @@ func BinaryFactory() android.Module {
 func BinaryHostFactory() android.Module {
 	module := &Binary{}
 
-	module.AddProperties(
+	module.addHostProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.binaryProperties)
-		&module.Module.protoProperties,
-		&module.binaryProperties)
 
 	module.Module.properties.Installable = proptools.BoolPtr(true)
 

java/plugin.go

@@ -24,10 +24,8 @@ func init() {
 func PluginFactory() android.Module {
 	module := &Plugin{}
 
-	module.AddProperties(
+	module.addHostProperties()
-		&module.Module.properties,
+	module.AddProperties(&module.pluginProperties)
-		&module.Module.protoProperties,
-		&module.pluginProperties)
 
 	InitJavaModule(module, android.HostSupported)
 	return module

java/robolectric.go

@@ -215,10 +215,9 @@ func (r *robolectricTest) writeTestRunner(w io.Writer, module, name string, test
 func RobolectricTestFactory() android.Module {
 	module := &robolectricTest{}
 
+	module.addHostProperties()
 	module.AddProperties(
-		&module.Module.properties,
 		&module.Module.deviceProperties,
-		&module.Module.protoProperties,
 		&module.robolectricProperties)
 
 	module.Module.dexpreopter.isTest = true

java/sdk_library.go

@@ -1535,13 +1535,8 @@ func (module *SdkLibrary) CreateInternalModules(mctx android.DefaultableHookCont
 }
 
 func (module *SdkLibrary) InitSdkLibraryProperties() {
-	module.AddProperties(
+	module.addHostAndDeviceProperties()
-		&module.sdkLibraryProperties,
+	module.AddProperties(&module.sdkLibraryProperties)
-		&module.properties,
-		&module.dexpreoptProperties,
-		&module.deviceProperties,
-		&module.protoProperties,
-	)
 
 	module.initSdkLibraryComponent(&module.ModuleBase)