Created min size for print preview dialog and modified to allow the

chrome/browser/printing/print_preview_dialog_controller.cc

Index: chrome/browser/printing/print_preview_dialog_controller.cc
diff --git a/chrome/browser/printing/print_preview_dialog_controller.cc b/chrome/browser/printing/print_preview_dialog_controller.cc
index 6211d5c39e99661f74f0542e1160510e06b3018d..7872ebb7072a15b077cda076d9f877f2a7906c20 100644
--- a/chrome/browser/printing/print_preview_dialog_controller.cc
+++ b/chrome/browser/printing/print_preview_dialog_controller.cc
@@ -78,9 +78,7 @@ PrintPreviewDialogDelegate::~PrintPreviewDialogDelegate() {
 }
 
 ui::ModalType PrintPreviewDialogDelegate::GetDialogModalType() const {
-  // Not used, returning dummy value.
-  NOTREACHED();
-  return ui::MODAL_TYPE_WINDOW;
+  return ui::MODAL_TYPE_CHILD;
 }
 
 base::string16 PrintPreviewDialogDelegate::GetDialogTitle() const {
@@ -99,7 +97,8 @@ void PrintPreviewDialogDelegate::GetWebUIMessageHandlers(
 
 void PrintPreviewDialogDelegate::GetDialogSize(gfx::Size* size) const {
   DCHECK(size);
-  const gfx::Size kMinDialogSize(800, 480);
+  // Modified to match requested size. Previously: 800, 480.

[Mike Wittman, 2016/08/12 22:51:34]

This comment likely can be removed. Anyone who wants to understand the history
of kMinDialogSize can look at the file's git history.

[rbpotter, 2016/08/15 18:03:45]

Done.

+  const gfx::Size kMinDialogSize(438, 455);

[Mike Wittman, 2016/08/12 22:51:34]

Should this be sized differently depending on whether
wm::kHasIndependentBoundsKey is supported?

[rbpotter, 2016/08/15 18:03:45]

If there's not independent bounds key, the dialog ignores its minimum size
entirely and just sizes down to the size of the parent window. The only platform
that could be affected is Mac, so I am adding a couple Mac only lines to ensure
that on Mac the behavior is the same as before.

   const int kBorder = 25;
   *size = kMinDialogSize;
 
@@ -114,7 +113,11 @@ void PrintPreviewDialogDelegate::GetDialogSize(gfx::Size* size) const {
     size->SetToMax(host->GetMaximumDialogSize());
   else
     size->SetToMax(outermost_web_contents->GetContainerBounds().size());
-  size->Enlarge(-2 * kBorder, -kBorder);
+  int expand_width =

[Mike Wittman, 2016/08/12 22:51:34]

Same question here.

[rbpotter, 2016/08/15 18:03:45]

See above

+      std::min(size->width() - kMinDialogSize.width(), 2 * kBorder);
+  int expand_height =
+      std::min(size->height() - kMinDialogSize.height(), kBorder);
+  size->Enlarge(-expand_width, -expand_height);
 
 #if defined(OS_MACOSX)
   // Limit the maximum size on MacOS X.
@@ -364,10 +367,16 @@ WebContents* PrintPreviewDialogController::CreatePrintPreviewDialog(
   base::AutoReset<bool> auto_reset(&is_creating_print_preview_dialog_, true);
 
   // The dialog delegates are deleted when the dialog is closed.
+#if defined(OS_MACOSX)

[Mike Wittman, 2016/08/12 22:51:34]

Can we drop the #if part here and just have ShowTopLevelConstrainedWebDialog
invoke ShowConstrainedWebDialog within its implementation on Mac?

[rbpotter, 2016/08/15 18:03:45]

Done.

+  ConstrainedWebDialogDelegate* web_dialog_delegate = ShowConstrainedWebDialog(
+      initiator->GetBrowserContext(), new PrintPreviewDialogDelegate(initiator),
+      initiator);
+#else
   ConstrainedWebDialogDelegate* web_dialog_delegate =
-      ShowConstrainedWebDialog(initiator->GetBrowserContext(),
-                               new PrintPreviewDialogDelegate(initiator),
-                               initiator);
+      ShowTopLevelConstrainedWebDialog(
+          initiator->GetBrowserContext(),
+          new PrintPreviewDialogDelegate(initiator), initiator);
+#endif
 
   WebContents* preview_dialog = web_dialog_delegate->GetWebContents();