summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/gtk/constrained_window_gtk.h
diff options
context:
space:
mode:
authorjschuh@chromium.org <jschuh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-02-16 15:36:10 +0000
committerjschuh@chromium.org <jschuh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-02-16 15:36:10 +0000
commitdb0cc04178dd44b9dcd03a89db854d45a36fc37b (patch)
treebfc1c6bd893fe1304dad97dad130ded31a016ea1 /chrome/browser/ui/gtk/constrained_window_gtk.h
parent76cfe6f5090be766ae44e84281356c1649c2ae32 (diff)
downloadchromium_src-db0cc04178dd44b9dcd03a89db854d45a36fc37b.zip
chromium_src-db0cc04178dd44b9dcd03a89db854d45a36fc37b.tar.gz
chromium_src-db0cc04178dd44b9dcd03a89db854d45a36fc37b.tar.bz2
Revert 182969
Speculation was wrong. Reverting the revert. > Revert 182878 > > Speculative revert because this appears to be causing > ExtensionApiTest.UpdateWindowShowState to fail on Win7. > I'll reland if this is not the culprit. > > > Use a uniform opaque type to represent web contents modal dialogs > > > > Since the WebContentsModalDialog interface is going away, the > > platform-independent code in WebContentsModalDialogManager requires an > > opaque type with which to identify dialogs. > > > > In a first attempt at this under https://codereview.chromium.org/12224020, > > comments indicated that the Mac implementation of the web contents > > modal dialog does not fit any of the abstracted native types provided > > by ui/gfx/native_widget_types.h. So, define a special typedef > > NativeWebContentsModalDialog that uses the existing gfx::NativeView > > abstraction on platforms where supported, mainly to document the expected > > type. On Mac, we use a void* which is the preferred type for this, per > > Sailesh in comments in this review. > > > > BUG=157161 > > > > > > Review URL: https://chromiumcodereview.appspot.com/12221055 > > TBR=wittman@chromium.org > Review URL: https://codereview.chromium.org/12293002 TBR=jschuh@chromium.org Review URL: https://codereview.chromium.org/12288022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@182974 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/gtk/constrained_window_gtk.h')
-rw-r--r--chrome/browser/ui/gtk/constrained_window_gtk.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/ui/gtk/constrained_window_gtk.h b/chrome/browser/ui/gtk/constrained_window_gtk.h
index a9807bd..9646665 100644
--- a/chrome/browser/ui/gtk/constrained_window_gtk.h
+++ b/chrome/browser/ui/gtk/constrained_window_gtk.h
@@ -59,7 +59,7 @@ class ConstrainedWindowGtk : public WebContentsModalDialog {
virtual void CloseWebContentsModalDialog() OVERRIDE;
virtual void FocusWebContentsModalDialog() OVERRIDE;
virtual void PulseWebContentsModalDialog() OVERRIDE;
- virtual gfx::NativeWindow GetNativeWindow() OVERRIDE;
+ virtual NativeWebContentsModalDialog GetNativeDialog() OVERRIDE;
// Called when the result of GetBackgroundColor may have changed.
void BackgroundColorChanged();