[WebModals] New API for browser-scoped popup management.

components/web_modal/native_web_contents_modal_dialog.h

 // Copyright (c) 2013 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 COMPONENTS_WEB_MODAL_NATIVE_WEB_CONTENTS_MODAL_DIALOG_H_
 #define COMPONENTS_WEB_MODAL_NATIVE_WEB_CONTENTS_MODAL_DIALOG_H_
 
 #include "ui/gfx/native_widget_types.h"
 
 namespace web_modal {
 
+// TODO(gbillock): rename this file
+
 #if defined(OS_MACOSX)
 // Use a void* since none of the gfx::Native* types are suitable for
 // representing the web contents modal dialog under Cocoa.
 typedef void* NativeWebContentsModalDialog;
 #else
 typedef gfx::NativeView NativeWebContentsModalDialog;
 #endif
 
+#if defined(OS_MACOSX)
+// Use a void* since none of the gfx::Native* types are suitable for

[msw, 2014/05/16 17:10:57]

Interesting, NSWindow (gfx::NativeWindow) and NSView (gfx::NativeView) aren't
applicable for neither bubbles nor dialogs?

[Mike Wittman, 2014/05/16 20:35:40]

See https://codereview.chromium.org/12224020 for the extended discussion on why
neither gfx::NativeWindow nor gfx::NativeView work for WCMDs. Similar logic may
or may not also apply to popups.

+// representing a popup window under Cocoa.
+typedef void* NativePopup;
+#else
+typedef gfx::NativeView NativePopup;
+#endif

[Finnur, 2014/05/16 12:38:14]

I probably would merge these two #if defs.

Out of curiosity... Anyone know the reason why we didn't append Ptr to these
names (as in NativeViewPtr and NativePopupPtr)? :)

[msw, 2014/05/16 17:10:57]

NativeWindow was at one point HWND, which is a handle, not a really a pointer.

+
 }  // namespace web_modal
 
 #endif  // COMPONENTS_WEB_MODAL_NATIVE_WEB_CONTENTS_MODAL_DIALOG_H_