[protobuf] Fix potential double definition build errors.

third_party/protobuf/BUILD.gn

 # Copyright 2014 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.
 
 config("protobuf_config") {
   include_dirs = [ "src" ]
   defines = [
     "GOOGLE_PROTOBUF_NO_RTTI",
     "GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER",
   ]
@@ skipping 18 matching lines @@
 
 # This config should be applied to targets using generated code from the proto
 # compiler. It sets up the include directories properly.
 config("using_proto") {
   include_dirs = [
     "src",
     "$root_gen_dir/protoc_out",
   ]
 }
 
+protobuf_globals_sources = [
+  "src/google/protobuf/globals.cc",
+  "src/google/protobuf/stubs/atomicops_internals_x86_gcc.cc",
+]
+
 protobuf_lite_sources = [
   "src/google/protobuf/arena.cc",
   "src/google/protobuf/arena.h",
   "src/google/protobuf/arenastring.cc",
   "src/google/protobuf/arenastring.h",
   "src/google/protobuf/extension_set.cc",
   "src/google/protobuf/extension_set.h",
   "src/google/protobuf/generated_message_util.cc",
   "src/google/protobuf/generated_message_util.h",
   "src/google/protobuf/io/coded_stream.cc",
@@ skipping 17 matching lines @@
   "src/google/protobuf/stubs/atomicops_internals_arm_qnx.h",
   "src/google/protobuf/stubs/atomicops_internals_atomicword_compat.h",
   "src/google/protobuf/stubs/atomicops_internals_generic_gcc.h",
   "src/google/protobuf/stubs/atomicops_internals_macosx.h",
   "src/google/protobuf/stubs/atomicops_internals_mips_gcc.h",
   "src/google/protobuf/stubs/atomicops_internals_pnacl.h",
   "src/google/protobuf/stubs/atomicops_internals_power.h",
   "src/google/protobuf/stubs/atomicops_internals_ppc_gcc.h",
   "src/google/protobuf/stubs/atomicops_internals_solaris.h",
   "src/google/protobuf/stubs/atomicops_internals_tsan.h",
-  "src/google/protobuf/stubs/atomicops_internals_x86_gcc.cc",
   "src/google/protobuf/stubs/atomicops_internals_x86_gcc.h",
   "src/google/protobuf/stubs/atomicops_internals_x86_msvc.cc",
   "src/google/protobuf/stubs/atomicops_internals_x86_msvc.h",
   "src/google/protobuf/stubs/atomic_sequence_num.h",
   "src/google/protobuf/stubs/bytestream.cc",
   "src/google/protobuf/stubs/bytestream.h",
   "src/google/protobuf/stubs/callback.h",
   "src/google/protobuf/stubs/casts.h",
   "src/google/protobuf/stubs/common.cc",
   "src/google/protobuf/stubs/common.h",
@@ skipping 66 matching lines @@
   # be shared across different copies of the library in each component.
   # protobuf_globals is a shared library that provides this state, but is
   # careful to prefix all exported symbols with 'cr_' so they don't conflict
   # with other versions of protobuf.
   component("protobuf_globals") {
     configs += [
       ":protobuf_config",
       ":protobuf_use_dlls",
     ]
     defines = [ "LIBPROTOBUF_EXPORTS" ]
-    sources = [
-      "src/google/protobuf/globals.cc",
-      "src/google/protobuf/stubs/atomicops_internals_x86_gcc.cc",
-    ]
+    sources = protobuf_globals_sources
   }
 }
 
 target(link_target_type, "protobuf_lite") {
   sources = protobuf_lite_sources
 
   configs -= [ "//build/config/compiler:chromium_code" ]
   configs += [
     "//build/config/compiler:no_chromium_code",
 
@@ skipping 15 matching lines @@
 
   deps = [
     "//build/config/sanitizers:deps",
   ]
 
   cflags = protobuf_lite_cflags
 
   if (is_component_build && is_linux && !is_chromeos) {
     deps += [ ":protobuf_globals" ]
   } else {
-    sources += [ "src/google/protobuf/globals.cc" ]
+    sources += protobuf_globals_sources
   }
 
   # Required for component builds. See http://crbug.com/172800.
   if (is_component_build && (!is_linux || is_chromeos)) {
     public_configs += [ ":protobuf_use_dlls" ]
     defines = [ "LIBPROTOBUF_EXPORTS" ]
   }
 }
 
 # This is the full, heavy protobuf lib that's needed for c++ .protos that don't
 # specify the LITE_RUNTIME option. The protocol compiler itself (protoc) falls
 # into that category. Do not use in Chrome code.
 static_library("protobuf_full") {
   # Prevent people from depending on this outside our file.
   visibility = [
     ":*",
 
     # requires descriptors & reflection; testonly.
     "//third_party/libprotobuf-mutator:*",
   ]
 
-  sources = protobuf_lite_sources
-  sources += [
+  sources = protobuf_lite_sources + protobuf_globals_sources + [
     "src/google/protobuf/any.cc",
     "src/google/protobuf/any.h",
     "src/google/protobuf/any.pb.cc",
     "src/google/protobuf/any.pb.h",
     "src/google/protobuf/api.pb.cc",
     "src/google/protobuf/api.pb.h",
     "src/google/protobuf/compiler/importer.cc",
     "src/google/protobuf/compiler/importer.h",
     "src/google/protobuf/compiler/parser.cc",
     "src/google/protobuf/compiler/parser.h",
     "src/google/protobuf/descriptor.cc",
     "src/google/protobuf/descriptor.h",
     "src/google/protobuf/descriptor.pb.cc",
     "src/google/protobuf/descriptor.pb.h",
     "src/google/protobuf/descriptor_database.cc",
     "src/google/protobuf/descriptor_database.h",
     "src/google/protobuf/duration.pb.cc",
     "src/google/protobuf/duration.pb.h",
     "src/google/protobuf/dynamic_message.cc",
     "src/google/protobuf/dynamic_message.h",
     "src/google/protobuf/empty.pb.cc",
     "src/google/protobuf/empty.pb.h",
     "src/google/protobuf/extension_set_heavy.cc",
     "src/google/protobuf/field_mask.pb.cc",
     "src/google/protobuf/field_mask.pb.h",
     "src/google/protobuf/generated_enum_reflection.h",
     "src/google/protobuf/generated_enum_util.h",
     "src/google/protobuf/generated_message_reflection.cc",
     "src/google/protobuf/generated_message_reflection.h",
-    "src/google/protobuf/globals.cc",
 
     # gzip_stream.cc pulls in zlib, but it's not actually used by protoc, just
     # by test code, so instead of compiling zlib for the host, let's just
     # exclude this.
     # "src/google/protobuf/io/gzip_stream.cc",
     # "src/google/protobuf/io/gzip_stream.h",
 
     "src/google/protobuf/io/printer.cc",
     "src/google/protobuf/io/printer.h",
     "src/google/protobuf/io/strtod.cc",
@@ skipping 420 matching lines @@
     ":copy_google_protobuf_internal",
     ":copy_six",
   ]
 
   # Targets that depend on this should depend on the copied data files.
   data = get_target_outputs(":copy_google")
   data += get_target_outputs(":copy_six")
   data += get_target_outputs(":copy_google_protobuf")
   data += get_target_outputs(":copy_google_protobuf_internal")
 }