Add alert box suppression check box to linux js dialogs.

Add alert box suppression check box to linux js dialogs.

BUG=12174

[Evan Martin, 2009-05-28 23:13:36]

http://codereview.chromium.org/115893/diff/1/2
File chrome/browser/app_modal_dialog_gtk.cc (right):

http://codereview.chromium.org/115893/diff/1/2#newcode43
Line 43: results = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(item->data));
Can't you stash the toggle button pointer in the AppModalDialog class?

[tony, 2009-05-28 23:36:06]

On 2009/05/28 23:13:36, Evan Martin wrote:
> http://codereview.chromium.org/115893/diff/1/2
> File chrome/browser/app_modal_dialog_gtk.cc (right):
> 
> http://codereview.chromium.org/115893/diff/1/2#newcode43
> Line 43: results =
gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(item->data));
> Can't you stash the toggle button pointer in the AppModalDialog class?

Ok, I've put the objects on the dialog via g_object_[gs]et_data.

[Evan Martin, 2009-05-28 23:47:31]

LGTM

http://codereview.chromium.org/115893/diff/1003/4
File chrome/browser/app_modal_dialog_gtk.cc (right):

http://codereview.chromium.org/115893/diff/1003/4#newcode21
Line 21: const char kPromptTextId[] = "prompt_text";
maybe prefix with chrome_ just in case

http://codereview.chromium.org/115893/diff/1003/4#newcode31
Line 31: result = UTF8ToWide(gtk_entry_get_text(GTK_ENTRY(widget)));
if (widget)
  return UTF8ToWide(...);
return EmptyString() ?

http://codereview.chromium.org/115893/diff/1003/4#newcode42
Line 42: result = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget));
same as above?