Do not build FreeType with HarfBuzz support on Android

third_party/freetype/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.
 
 import("//build/config/chromecast_build.gni")
 import("//build/config/freetype/freetype.gni")
 import("//third_party/harfbuzz-ng/harfbuzz.gni")
 
 config("freetype_config") {
   include_dirs = [
@@ skipping 102 matching lines @@
 
   if (!is_android) {
     sources += [
       "src/src/cid/type1cid.c",
       "src/src/psaux/psaux.c",
       "src/src/type1/type1.c",
     ]
 
     # Selects those three modules in freetype-custom-config/ftmodule.h.
     defines += [ "PDFIUM_REQUIRED_MODULES" ]
+  } else {
+    # Using HarfBuzz assisted autohinting has a performance impact that we do not

[Dirk Pranke, 2017/05/23 19:26:38]

Nit: fix the line length / wrapping.

+    # want to take on Android, see crbug.com/724095.
+    defines += [ "WITHOUT_HARFBUZZ" ]
   }
 
   if (is_linux || is_chromecast) {
     # Needed for content_shell on Linux and Chromecast, since fontconfig
     # requires FT_Get_BDF_Property.
     sources += [ "src/src/base/ftbdf.c" ]
   }
 
   if (is_linux || is_chromecast) {
     # Needed on Fedora whose libfreetype builds ftsynth.c containing
@@ skipping 34 matching lines @@
     "//third_party/zlib",
   ]
 
   if (!use_system_freetype) {
     deps += [
     ":bootstrap_freetype_for_harfbuzz"
     ]
   }
 
   if (!use_system_harfbuzz) {
-    deps += [
-      "//third_party/harfbuzz-ng:harfbuzz-ng-ft",
-      "//third_party/harfbuzz-ng:harfbuzz-ng-without-freetype",
-    ]
+    if (!is_android) {
+      # See "WITHOUT_HARFBUZZ" above, harfbuzz-ng-ft is only needed if we build
+      # FreeType with HarfBuzz support.
+      deps += [ "//third_party/harfbuzz-ng:harfbuzz-ng-ft" ]
+    }
+    deps += [ "//third_party/harfbuzz-ng:harfbuzz-ng-without-freetype" ]
   } else {
     deps += [ "//third_party/harfbuzz-ng:harfbuzz-ng" ]
   }
 }