Add import progress indicator dialog.

Make the import feature a little more complete:
- Now we show import progress of individual items
- In case Firefox is running we display a warning and allow user to close it before proceeding
- Delete the Firefox lock when import finishes

BUG=11191

[Evan Martin, 2009-05-22 22:37:35]

http://codereview.chromium.org/114047/diff/1/2
File chrome/browser/gtk/import_dialog_gtk.cc (right):

http://codereview.chromium.org/114047/diff/1/2#newcode35
Line 35: GTK_RESPONSE_ACCEPT,
This should stay in its original position, on the right (it's a GTK thing).

http://codereview.chromium.org/114047/diff/1/6
File chrome/browser/gtk/import_progress_dialog_gtk.cc (right):

http://codereview.chromium.org/114047/diff/1/6#newcode181
Line 181: gtk_widget_hide_all(dialog_);
Do we ever destroy the dialog and "delete this" in this case?

http://codereview.chromium.org/114047/diff/1/7
File chrome/browser/gtk/import_progress_dialog_gtk.h (right):

http://codereview.chromium.org/114047/diff/1/7#newcode17
Line 17: static void StartImport(GtkWindow* parent, int16 items,
We don't use int16 elsewhere, should this be int?

http://codereview.chromium.org/114047/diff/1/8
File chrome/browser/importer/firefox_importer_utils.cc (right):

http://codereview.chromium.org/114047/diff/1/8#newcode104
Line 104: // We do not read registry on other platforms
Would it make more sense to just not define or call these functions on other
platforms?

http://codereview.chromium.org/114047/diff/1/12
File chrome/browser/importer/importer.h (right):

http://codereview.chromium.org/114047/diff/1/12#newcode434
Line 434: #elif defined(OS_LINUX)
Rather than this, you can use gfx::NativeWindow and share the prototype.

[kuchhal, 2009-05-26 19:06:14]

Two comments below. Everything else fixed. Please take another look.

http://codereview.chromium.org/114047/diff/1/6
File chrome/browser/gtk/import_progress_dialog_gtk.cc (right):

http://codereview.chromium.org/114047/diff/1/6#newcode181
Line 181: gtk_widget_hide_all(dialog_);
The ImporterHost is supposed to call ImportEnded() when done, where we destroy
the dialog.

On 2009/05/22 22:37:36, Evan Martin wrote:
> Do we ever destroy the dialog and "delete this" in this case?

http://codereview.chromium.org/114047/diff/1/7
File chrome/browser/gtk/import_progress_dialog_gtk.h (right):

http://codereview.chromium.org/114047/diff/1/7#newcode17
Line 17: static void StartImport(GtkWindow* parent, int16 items,
I just carried over the same type that was defined on Windows in
StartImportingWithUI(). I can change it to int from here on but don't see any
advantage if we already have it at one place.

On 2009/05/22 22:37:36, Evan Martin wrote:
> We don't use int16 elsewhere, should this be int?

[Evan Martin, 2009-05-26 22:32:12]

OK, LG