Enable AutoResize for Constrained Web Dialogs

chrome/browser/ui/webui/constrained_web_dialog_ui.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 CHROME_BROWSER_UI_WEBUI_CONSTRAINED_WEB_DIALOG_UI_H_
 #define CHROME_BROWSER_UI_WEBUI_CONSTRAINED_WEB_DIALOG_UI_H_
 
 #include "base/compiler_specific.h"
 #include "components/web_modal/native_web_contents_modal_dialog.h"
 #include "content/public/browser/web_ui_controller.h"
 #include "ui/gfx/native_widget_types.h"
 
+namespace gfx {
+  class Size;

[Bernhard Bauer, 2014/11/27 12:41:34]

Don't indent inside of namespaces.

[apacible, 2014/12/01 17:35:57]

Done.

+}
+
 namespace content {
 class BrowserContext;
 class RenderViewHost;
 class WebContents;
 }
 
 namespace ui {
 class WebDialogDelegate;
 class WebDialogWebContentsDelegate;
 }
@@ skipping 45 matching lines @@
   // Returns NULL if no such delegate is set.
   ConstrainedWebDialogDelegate* GetConstrainedDelegate();
 
  private:
   // JS Message Handler
   void OnDialogCloseMessage(const base::ListValue* args);
 
   DISALLOW_COPY_AND_ASSIGN(ConstrainedWebDialogUI);
 };
 
-// Create a constrained HTML dialog. The actual object that gets created
-// is a ConstrainedWebDialogDelegate, which later triggers construction of a
-// ConstrainedWebDialogUI object.
+// Create and show a constrained HTML dialog. The actual object that gets
+// created is a ConstrainedWebDialogDelegate, which later triggers construction
+// of a ConstrainedWebDialogUI object.
 // |browser_context| is used to construct the constrained HTML dialog's
 //                   WebContents.
 // |delegate| controls the behavior of the dialog.
 // |overshadowed| is the tab being overshadowed by the dialog.
-ConstrainedWebDialogDelegate* CreateConstrainedWebDialog(
+ConstrainedWebDialogDelegate* ShowConstrainedWebDialog(
     content::BrowserContext* browser_context,
     ui::WebDialogDelegate* delegate,
     content::WebContents* overshadowed);
 
+// Creates a constrained HTML dialog with auto-resize enabled. The dialog

[Bernhard Bauer, 2014/11/27 12:41:34]

Nit: Either use imperative style ("create") like above, or change both to be
descriptive.

[apacible, 2014/12/01 17:35:57]

Done.

+// is shown by calling constrained_window::ShowModalDialogViews() after
+// document load has completed to avoid UI jankiness.
+// |browser_context| is used to construct the constrained HTML dialog's
+//                   WebContents.
+// |delegate| controls the behavior of the dialog.
+// |overshadowed| is the tab being overshadowed by the dialog.
+// |min_size| is the minimum size of the dialog.
+// |max_size| is the maximum size of the dialog.
+ConstrainedWebDialogDelegate* CreateConstrainedWebDialogWithAutoResize(
+    content::BrowserContext* browser_context,
+    ui::WebDialogDelegate* delegate,
+    content::WebContents* overshadowed,
+    gfx::Size min_size,
+    gfx::Size max_size);
+
 #endif  // CHROME_BROWSER_UI_WEBUI_CONSTRAINED_WEB_DIALOG_UI_H_