Add proto for TLS error assistant, refactor proto generator code.

chrome/browser/resources/protobufs/binary_proto_generator.py

Index: chrome/browser/resources/protobufs/binary_proto_generator.py
diff --git a/chrome/browser/resources/protobufs/binary_proto_generator.py b/chrome/browser/resources/protobufs/binary_proto_generator.py
new file mode 100755
index 0000000000000000000000000000000000000000..acffbe035e0c1a79ec322a0412ba6dc5d02fa581
--- /dev/null
+++ b/chrome/browser/resources/protobufs/binary_proto_generator.py
@@ -0,0 +1,146 @@
+#!/usr/bin/python
+# 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.
+
+"""
+ Converts a given ASCII proto into a binary resource.
+
+"""
+
+import abc
+import optparse
+import os
+import re
+import subprocess
+import sys
+import traceback
+
+class BinaryProtoGenerator:
+
+  # If the script is run in a virtualenv
+  # (https://virtualenv.pypa.io/en/stable/), then no google.protobuf library
+  # should be brought in from site-packages. Passing -S into the interpreter in
+  # a virtualenv actually destroys the ability to import standard library
+  # functions like optparse, so this script should not be wrapped if we're in a
+  # virtualenv.
+  def _IsInVirtualEnv(self):
+    # This is the way used by pip and other software to detect virtualenv.
+    return hasattr(sys, 'real_prefix')
+
+  def _ImportProtoModules(self, paths):
+    """Import the protobuf modules we need. |paths| is list of import paths"""
+    print paths

[Nathan Parker, 2016/12/10 00:56:50]

I think you'll want to remove any "print"s since they'll show up in people's
ninja builds

[meacer, 2016/12/13 02:30:16]

Eeek, of course. Debugging artifact :)

+    for path in paths:
+      # Put the path to our proto libraries in front, so that we don't use
+      # system protobuf.
+      sys.path.insert(1, path)
+
+    import google.protobuf.text_format as text_format
+    globals()['text_format'] = text_format
+    self.ImportProtoModule()

[Nathan Parker, 2016/12/10 00:56:50]

What does this do? Is this so a child class can override?

[meacer, 2016/12/13 02:30:16]

Yes, it's an abstract method to be implemented by the child class.

+
+  def _GenerateBinaryProtos(self, opts):
+    """ Read the ASCII proto and generate one or more binary protos. """
+    # Read the ASCII
+    ifile = open(opts.infile, 'r')
+    ascii_pb_str = ifile.read()
+    ifile.close()
+
+    # Parse it into a structured PB
+    full_pb = self.EmptyProtoInstance()
+    text_format.Merge(ascii_pb_str, full_pb)
+
+    self.ValidatePb(opts, full_pb);
+    self.ProcessPb(opts, full_pb)
+
+  @abc.abstractmethod

[Nathan Parker, 2016/12/10 00:56:50]

I'm not familiar with @abc.. is this to ensure it doesn't get called w/o a real
impl?

[meacer, 2016/12/13 02:30:16]

abc stands for abstract base class, and yes the annotation is to ensure the
child class implements the method.

+  def ImportProtoModule(self):
+    """ Import the proto module to be used by the generator. """
+    pass
+
+  @abc.abstractmethod
+  def EmptyProtoInstance(self):
+    """ Returns an empty proto instance to be filled by the generator."""
+    pass
+
+  @abc.abstractmethod
+  def ValidatePb(self, opts, pb):
+    """ Validate the basic values of the protobuf.  The
+        file_type_policies_unittest.cc will also validate it by platform,
+        but this will catch errors earlier.
+    """
+    pass
+
+  @abc.abstractmethod
+  def ProcessPb(self, opts, pb):
+    """ Process the parsed prototobuf. """
+    pass
+
+  def AddCommandLineOptions(self, parser):
+    """ Allows subclasses to add any options the command line parser. """
+    pass
+
+  def AddExtraCommandLineArgsForVirtualEnvRun(self, opts, command):
+    """ Allows subclasses to add any extra command line arguments when running
+        this under a virtualenv."""
+    pass
+
+  def VerifyArgs(self, opts):
+    """ Allows subclasses to check command line parameters before running. """
+    return True
+
+  def Run(self):
+    parser = optparse.OptionParser()
+    # TODO(nparker): Remove this once the bug is fixed.

[Nathan Parker, 2016/12/10 00:56:50]

That bug is actually fixed, but we still need this work around (remove ref to
that bug).  I proposed a better fix in http://crbug.com/614082, which I'd be
glad for you to try. Could be in this CL, or a later one.

[meacer, 2016/12/13 02:30:16]

Updated the reference.

+    parser.add_option('-w', '--wrap', action="store_true", default=False,
+                      help='Wrap this script in another python '
+                      'execution to disable site-packages.  This is a '
+                      'fix for http://crbug.com/605592')
+
+    parser.add_option('-i', '--infile',
+                      help='The ASCII DownloadFileType-proto file to read.')

[Nathan Parker, 2016/12/10 00:56:50]

remove reference to the DownloadFileType file name.

[meacer, 2016/12/13 02:30:16]

Done.

+    parser.add_option('-d', '--outdir',
+                      help='Directory underwhich binary file(s) will be ' +
+                           'written')
+    parser.add_option('-o', '--outbasename',
+                      help='Basename of the binary file to write to.')
+    parser.add_option('-p', '--path', action="append",
+                      help='Repeat this as needed.  Directory(s) containing ' +
+                      'the known_certs_pb2.py and ' +

[Nathan Parker, 2016/12/10 00:56:50]

Maybe generalize the known_certs_pb2.py to "your_proto_definition_pb2.py" or
some such

[meacer, 2016/12/13 02:30:16]

Done.

+                      'google.protobuf.text_format modules')
+    self.AddCommandLineOptions(parser)
+
+    (opts, args) = parser.parse_args()
+    if opts.infile is None or opts.outdir is None or opts.outbasename is None:
+      parser.print_help()
+      return 1
+
+    if opts.wrap and not self._IsInVirtualEnv():
+      # Run this script again with different args to the interpreter to suppress
+      # the inclusion of libraries, like google.protobuf, from site-packages,
+      # which is checked before sys.path when resolving imports. We want to
+      # specifically import the libraries injected into the sys.path in
+      # ImportProtoModules().
+      command = [sys.executable, '-S', '-s', sys.argv[0]]
+      command += ['-i', opts.infile]
+      command += ['-d', opts.outdir]
+      command += ['-o', opts.outbasename]
+      for path in opts.path:
+        command += ['-p', path]
+
+      self.AddExtraCommandLineArgsForVirtualEnvRun(opts, command);
+      sys.exit(subprocess.call(command))
+
+    self._ImportProtoModules(opts.path)
+
+    if not self.VerifyArgs(opts):
+      print "Wrong arguments"
+      return 1
+
+    try:
+      self._GenerateBinaryProtos(opts)
+    except Exception as e:
+      print "ERROR: Failed to render binary version of %s:\n  %s\n%s" % (
+          opts.infile, str(e), traceback.format_exc())
+      return 1