Componentize app_modal_dialog

components/app_modal_dialogs/app_modal_dialog.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // 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>
 
 #include "base/basictypes.h"
 #include "base/strings/string16.h"
 #include "build/build_config.h"
 
 class NativeAppModalDialog;
 
 namespace content {
@@ skipping 65 matching lines @@
   bool valid_;
 
   // The toolkit-specific implementation of the app modal dialog box.
   NativeAppModalDialog* native_dialog_;
 
   content::WebContents* web_contents_;
 
   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_