Revert of Use bundled harfbuzz on Linux (patchset #3 id:40001 of https://codereview.chromium.org/1197683002/)

Reason for revert:
http://crbug.com/503858 says there are some situation where chrome fails to boot. I'll revert this once to investigate further.

Original issue's description:
> Use bundled harfbuzz on Linux
>
> crbug.com/353127 changed to use the bundled harfbuzz for official builds
> on Linux, but not for non-official Linux builds due to link warnings and
> gfx_unittests crash on Trusty.
>
> This patch changes non-official Linux builds to use the bundled harfbuzz.
> Symbols from the bundled harfbuzz-ng are exported in non-official builds
> so that pangoft2 can see them. gfx_unittests crashes are no longer
> observed.
>
> BUG=462689, 353127
>
> Committed: https://crrev.com/a2940327975938a0da0c6f8a49ba26fd78ed8d51
> Cr-Commit-Position: refs/heads/master@{#335865}

TBR=eae@chromium.org,drott@chromium.org,behdad@chromium.org,behdad@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=462689, 353127

Review URL: https://codereview.chromium.org/1201253004

Cr-Commit-Position: refs/heads/master@{#335884}

2 files changed, 16 insertions, 18 deletions

diff --git a/third_party/harfbuzz-ng/BUILD.gn b/third_party/harfbuzz-ng/BUILD.gn
index 24c6927..2ce1280 100644
--- a/third_party/harfbuzz-ng/BUILD.gn
+++ b/third_party/harfbuzz-ng/BUILD.gn
@@ -16,15 +16,18 @@ import("//build/config/linux/pkg_config.gni")
 declare_args() {
   # Since version 1.31.0, pangoft2 which we depend on pulls in harfbuzz
   # anyways. However, we want to have control of the version of harfbuzz
-  # we use, so don't use system harfbuzz unless we are building for
-  # chrome os, where we have the system harfbuzz under control.
-  use_system_harfbuzz =
-      is_linux && is_chromeos && exec_script(pkg_config_script,
-                                             pkg_config_args + [
-                                                   "--atleast-version=1.31.0",
-                                                   "pangoft2",
-                                                 ],
-                                             "value")
+  # we use, so don't use system harfbuzz for official builds, unless we
+  # are building for chrome os, where we have the system harfbuzz under
+  # control as well. We also do not want to use the system harfbuzz
+  # when cross-compiling for ARM.
+  use_system_harfbuzz = is_linux && (is_chromeos || (!is_official_build &&
+                                                     current_cpu != "arm")) &&
+                        exec_script(pkg_config_script,
+                                    pkg_config_args + [
+                                          "--atleast-version=1.31.0",
+                                          "pangoft2",
+                                        ],
+                                    "value")
 }
 
 if (use_system_harfbuzz) {
@@ -163,8 +166,5 @@ if (use_system_harfbuzz) {
       ]
       defines += [ "HAVE_CORETEXT" ]
     }
-    if (is_linux && !is_chromeos && !is_official_build && current_cpu != "arm") {
-      configs -= [ "//build/config/gcc:symbol_visibility_hidden" ]
-    }
   }
 }
diff --git a/third_party/harfbuzz-ng/harfbuzz.gyp b/third_party/harfbuzz-ng/harfbuzz.gyp
index c277e8e..f381e74 100644
--- a/third_party/harfbuzz-ng/harfbuzz.gyp
+++ b/third_party/harfbuzz-ng/harfbuzz.gyp
@@ -8,11 +8,12 @@
   ],
   'variables': {
     'conditions': [
-      ['OS=="linux" and chromeos==1', {
+      ['OS=="linux" and (buildtype!="Official" or chromeos==1) and embedded==0', {
         # Since version 1.31.0, pangoft2 which we depend on pulls in harfbuzz
         # anyways. However, we want to have control of the version of harfbuzz
-        # we use, so don't use system harfbuzz unless we are building for
-        # chrome os, where we have the system harfbuzz under control.
+        # we use, so don't use system harfbuzz for official builds, unless we
+        # are building for chrome os, where we have the system harfbuzz under
+        # control as well.
         'use_system_harfbuzz%': '<!(python ../../build/check_return_value.py <(pkg-config) --atleast-version=1.31.0 pangoft2)',
       }, {
         'use_system_harfbuzz': 0,
@@ -151,9 +152,6 @@
                 'src/hb-coretext.h',
               ],
             }],
-            ['OS=="linux" and chromeos==0 and buildtype!="Official" and target_arch!="arm"', {
-              'cflags!': ['-fvisibility=hidden'],
-            }],
           ],
         },
       ],