Promotes Add/RemoveObserver to MessageLoop for linux. Because...

Promotes Add/RemoveObserver to MessageLoop for linux. Because
message_loop.h now includes message_pump_glib.h I needed to update the
dependencies of a couple of projects.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=16317

[Evan Martin, 2009-05-18 17:18:11]

http://codereview.chromium.org/112032/diff/1/4
File base/message_loop.h (right):

http://codereview.chromium.org/112032/diff/1/4#newcode26
Line 26: #include "base/message_pump_glib.h"
I guess you probably only want this on Linux, not Mac.  (libevent is shared by
linux/mac)

http://codereview.chromium.org/112032/diff/1/2
File media/media.gyp (right):

http://codereview.chromium.org/112032/diff/1/2#newcode111
Line 111: '../build/linux/system.gyp:gtk',
Isn't this only glib?  I think maybe better is to add a glib section to
system.gyp

http://codereview.chromium.org/112032/diff/1/3
File net/net.gyp (right):

http://codereview.chromium.org/112032/diff/1/3#newcode326
Line 326: '../build/linux/system.gyp:gtk',
same glib comment here

[sky, 2009-05-18 17:53:54]

New snapshot uploaded.

http://codereview.chromium.org/112032/diff/1/4
File base/message_loop.h (right):

http://codereview.chromium.org/112032/diff/1/4#newcode26
Line 26: #include "base/message_pump_glib.h"
On 2009/05/18 17:18:11, Evan Martin wrote:
> I guess you probably only want this on Linux, not Mac.  (libevent is shared by
> linux/mac)

Done.

http://codereview.chromium.org/112032/diff/1/2
File media/media.gyp (right):

http://codereview.chromium.org/112032/diff/1/2#newcode111
Line 111: '../build/linux/system.gyp:gtk',
On 2009/05/18 17:18:11, Evan Martin wrote:
> Isn't this only glib?  I think maybe better is to add a glib section to
> system.gyp

As discussed over IM we're sticking with gtk for this. I was able to figure out
how to make it so that I don't have to add this dependency to each target
though.

[Evan Martin, 2009-05-18 18:19:45]

lgtm