Simplify the macro for this deprecation. g_type_init was deprecated in glib version 2.36. The prev…

net/tools/net_watcher/net_watcher.cc

Index: net/tools/net_watcher/net_watcher.cc
diff --git a/net/tools/net_watcher/net_watcher.cc b/net/tools/net_watcher/net_watcher.cc
index 0a8b4dfbb7d8530bd0df5b6017c465d1d05f0cc1..b1004211eaf637734259efca9e4d7e127bfed30e 100644
--- a/net/tools/net_watcher/net_watcher.cc
+++ b/net/tools/net_watcher/net_watcher.cc
@@ -136,20 +136,9 @@ int main(int argc, char* argv[]) {
   // Normally handled by BrowserMainLoop::InitializeToolkit().
   // From glib version 2.36 onwards, g_type_init is implicitly called and it is
   // deprecated.
-  // TODO(yael) Simplify this once Ubuntu 10.04 is no longer supported.
-#if defined(G_GNUC_BEGIN_IGNORE_DEPRECATIONS) && \
-    defined(G_GNUC_END_IGNORE_DEPRECATIONS)
-#define USE_GLIB_DEPRECATIONS_MACROS
-#endif
-
-#if defined(USE_GLIB_DEPRECATIONS_MACROS)
-G_GNUC_BEGIN_IGNORE_DEPRECATIONS
-#endif
+#if (GLIB_MAJOR_VERSION <= 2 && GLIB_MINOR_VERSION < 36)

[akalin, 2013/10/04 17:24:49]

Shouldn't this be something like:

(GLIB_MAJOR_VERSION < 2) ||
(GLIB_MAJOR_VERSION == 2 && GLIB_MINOR_VERSION < 36)

?

However, it seems better to me to mirror the code in browser_main_loop.cc, which
looks like:

#if defined(OS_POSIX) && !defined(OS_MACOSX) && !defined(OS_ANDROID)
  // g_type_init will be deprecated in 2.36. 2.35 is the development
  // version for 2.36, hence do not call g_type_init starting 2.35.
  //
http://developer.gnome.org/gobject/unstable/gobject-Type-Information.html#g-t...
#if !GLIB_CHECK_VERSION(2, 35, 0)
  // Glib type system initialization. Needed at least for gconf,
  // used in net/proxy/proxy_config_service_linux.cc. Most likely
  // this is superfluous as gtk_init() ought to do this. It's
  // definitely harmless, so retained as a reminder of this
  // requirement for gconf.
  g_type_init();
#endif

   g_type_init();
-#if defined(USE_GLIB_DEPRECATIONS_MACROS)
-G_GNUC_END_IGNORE_DEPRECATIONS
 #endif
-#undef USE_GLIB_DEPRECATIONS_MACROS
 #endif  // (defined(OS_LINUX) || defined(OS_OPENBSD)) && !defined(OS_CHROMEOS)
   base::AtExitManager exit_manager;
   CommandLine::Init(argc, argv);