Devirtualize CHROMEGTK_CALLBACK.

Devirtualize CHROMEGTK_CALLBACK.

BUG=103827
TEST=compiles on clang without warnings and passes tests

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

[dcheng, 2011-11-11 23:04:26]

Same thing, but for CHROMEGTK_CALLBACK.

One thing I noticed is that most of NativeWidgetGtk doesn't need to use virtual.
I left all the signal callbacks as virtual for the time being though, for
consistency.

[dcheng, 2011-11-11 23:04:27]

Same thing, but for CHROMEGTK_CALLBACK.

One thing I noticed is that most of NativeWidgetGtk doesn't need to use virtual.
I left all the signal callbacks as virtual for the time being though, for
consistency.

[Elliot Glaysher, 2011-11-11 23:26:44]

On 2011/11/11 23:04:27, dcheng wrote:
> Same thing, but for CHROMEGTK_CALLBACK.
> 
> One thing I noticed is that most of NativeWidgetGtk doesn't need to use
virtual.
> I left all the signal callbacks as virtual for the time being though, for
> consistency.

Quick look over LGTM.

[Evan Stade, 2011-11-12 00:07:31]

lgtm

[dcheng, 2011-11-12 00:11:37]

Adding sky for OWNERS review for content/browser/*, ui/base/*, and views/*.

I'm not going to submit until I get a green trybot run--the last run failed
because LKGR hadn't caught up to my previous patch.

[sky, 2011-11-12 00:46:12]

LGTM