Switch from xml.dom.minidom to xml.etree

gclient_utils.py

 # Copyright (c) 2010 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.
 
 """Generic utils."""
 
 import errno
 import logging
 import os
 import Queue
 import re
 import stat
 import subprocess
 import sys
 import threading
 import time
-import xml.dom.minidom
-import xml.parsers.expat
 
 
 def hack_subprocess():
   """subprocess functions may throw exceptions when used in multiple threads.
 
   See http://bugs.python.org/issue1731717 for more information.
   """
   subprocess._cleanup = lambda: None
 
 
@@ skipping 78 matching lines @@
     # Make sure ssh://user-name@example.com/~/test.git@stable works
     regex = r'(ssh://(?:[-\w]+@)?[-\w:\.]+/[-~\w\./]+)(?:@(.+))?'
     components = re.search(regex, url).groups()
   else:
     components = url.split('@', 1)
     if len(components) == 1:
       components += [None]
   return tuple(components)
 
 
-def ParseXML(output):
-  try:
-    return xml.dom.minidom.parseString(output)
-  except xml.parsers.expat.ExpatError:
-    return None
-
-
-def GetNamedNodeText(node, node_name):
-  child_nodes = node.getElementsByTagName(node_name)
-  if not child_nodes:
-    return None
-  assert len(child_nodes) == 1 and child_nodes[0].childNodes.length == 1
-  return child_nodes[0].firstChild.nodeValue
-
-
-def GetNodeNamedAttributeText(node, node_name, attribute_name):
-  child_nodes = node.getElementsByTagName(node_name)
-  if not child_nodes:
-    return None
-  assert len(child_nodes) == 1
-  return child_nodes[0].getAttribute(attribute_name)
-
-
 def SyntaxErrorToError(filename, e):
   """Raises a gclient_utils.Error exception with the human readable message"""
   try:
     # Try to construct a human readable error message
     if filename:
       error_message = 'There is a syntax error in %s\n' % filename
     else:
       error_message = 'There is a syntax error\n'
     error_message += 'Line #%s, character %s: "%s"' % (
         e.lineno, e.offset, re.sub(r'[\r\n]*$', '', e.text))
@@ skipping 553 matching lines @@
         logging.info('Caught exception in thread %s' % self.item.name)
         logging.info(str(sys.exc_info()))
         work_queue.exceptions.put(sys.exc_info())
       logging.info('Task %s done' % self.item.name)
 
       work_queue.ready_cond.acquire()
       try:
         work_queue.ready_cond.notifyAll()
       finally:
         work_queue.ready_cond.release()