[components] Switch {RefCounted}ProfileKeyedService to use BrowserContext

build/linux/system.gyp

Index: build/linux/system.gyp
diff --git a/build/linux/system.gyp b/build/linux/system.gyp
index cbd2e59c4919d6753f0831496993faa6916dbbb6..b925792c33c2fc7e7c974a0babbddeb78625a417 100644
--- a/build/linux/system.gyp
+++ b/build/linux/system.gyp
@@ -43,7 +43,7 @@
           },
           'conditions': [
             ['_toolset=="target"', {
-              'direct_dependent_settings': {
+              'all_dependent_settings': {

[Jói, 2013/04/23 22:45:22]

Were the changes in this file meant to be part of this changelist?

[Paweł Hajdan Jr., 2013/04/23 22:58:11]

Unfortunately, yes. Otherwise it doesn't compile.

[Jói, 2013/04/24 09:09:10]

Hmm, do you know why?

[Paweł Hajdan Jr., 2013/04/25 17:51:43]

The error message is:

  CXX(target) out/Debug/obj.target/apps/apps/app_restore_service_factory.o
In file included from ./content/public/browser/browser_context.h:13:0,
                 from ./chrome/browser/profiles/profile.h:16,
                 from apps/app_restore_service_factory.cc:8:
./ui/base/clipboard/clipboard.h:23:21: fatal error: gdk/gdk.h: No such file or
directory

And it's caused by GTK+ include path not being propagated to the build target
that contains apps/app_restore_service_factory.cc . The problem is: why should
apps gyp target explicitly include a gtk+ dependency, while it exists there only
due to transitive header inclusion?

                 'cflags': [
                   '<!@(<(pkg-config) --cflags <(gtk_packages))',
                 ],
@@ -57,7 +57,7 @@
                 ],
               },
             }, {
-              'direct_dependent_settings': {
+              'all_dependent_settings': {
                 'cflags': [
                   '<!@(pkg-config --cflags <(gtk_packages))',
                 ],