* enable message box...

* enable MessageBoxView
  convert wstring to string16 when necessary using WideToUTF16Hack, which is no-op on Windows.
* updated checkbox to get state update working
  state handing is a bit hacky, esp. b/c gtk sends signal even when the state is changed via API.
  If there is better way, please let me know.

[sky, 2009-09-03 23:31:54]

http://codereview.chromium.org/194014/diff/5002/5005
File views/controls/button/native_button_gtk.cc (right):

http://codereview.chromium.org/194014/diff/5002/5005#newcode154
Line 154: }
Wouldn't it be better to set deliver_click_event_ back to true here rather than
relying on OnClicked to set it?

http://codereview.chromium.org/194014/diff/5002/5006
File views/controls/button/native_button_gtk.h (right):

http://codereview.chromium.org/194014/diff/5002/5006#newcode24
Line 24: 
nit: As GetView is part of NativeButtonWrapper, don't add a newline here.

http://codereview.chromium.org/194014/diff/5002/5006#newcode78
Line 78: bool deliver_click_event_;
description?

[oshima, 2009-09-09 02:36:02]

http://codereview.chromium.org/194014/diff/5002/5005
File views/controls/button/native_button_gtk.cc (right):

http://codereview.chromium.org/194014/diff/5002/5005#newcode154
Line 154: }
On 2009/09/03 23:31:54, sky wrote:
> Wouldn't it be better to set deliver_click_event_ back to true here rather
than
> relying on OnClicked to set it?

The problem is that set_active emit the clicked signal. I may be wrong, but
there seems to be no way to tell how the singal is generated (by a user or via
API),
so I had to set the flag here. Yes, this is a bit confusing & messy. I updated
the comment, but please let me know if there is better solution.

http://codereview.chromium.org/194014/diff/5002/5006
File views/controls/button/native_button_gtk.h (right):

http://codereview.chromium.org/194014/diff/5002/5006#newcode24
Line 24: 
On 2009/09/03 23:31:54, sky wrote:
> nit: As GetView is part of NativeButtonWrapper, don't add a newline here.

Done.

http://codereview.chromium.org/194014/diff/5002/5006#newcode78
Line 78: bool deliver_click_event_;
On 2009/09/03 23:31:54, sky wrote:
> description?

Done.

[oshima, 2009-09-10 00:47:59]

http://codereview.chromium.org/194014/diff/5002/5005
File views/controls/button/native_button_gtk.cc (right):

http://codereview.chromium.org/194014/diff/5002/5005#newcode154
Line 154: }
On 2009/09/09 02:36:02, oshima wrote:
> On 2009/09/03 23:31:54, sky wrote:
> > Wouldn't it be better to set deliver_click_event_ back to true here rather
> than
> > relying on OnClicked to set it?
> 
> The problem is that set_active emit the clicked signal. I may be wrong, but
> there seems to be no way to tell how the singal is generated (by a user or via
> API),
> so I had to set the flag here. Yes, this is a bit confusing & messy. I updated
> the comment, but please let me know if there is better solution.
> 

Done.

[sky, 2009-09-10 15:49:13]

LGTM