CEF1: Linux compilation error, GTK deprecated function (with correction)

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Setup Chromium source directory (rev 131752).
2. Setup Cef source directory in gyp (rev 600).
3. Build cef using "make -j4 cefclient" command.

What is the expected output? What do you see instead?
Compilation Fails with the following message:
  CXX(target) out/Debug/obj.target/libcef_static/cef/libcef/web_drag_source_gtk.o
cef/libcef/web_drag_source_gtk.cc: In member function ‘void 
WebDragSource::StartDragging(const WebDropData&, WebKit::WebDragOperationsMask, 
const WebKit::WebImage&, const WebKit::WebPoint&)’:
cef/libcef/web_drag_source_gtk.cc:101:5: erreur: ‘void 
gdk_pixbuf_unref(GdkPixbuf*)’ is deprecated (declared at 
/usr/include/gdk-pixbuf-2.0/gdk-pixbuf/gdk-pixbuf-core.h:243): Use 
'g_object_unref' instead [-Werror=deprecated-declarations]
cef/libcef/web_drag_source_gtk.cc:101:28: erreur: ‘void 
gdk_pixbuf_unref(GdkPixbuf*)’ is deprecated (declared at 
/usr/include/gdk-pixbuf-2.0/gdk-pixbuf/gdk-pixbuf-core.h:243): Use 
'g_object_unref' instead [-Werror=deprecated-declarations]
cc1plus: all warnings being treated as errors
make: *** [out/Debug/obj.target/libcef_static/cef/libcef/web_drag_source_gtk.o] 
Erreur 1

What version of the product are you using? On what operating system?
Lubuntu 12.04, 32 bits.

Please provide any additional information below.
As the error message says, changing "gdk_pixbuf_unref" with "g_object_unref" 
resolves the compilation issue.

Original issue reported on code.google.com by jfver...@gmail.com on 8 May 2012 at 10:37

[GoogleCodeExporter]

It is a simple change. I made it and it worked without any other change.

Original comment by Fireboy.mrtz on 17 May 2012 at 10:05

[GoogleCodeExporter]

Thanks, fixed in revision 634.

Original comment by magreenb...@gmail.com on 18 May 2012 at 9:41

• Changed state: Fixed

[GoogleCodeExporter]

Is it possible to get this fix merged back into the 1084 branch?

Original comment by zombie.k...@gmail.com on 10 Jun 2012 at 12:52

[GoogleCodeExporter]

Merged to 1025 and 1084 branches in revision 693.

Original comment by magreenb...@gmail.com on 11 Jun 2012 at 8:47