IDL compiler: Rename [ActivityLogging=Access*] => [ActivityLogging=*]

Source/bindings/scripts/unstable/v8_utilities.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 96 matching lines @@
 
 # [ActivityLogging]
 def activity_logging_world_list(member, access_type=None):
     """Returns a set of world suffixes for which a definition member has activity logging, for specified access type.
 
     access_type can be 'Getter' or 'Setter' if only checking getting or setting.
     """
     if 'ActivityLogging' not in member.extended_attributes:
         return set()
     activity_logging = member.extended_attributes['ActivityLogging']
-    # [ActivityLogging=Access*] means log for all access, otherwise check that
-    # value agrees with specified access_type.
-    has_logging = (activity_logging.startswith('Access') or
+    # [ActivityLogging=For*] (no prefix, starts with the worlds suffix) means
+    # "log for all use (method)/access (attribute)", otherwise check that value
+    # agrees with specified access_type (Getter/Setter).
+    has_logging = (activity_logging.startswith('For') or
                    (access_type and activity_logging.startswith(access_type)))
     if not has_logging:
         return set()
     includes.add('bindings/v8/V8DOMActivityLogger.h')
     if activity_logging.endswith('ForIsolatedWorlds'):
         return set([''])
-    if activity_logging.endswith('ForAllWorlds'):
-        return set(['', 'ForMainWorld'])
-    raise 'Unrecognized [ActivityLogging] value: "%s"' % activity_logging

[Nils Barth (inactive), 2013/11/05 10:56:13]

This check and exception are unneeded, b/c already covered by
IDL extended attribute validator.
(Remove for consistency and simplicity.)

+    return set(['', 'ForMainWorld'])  # endswith('ForAllWorlds')
 
 
 # [CallWith]
 CALL_WITH_ARGUMENTS = {
     'ScriptState': '&state',
     'ExecutionContext': 'scriptContext',
     'ScriptArguments': 'scriptArguments.release()',
     'ActiveWindow': 'activeDOMWindow()',
     'FirstWindow': 'firstDOMWindow()',
 }
@@ skipping 67 matching lines @@
 def cpp_name(definition_or_member):
     return definition_or_member.extended_attributes.get('ImplementedAs', definition_or_member.name)
 
 
 # [MeasureAs]
 def generate_measure_as(definition_or_member, contents):
     if 'MeasureAs' not in definition_or_member.extended_attributes:
         return
     contents['measure_as'] = definition_or_member.extended_attributes['MeasureAs']
     includes.add('core/page/UseCounter.h')