Implement a couple more methods for dialog boxes.

Implement a couple more methods for dialog boxes including the prompt box.

BUG=9623

[Evan Martin, 2009-04-28 21:56:59]

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

http://codereview.chromium.org/100124/diff/4/5#newcode20
Line 20: // return it.
This feels fragile.  Is there no good place to stick the GtkEntry*?  Like in the
AppModalDialog* or something.

http://codereview.chromium.org/100124/diff/4/5#newcode25
Line 25: for (guint i = 0; (item = g_list_nth(first_child, i)); ++i) {
C'mon, topcoder man!

for (GList* item = first_child; item; item = g_list_next(item))

http://codereview.chromium.org/100124/diff/4/5#newcode82
Line 82: // http://crbug.com/9623
If this is fixed, your should remove this comment.

[tony, 2009-04-28 22:05:46]

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

http://codereview.chromium.org/100124/diff/4/5#newcode20
Line 20: // return it.
On 2009/04/28 21:56:59, Evan Martin wrote:
> This feels fragile.  Is there no good place to stick the GtkEntry*?  Like in
the
> AppModalDialog* or something.

Here were the choices:
1) Add an if defined(OS_LINUX) in the shared app_modal_dialog.h for the
text_entry.
2) Change dialog_ to be a struct/class on linux with the GtkDialog* and
GtkEntry* pointers.
3) Stash the pointer on the GtkDialog by name.
4) Iterate over the vbox to find the text entry.

I chose to go with option 4 since it seemed the least nasty.

http://codereview.chromium.org/100124/diff/4/5#newcode25
Line 25: for (guint i = 0; (item = g_list_nth(first_child, i)); ++i) {
On 2009/04/28 21:56:59, Evan Martin wrote:
> C'mon, topcoder man!
> 
> for (GList* item = first_child; item; item = g_list_next(item))

Done.

http://codereview.chromium.org/100124/diff/4/5#newcode82
Line 82: // http://crbug.com/9623
On 2009/04/28 21:56:59, Evan Martin wrote:
> If this is fixed, your should remove this comment.

Done.

[Evan Martin, 2009-04-28 22:10:58]

ship it