Initial gtk3 support

build/linux/system.gyp

 # Copyright (c) 2012 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.
 
 {
   'variables': {
     # If any of the linux_link_FOO below are set to 1, then the corresponding
     # target will be linked against the FOO library (either dynamically or
     # statically, depending on the pkg-config files), as opposed to loading the
     # FOO library dynamically with dlopen.
@@ skipping 95 matching lines @@
             }],
           ],
         },
         {
           'target_name': 'gdk',
           'type': 'none',
           'conditions': [
             ['_toolset=="target"', {
               'direct_dependent_settings': {
                 'cflags': [
-                  '<!@(<(pkg-config) --cflags gdk-2.0)',

[Elliot Glaysher, 2015/07/20 19:31:40]

Please don't do this. We need to support both GTK 2 and 3 in the same build.
Instead make a 'gdk2' and 'gdk3' targets, have libgtk2ui depend on gdk2 (&etc)
and have libgtk3ui depend on gdk3 (&etc).

[knthzh, 2015/07/21 15:48:27]

I can't. I'm too terrified of triggering another eight-hour rebuild to touch the
build system.

[Elliot Glaysher, 2015/07/28 17:54:36]

(Just to be clear, I'm fine with you having this hack in while you're developing
the rest of the theme code, but this can not land as is; we need to be able to
continue to support gtk2 for the foreseeable future.)

+                  '<!@(<(pkg-config) --cflags gdk-3.0)',
                 ],
               },
               'link_settings': {
                 'ldflags': [
-                  '<!@(<(pkg-config) --libs-only-L --libs-only-other gdk-2.0)',
+                  '<!@(<(pkg-config) --libs-only-L --libs-only-other gdk-3.0)',
                 ],
                 'libraries': [
-                  '<!@(<(pkg-config) --libs-only-l gdk-2.0)',
+                  '<!@(<(pkg-config) --libs-only-l gdk-3.0)',
                 ],
               },
             }],
           ],
         },
         {
           'target_name': 'gtk',
           'type': 'none',
           'toolsets': ['host', 'target'],
           'variables': {
             # gtk requires gmodule, but it does not list it as a dependency
             # in some misconfigured systems.
-            'gtk_packages': 'gmodule-2.0 gtk+-2.0 gthread-2.0',
+            'gtk_packages': 'gmodule-2.0 gtk+-3.0 gthread-2.0',
           },
           'conditions': [
             ['_toolset=="target"', {
               'all_dependent_settings': {
                 'cflags': [
                   '<!@(<(pkg-config) --cflags <(gtk_packages))',
                 ],
               },
               'link_settings': {
                 'ldflags': [
@@ skipping 20 matching lines @@
             }],
           ],
         },
         {
           'target_name': 'gtkprint',
           'type': 'none',
           'conditions': [
             ['_toolset=="target"', {
               'direct_dependent_settings': {
                 'cflags': [
-                  '<!@(<(pkg-config) --cflags gtk+-unix-print-2.0)',
+                  '<!@(<(pkg-config) --cflags gtk+-unix-print-3.0)',
                 ],
               },
               'link_settings': {
                 'ldflags': [
-                  '<!@(<(pkg-config) --libs-only-L --libs-only-other gtk+-unix-print-2.0)',
+                  '<!@(<(pkg-config) --libs-only-L --libs-only-other gtk+-unix-print-3.0)',
                 ],
                 'libraries': [
-                  '<!@(<(pkg-config) --libs-only-l gtk+-unix-print-2.0)',
+                  '<!@(<(pkg-config) --libs-only-l gtk+-unix-print-3.0)',
                 ],
               },
             }],
           ],
         },
       ],  # targets
     }],
     [ 'use_x11==1 or ozone_platform_ozonex==1', {
       # Hide X11 and related dependencies when use_x11=0
       'targets': [
@@ skipping 1016 matching lines @@
                   },
                 }],
               ],
             }],
           ]
         }],
       ],
     },
   ],
 }