Split onResourcesLoaded into multiple functions.

build/android/gyp/process_resources.py

 #!/usr/bin/env python
 #
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Process Android resources to generate R.java, and prepare for packaging.
 
 This will crunch images and generate v14 compatible resources
 (see generate_v14_compatible_resources.py).
@@ skipping 187 matching lines @@
 def _CreateRJavaFile(package, resources_by_type, shared_resources):
   """Generates the contents of a R.java file."""
   # Keep these assignments all on one line to make diffing against regular
   # aapt-generated files easier.
   create_id = ('{{ e.resource_type }}.{{ e.name }} = '
                '({{ e.resource_type }}.{{ e.name }} & 0x00ffffff) |'
                ' (packageId << 24);')
   create_id_arr = ('{{ e.resource_type }}.{{ e.name }}[i] = '
                    '({{ e.resource_type }}.{{ e.name }}[i] & 0x00ffffff) |'
                    ' (packageId << 24);')
+  # Here we diverge from what aapt does. Because we have so many
+  # resources, the onResourcesLoaded method was exceeding the 64KB limit that
+  # Java imposes. For this reason we split onResourcesLoaded into different
+  # methods for each resource type.
   template = Template("""/* AUTO-GENERATED FILE.  DO NOT MODIFY. */
 
 package {{ package }};
 
 public final class R {
     {% for resource_type in resource_types %}
     public static final class {{ resource_type }} {
         {% for e in resources[resource_type] %}
         {% if shared_resources %}
         public static {{ e.java_type }} {{ e.name }} = {{ e.value }};
         {% else %}
         public static final {{ e.java_type }} {{ e.name }} = {{ e.value }};
         {% endif %}
         {% endfor %}
     }
     {% endfor %}
     {% if shared_resources %}
     public static void onResourcesLoaded(int packageId) {
         {% for resource_type in resource_types %}
-        {% for e in resources[resource_type] %}
-        {% if resource_type != 'styleable' and e.java_type != 'int[]' %}
-        """ + create_id + """
-        {% endif %}
-        {% endfor %}
+        onResourcesLoaded{{ resource_type|title }}(packageId);
         {% for e in resources[resource_type] %}
         {% if e.java_type == 'int[]' %}
         for(int i = 0; i < {{ e.resource_type }}.{{ e.name }}.length; ++i) {
             """ + create_id_arr + """
         }
         {% endif %}
         {% endfor %}
         {% endfor %}
     }
+    {% for res_type in resource_types %}
+    private static void onResourcesLoaded{{ res_type|title }}(int packageId) {
+        {% for e in resources[res_type] %}
+        {% if res_type != 'styleable' and e.java_type != 'int[]' %}
+        """ + create_id + """
+        {% endif %}
+        {% endfor %}
+    }
+    {% endfor %}
     {% endif %}
 }
 """, trim_blocks=True, lstrip_blocks=True)
 
   return template.render(package=package,
                          resources=resources_by_type,
                          resource_types=sorted(resources_by_type),
                          shared_resources=shared_resources)
 
 
@@ skipping 272 matching lines @@
       options,
       input_paths=input_paths,
       input_strings=input_strings,
       output_paths=output_paths,
       # TODO(agrieve): Remove R_dir when it's no longer used (used only by GYP).
       force=options.R_dir)
 
 
 if __name__ == '__main__':
   main(sys.argv[1:])