Set rpath to let content_shell find our own libfreetype

content/shell/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/features.gni")
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/config/ui.gni")
 import("//build/config/win/console_app.gni")
 import("//build/config/win/manifest.gni")
 import("//media/media_options.gni")
@@ skipping 444 matching lines @@
       ]
     }
 
     if (is_android) {
       # Some tests rely on this tool. It might be nicer if these tests relied on
       # image diff rather than having content shell depend on it.
       data_deps = [
         "//tools/imagediff($host_toolchain)",
       ]
     }
+
+    if (is_linux && !is_component_build) {
+      # Set rpath to find our own libfreetype even in a non-component build.
+      configs += [ "//build/config/gcc:rpath_for_built_shared_libraries" ]
+    }

[Dirk Pranke, 2016/03/10 01:37:12]

This is basically the right fix, but I think it might be better to make this
config be a part of the public_configs for freetype2 (i.e., move this line to
//third_party/freetype2/BUILD.gn:12).

That way things can't get out of sync.

Does that work?

[Mike West, 2016/03/10 08:46:46]

I'd be fine with that.

[Xianzhu, 2016/03/10 17:31:26]

I think the current method is better because:
- It's consistent with the method used for other binaries in the similar
situation (https://codereview.chromium.org/1535863002);
- It can also handle other dynamic libraries (if any) linked to content_shell.

   }
 } else {
   group("content_shell") {
     testonly = true
     deps = [
       ":content_shell_lib",
     ]
   }
 }
 
@@ skipping 28 matching lines @@
     sources = [
       "app/shell_content_main.cc",
       "app/shell_content_main.h",
     ]
 
     deps = [
       ":content_shell_lib",
     ]
   }
 }