Sync up two accessibility enum files: ax_enums.idl and automation.idl

ui/accessibility/PRESUBMIT.py

Index: ui/accessibility/PRESUBMIT.py
diff --git a/ui/accessibility/PRESUBMIT.py b/ui/accessibility/PRESUBMIT.py
new file mode 100644
index 0000000000000000000000000000000000000000..ef41ab093348f5b1dd6d8f57890d00785c688324
--- /dev/null
+++ b/ui/accessibility/PRESUBMIT.py
@@ -0,0 +1,95 @@
+# Copyright 2016 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.
+
+"""Presubmit script for ui/accessibility."""
+
+import os, re
+
+AX_IDL = 'ui/accessibility/ax_enums.idl'
+AUTOMATION_IDL = 'chrome/common/extensions/api/automation.idl'

[David Tseng, 2016/08/31 19:39:29]

Thanks for doing this. Can we check that anyone who touched automation.idl also
makes changes to
chrome/renderer/resources/extensions/automation/automation_node.js? (perhaps
just a warning).

[dmazzoni, 2016/08/31 19:51:42]

Sounds like that'd be a good follow-up. I don't want to warn
for any change to automation.idl because, for example, a change to
an enum requires a change to automation.idl but not to
automation_node.js

[David Tseng, 2016/08/31 20:51:46]

How about just changes to ax_enums.idl?

 I mention it because a change to an enum could very well result in a change to
automation_node.js. Without it, the Automation API never installs the property
getter for an attribute since we keep a hard coded list of attributes there,
too. That list, is a subset of the one in the idl, but should just be
everything. For example, indirect children might be useful for ChromeVox table
navigation, but currently isn't listed in automation_node.js.

To be clear:
- automation.idl: used for documentation (published to Chrome Extension API
docs), also used for installing the bindings chrome.automation.[Event|Role]Type
- automation_node.js: lists the properties that get exposed on an instance of
automation node
- ax_enums.idl: the authorative list of enums
- chrome_extension_externs.js: used in Closure compilation upload presubmit
check

I think we need all four to be in sync or should think about combining them
eventually.

+
+def InitialLowerCamelCase(unix_name):
+  words = unix_name.split('_')
+  return words[0] + ''.join(word.capitalize() for word in words[1:])
+
+# Given a full path to an IDL file containing enum definitions,
+# parse the file for enums and return a dict mapping the enum name
+# to a list of values for that enum.
+def GetEnumsFromFile(fullpath):
+  enum_name = None
+  enums = {}
+  for line in open(fullpath).readlines():
+    # Strip out comments
+    line = re.sub('//.*', '', line)
+
+    # Look for lines of the form "enum ENUM_NAME {" and get the enum_name
+    m = re.search('enum ([\w]+) {', line)
+    if m:
+      enum_name = m.group(1)
+      continue
+
+    # Look for a "}" character signifying the end of an enum
+    if line.find('}') >= 0:
+      enum_name = None
+      continue
+
+    if not enum_name:
+      continue
+
+    # If we're inside an enum definition, add the first string consisting of
+    # alphanumerics plus underscore ("\w") to the list of values for that enum.
+    m = re.search('([\w]+)', line)
+    if m:
+      enums.setdefault(enum_name, [])
+      enums[enum_name].append(m.group(1))
+
+  return enums
+
+def CheckMatchingEnum(ax_enums,
+                      ax_enum_name,
+                      automation_enums,
+                      automation_enum_name,
+                      errs,
+                      output_api):
+  if ax_enum_name not in ax_enums:
+    errs.append(output_api.PresubmitError(
+        'Expected %s to have an enum named %s' % (AX_IDL, ax_enum_name)))
+    return
+  if automation_enum_name not in automation_enums:
+    errs.append(output_api.PresubmitError(
+        'Expected %s to have an enum named %s' % (
+            AUTOMATION_IDL, automation_enum_name)))
+    return
+  src = ax_enums[ax_enum_name]
+  dst = automation_enums[automation_enum_name]
+  for value in src:
+    if InitialLowerCamelCase(value) not in dst:
+      errs.append(output_api.PresubmitError(
+          'Found %s.%s in %s, but did not find %s.%s in %s' % (
+              ax_enum_name, value, AX_IDL,
+              automation_enum_name, InitialLowerCamelCase(value),
+              AUTOMATION_IDL)))
+
+def CheckEnumsMatch(input_api, output_api):
+  repo_root = input_api.change.RepositoryRoot()
+  ax_enums = GetEnumsFromFile(os.path.join(repo_root, AX_IDL))
+  automation_enums = GetEnumsFromFile(os.path.join(repo_root, AUTOMATION_IDL))
+  errs = []
+  CheckMatchingEnum(ax_enums, 'AXRole', automation_enums, 'RoleType', errs,
+                    output_api)
+  CheckMatchingEnum(ax_enums, 'AXState', automation_enums, 'StateType', errs,
+                    output_api)
+  CheckMatchingEnum(ax_enums, 'AXEvent', automation_enums, 'EventType', errs,
+                    output_api)
+  return errs
+
+def CheckChangeOnUpload(input_api, output_api):
+  if AX_IDL not in input_api.LocalPaths():
+    return []
+  return CheckEnumsMatch(input_api, output_api)
+
+def CheckChangeOnCommit(input_api, output_api):
+  if AX_IDL not in input_api.LocalPaths():
+    return []
+  return CheckEnumsMatch(input_api, output_api)