Move API functions registrations out of ExtensionFunctionRegistry.

tools/json_schema_compiler/cpp_util.py

 # 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.
 
 """Utilies and constants specific to Chromium C++ code.
 """
 
 from datetime import datetime
 from model import Property, PropertyType, Type
 import os
+import re
 
 CHROMIUM_LICENSE = (
 """// Copyright (c) %d 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.""" % datetime.now().year
 )
 GENERATED_FILE_MESSAGE = """// GENERATED FROM THE API DEFINITION IN
 //   %s
 // DO NOT EDIT.
 """
 GENERATED_BUNDLE_FILE_MESSAGE = """// GENERATED FROM THE API DEFINITIONS IN
 //   %s
 // DO NOT EDIT.
 """
 
 def Classname(s):
   """Translates a namespace name or function name into something more
   suited to C++.
 
   eg experimental.downloads -> Experimental_Downloads
   updateAll -> UpdateAll.
   """
-  return '_'.join([x[0].upper() + x[1:] for x in s.split('.')])
+  new_s = re.sub(r'[^a-zA-Z0-9]', '.',s)
+  return '_'.join([x[0].upper() + x[1:] for x in re.split('\.', new_s)])

[Yoyo Zhou, 2013/02/05 23:42:13]

No, the point of using re.split is that you don't need to use re.sub. Please see
the documentation on re.split in the Python manual.

[SanjoyPal, 2013/02/06 00:47:21]

Done.

 
 def GetAsFundamentalValue(type_, src, dst):
   """Returns the C++ code for retrieving a fundamental type from a
   Value into a variable.
 
   src: Value*
   dst: Property*
   """
   return {
       PropertyType.STRING: '%s->GetAsString(%s)',
@@ skipping 40 matching lines @@
   e.g chrome/extensions/gen, file.h becomes CHROME_EXTENSIONS_GEN_FILE_H__.
   """
   return (('%s_%s_H__' % (path, filename))
           .upper().replace(os.sep, '_').replace('/', '_'))
 
 def PadForGenerics(var):
   """Appends a space to |var| if it ends with a >, so that it can be compiled
   within generic types.
   """
   return ('%s ' % var) if var.endswith('>') else var