Componentize app_modal_dialog

components/app_modal_dialogs/app_modal_dialog.h

Index: components/app_modal_dialogs/app_modal_dialog.h
diff --git a/chrome/browser/ui/app_modal_dialogs/app_modal_dialog.h b/components/app_modal_dialogs/app_modal_dialog.h
similarity index 87%
rename from chrome/browser/ui/app_modal_dialogs/app_modal_dialog.h
rename to components/app_modal_dialogs/app_modal_dialog.h
index 28cf5841f3c05e87d40a647b5ab15ba02bcea3c0..34580b37e877825a34e9007d5a40d40c8a2fb1af 100644
--- a/chrome/browser/ui/app_modal_dialogs/app_modal_dialog.h
+++ b/components/app_modal_dialogs/app_modal_dialog.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_UI_APP_MODAL_DIALOGS_APP_MODAL_DIALOG_H_
-#define CHROME_BROWSER_UI_APP_MODAL_DIALOGS_APP_MODAL_DIALOG_H_
+#ifndef COMPONENTS_APP_MODAL_DIALOGS_APP_MODAL_DIALOG_H_
+#define COMPONENTS_APP_MODAL_DIALOGS_APP_MODAL_DIALOG_H_
 
 #include <string>
 
@@ -89,4 +89,14 @@ class AppModalDialog {
   DISALLOW_COPY_AND_ASSIGN(AppModalDialog);
 };
 
-#endif  // CHROME_BROWSER_UI_APP_MODAL_DIALOGS_APP_MODAL_DIALOG_H_
+// An interface to notify that a modal dialog is shown.
+class AppModalDialogNotifier {

[msw, 2014/10/23 18:41:22]

nit: technically this is a notify-ee (one that is notified)... Consider
AppModalDialogObserver?

[oshima, 2014/10/23 21:44:18]

Done.

+ public:
+  AppModalDialogNotifier();
+  virtual ~AppModalDialogNotifier();
+
+  // Called when the modal dialog is shown.
+  virtual void Notify(AppModalDialog* dialog) = 0;
+};

[msw, 2014/10/23 18:41:22]

nit: private DISALLOW_COPY_AND_ASSIGN?

[oshima, 2014/10/23 21:44:18]

Done.

+
+#endif  // COMPONENTS_APP_MODAL_DIALOGS_APP_MODAL_DIALOG_H_