[Chromecast] Depend on Chromium's freetype-android

content/shell/BUILD.gn

Index: content/shell/BUILD.gn
diff --git a/content/shell/BUILD.gn b/content/shell/BUILD.gn
index c07b215604842919edfaacc5ecf355402ef30bbb..ffcd2491822d2a2a60a8cad2b84cfcd9cdb35628 100644
--- a/content/shell/BUILD.gn
+++ b/content/shell/BUILD.gn
@@ -313,7 +313,7 @@ static_library("content_shell_lib") {
     ]
   }
 
-  if (is_linux && !is_chromecast) {
+  if (is_linux) {
     deps += [ "//third_party/freetype2" ]

[Dirk Pranke, 2016/01/28 01:09:21]

is this right? you'll get //third_party/freetype2 in content_shell even for
chromecast, rather than //third_party/freetype-android ?

(just checking that that's the intent)

[bcf, 2016/01/28 21:37:28]

This logic is from this CL: https://codereview.chromium.org/1192933002

It seems like the issue from that CL no longer exists with statically linked
freetype

[Nico, 2016/01/28 21:55:47]

This looks wrong to me too -- you probably don't want to depend on different
freetypes? Don't you want build/linux/freetype here as dep (and everywhere else
too)?

[bcf, 2016/01/28 22:31:04]

We discussed in the predecessor of this CL that third_party/freetype2 is only
for testing. (See third_party/freetype2/README.chromium). content_shell_lib
should always use third_party/freetype2 to ensure fonts always render the same
way for tests. 

[Nico, 2016/01/28 22:33:01]

Does android use freetype2 for tests and freetype-android for production too?
Seems weird to test with one freetype and ship another, no?

   }