MessageBubble which does not require separate window and does not grab input on mouse press event.

MessageBubble which does not require separate window and does not grab input on mouse press event.
 * Added new constructor to InfoBubble for chromeos which takes WidgetGtk::Type parameter
 * Made DoGrab/HandleGrabBroke so that subclass can customize its behavior.
 * Added MessageBubble::ShowNoGrab that shows a child MessageBubble which does not grab input on mouse press event.
 * Explicitly remove accelerator when closed. This is necessary for child, but doing for all case as it's harmless to do so.
 * Moved HandleGrabBroke to protected and made virtual so that sub class
   can trap the event. ScreenLocker will use this event to detect this.

This will be used for screen locker.

BUG=none
TEST=none

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

[sky, 2010-06-07 14:41:31]

http://codereview.chromium.org/2682004/diff/11001/12005
File views/widget/widget_gtk.h (right):

http://codereview.chromium.org/2682004/diff/11001/12005#newcode280
views/widget/widget_gtk.h:280: virtual void HandleGrabBroke();
Move the implementation to match the new position in the header. Did you even
need to make this virtual though? I don't see you overriding it in this cl.

[oshima, 2010-06-07 16:05:54]

http://codereview.chromium.org/2682004/diff/11001/12005
File views/widget/widget_gtk.h (right):

http://codereview.chromium.org/2682004/diff/11001/12005#newcode280
views/widget/widget_gtk.h:280: virtual void HandleGrabBroke();
On 2010/06/07 14:41:31, sky wrote:
> Move the implementation to match the new position in the header. Did you even
> need to make this virtual though? I don't see you overriding it in this cl.

Done. I need this to be virtual so that screen locker implementation can detect
when grab is broken, which is in next CL. Sorry i forgot to mention in the
description.

[sky, 2010-06-07 16:11:43]

LGTM