GTK: Hardy build fix for r84860.

GTK: Hardy build fix for r84860.

(Aren't we supposed to be deprecating that?)

BUG=30213
TEST=copmiles

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

[Elliot Glaysher, 2011-05-10 22:02:39]

Need a quick stamp for a build fix for older Hardy.

[Evan Martin, 2011-05-10 22:06:01]

LGTM

[Evan Martin, 2011-05-10 22:06:27]

Wait a sec, is this a compile-time check?

[Elliot Glaysher, 2011-05-10 22:07:03]

Yes. It has to be because the API I'm calling isn't in 2.12.

On Tue, May 10, 2011 at 3:06 PM,  <evan@chromium.org> wrote:
> Wait a sec, is this a compile-time check?
>
> http://codereview.chromium.org/6982010/
>

[Evan Martin, 2011-05-10 22:10:44]

On 2011/05/10 22:07:03, Elliot Glaysher wrote:
> Yes. It has to be because the API I'm calling isn't in 2.12.

Won't that mean we always compile this out for all our binaries?

[Elliot Glaysher, 2011-05-10 22:14:00]

On 2011/05/10 22:10:44, Evan Martin wrote:
> On 2011/05/10 22:07:03, Elliot Glaysher wrote:
> > Yes. It has to be because the API I'm calling isn't in 2.12.
> 
> Won't that mean we always compile this out for all our binaries?

Why? We build on Lucid now, right? The only compile problem I'm having is some
memory bot.

[Evan Martin, 2011-05-10 22:25:49]

On 2011/05/10 22:14:00, Elliot Glaysher wrote:
> On 2011/05/10 22:10:44, Evan Martin wrote:
> > On 2011/05/10 22:07:03, Elliot Glaysher wrote:
> > > Yes. It has to be because the API I'm calling isn't in 2.12.
> > 
> > Won't that mean we always compile this out for all our binaries?
> 
> Why? We build on Lucid now, right? The only compile problem I'm having is some
> memory bot.

I don't know.  :(
Lei or Michael would know...

[Elliot Glaysher, 2011-05-10 22:26:42]

Talked to thestig. We don't build on lucid yet, but that's the plan by
M13. I'll just dlsym it. sigh.

On Tue, May 10, 2011 at 3:14 PM,  <erg@chromium.org> wrote:
> On 2011/05/10 22:10:44, Evan Martin wrote:
>>
>> On 2011/05/10 22:07:03, Elliot Glaysher wrote:
>> > Yes. It has to be because the API I'm calling isn't in 2.12.
>
>> Won't that mean we always compile this out for all our binaries?
>
> Why? We build on Lucid now, right? The only compile problem I'm having is
> some
> memory bot.
>
> http://codereview.chromium.org/6982010/
>