third_party/protobuf: Update to HEAD (f52e188fe4)

third_party/protobuf/README.chromium

 Name: Protocol Buffers
 Short Name: protobuf
 URL: https://github.com/google/protobuf
 License: BSD
 License File: LICENSE
-Version: 3.0.0-beta-3
-Revision: 3470b6895aa659b7559ed678e029a5338e535f14
+Version: 3.1.0
+Revision: f52e188fe465c10377f5a2b5765f481cb6fcb2f4
 Security Critical: yes
 
 Steps used to create the current version:
 1. Pull the release from https://github.com/google/protobuf/releases
-2. Add build files (BUILD.gn, proto_library.gni, protobuf.gyp,
-   protobuf_lite.gypi, protobuf_nacl.gyp).
+2. Add build files (BUILD.gn, proto_library.gni).
 
    Be sure to update the list of source files, as additional .cc files and
    headers might have been added -- you need to find the transitive closure of
    include files required by targets.
 
    Other things to care about are defines required by protobuf on various
    platforms, warnings generated by compilers, and new dependencies introduced.
 3. Get open-source library six.py from https://pypi.python.org/pypi/six/ and add
    it to protobuf/third_party/six/six.py.
 4. Apply patches in patches/ (see the description below):
 
    $ for patch in patches/*; do patch -s -p1 < $patch; done
 
    For future releases, it will be worth looking into which patches still need
-   to be applied.
-5. Generate descriptor_pb2.py using something like the following steps. Make
-   sure you've regenerated your buildfiles and will build protoc from the
-   newly-modified sources above.
+   to be applied. In case of conflict, update those patches accordingly and save
+   them back in place (i.e. in patches directory).
+5. Generate well_known_types_embed.cc and descriptor_pb2.py using something like
+   the following steps. Make sure you've regenerated your buildfiles and will
+   build js_embed then protoc from the newly-modified sources above.
 
    $ cd $SRC_DIR
+   $ gn gen out/Debug
+   $ ninja -C out/Debug js_embed
+   $ cd third_party/protobuf/src/google/protobuf/compiler/js
+   $ $SRC_DIR/out/Debug/js_embed                                    \
+        ./well_known_types/any.js ./well_known_types/struct.js      \
+        ./well_known_types/timestamp.js > well_known_types_embed.cc
+   $ cd $SRC_DIR
    $ ninja -C out/Debug protoc
    $ cd third_party/protobuf/src
-   $ ../../../out/Debug/protoc --python_out=../python google/protobuf/descriptor.proto
+   $ $SRC_DIR/out/Debug/protoc --python_out=../python google/protobuf/descriptor.proto

[Peter Kasting, 2016/12/23 10:51:42]

Not for this patch: I'm pretty sure we can set up the build files to generate
all these files into the directory we use for other "generated output" files,
then use them when building libprotobuf.  In other words, we can probably get to
the point eventually where we don't check these files in or have steps to
generate them here, but instead do it on developer machines during normal build
time.

But don't look into that right now -- this patch has been stalled on enough
things.  Either do that after this lands, or file a bug to look into it.

[jbriance, 2016/12/23 12:15:35]

Acknowledged.

 
 6. Add an __init__.py to protobuf/ that adds third_party/six/ to Python path.
 7. Update README.chromium.
 
 Description of the patches:
 
 - 0003-remove-static-initializers.patch
 
   This patch removes all static initializers from Chromium. The change in Status
   class is not completely compatible with upstream, but it's compatible enough
   to work in Chromium, which doesn't use this functionality yet. The work on
   upstreaming the removal of static initializers is in progress:
   https://github.com/google/protobuf/issues/1404
 
 - 0004-fix-integer-types-and-shared-library-exports.patch
 
   This patch makes protobuf int64 to be int64_t (as opposed to long long in
   upstream), and similarly for other integer types. It also allows exporting
   protobuf symbols in Linux .so libraries, so that protobuf can be built as a
   component (see http://crrev.com/179806).
 
-- 0005-fix-include-js-generator.protobuf
-
-  During merge with internal branch, the <> in one of the #includes were
-  accidentally replaced with "", which results in a failure in checkdeps.
-
-  Fixed in https://github.com/google/protobuf/pull/1547.
-
 - 0007-uninline_googleonce.patch
 - 0008-uninline_get_empty_string.patch
 - 0009-uninline-arenastring.patch
 - 0010-uninline-generated-code.patch
 
   These patches uninline some functions, resulting in a significant reduction
   (somewhere between 500 KB and 1 MB) of binary size.
 
-- 0011-libprotobuf_export.patch
+- 0012-fixes-for-js-embed-cc.patch
 
-  During merge with internal branch, the dll export attribute was accidentally
-  removed in C++11 mode.
-
-  Fixed in https://github.com/google/protobuf/pull/1549
+  Fix include style for src/google/protobuf/compiler/js/well_known_types_embed.cc

[Peter Kasting, 2016/12/23 10:51:42]

80 columns

[jbriance, 2016/12/23 12:15:35]

Done in patch set 2.

+  (genereated in step 5, see above) and also remote it from .gitignore file as we

[Peter Kasting, 2016/12/23 10:51:43]

Nit: remote -> remove

[jbriance, 2016/12/23 12:15:35]

Done in patch set 2.

+  do not generate it in on the fly.