Content shell: Javascript dialogs, first pass, just Mac for now.

content/shell/shell_js_dialog.h

+// Copyright (c) 2012 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 CONTENT_SHELL_SHELL_JS_DIALOG_H_

[jochen (gone - plz use gerrit), 2012/03/28 18:38:18]

I think I would prefer shell_javascript_dialog instead of "js"

[Avi (use Gerrit), 2012/03/28 20:43:43]

Done.

+#define CONTENT_SHELL_SHELL_JS_DIALOG_H_
+#pragma once
+
+#include "content/public/browser/javascript_dialogs.h"
+
+#if defined(OS_MACOSX)
+#if __OBJC__
+@class NSAlert;
+@class ShellJavaScriptDialogHelper;
+#else
+class NSAlert;
+class ShellJavaScriptDialogHelper;
+#endif  // __OBJC__
+#endif  // defined(OS_MACOSX)
+
+namespace content {
+
+class ShellJavaScriptDialogCreator;
+
+class ShellJavaScriptDialog {
+ public:
+  ShellJavaScriptDialog(
+      ShellJavaScriptDialogCreator* creator,
+      ui::JavascriptMessageType javascript_message_type,
+      const string16& message_text,
+      const string16& default_prompt_text,
+      const JavaScriptDialogCreator::DialogClosedCallback& callback);
+  ~ShellJavaScriptDialog();

[jochen (gone - plz use gerrit), 2012/03/28 18:38:18]

virtual

[Avi (use Gerrit), 2012/03/28 20:43:43]

Why? This isn't an interface, it extends nothing, and has no virtual methods.

+
+  // Called to cancel a dialog mid-flight.
+  void Cancel();
+
+ private:
+  ShellJavaScriptDialogCreator* creator_;
+  JavaScriptDialogCreator::DialogClosedCallback callback_;
+
+#if defined(OS_MACOSX)
+  ShellJavaScriptDialogHelper* helper_;  // owned
+  NSAlert* alert_; // weak, owned by |helper_|.
+#endif  // defined(OS_MACOSX)
+
+  DISALLOW_COPY_AND_ASSIGN(ShellJavaScriptDialog);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_SHELL_SHELL_JS_DIALOG_H_