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

components/web_modal/web_contents_modal_dialog_manager.h

Index: components/web_modal/web_contents_modal_dialog_manager.h
diff --git a/components/web_modal/web_contents_modal_dialog_manager.h b/components/web_modal/web_contents_modal_dialog_manager.h
index 9d964d1c4edc46fdf58cac519245bf22759850fc..51558ce1072a3108f92ac438fdeba5bff42e2934 100644
--- a/components/web_modal/web_contents_modal_dialog_manager.h
+++ b/components/web_modal/web_contents_modal_dialog_manager.h
@@ -7,6 +7,7 @@
 
 #include <deque>
 #include <memory>
+#include <set>
 
 #include "base/macros.h"
 #include "build/build_config.h"
@@ -54,6 +55,8 @@ class WebContentsModalDialogManager
   // SingleWebContentsDialogManagerDelegate:
   content::WebContents* GetWebContents() const override;
   void WillClose(gfx::NativeWindow dialog) override;
+  bool IsNonClipped(gfx::NativeWindow dialog) const override;
+  void AddNonClippedDialog(gfx::NativeWindow dialog) override;

[Mike Wittman, 2016/08/19 14:57:29]

why do we need this function?

[rbpotter, 2016/08/19 16:59:42]

I added it in since ShowModalDialog moved to show_modal_dialog_views.cc in
constrained window (it was previously defined here as a public function in the
web contents modal dialog manager). I wanted to keep ShowNonClippedModalDialog
with ShowModalDialog so I moved it to
constrained_window/show_modal_dialog_views.cc. From this new location it can't
access the nonclipped_child_dialogs_. The dialogs still need to be added to that
set though, so I added this function to allow the new ShowNonClippedModalDialog
function to add the dialogs to the manager like it did before. Should I just
make the nonclipped_child_dialogs_ public instead? 

[Mike Wittman, 2016/08/19 17:51:42]

I see. This approach is kind of odd because at the call site it's telling the
manager how to deal with the dialog before it has started managing it.

Given the new state of the code, I think a better approach would be to pass an
is_nonclipped parameter to the NativeWebContentsModalDialogManagerViews
constructor and through to ManageDialog, and revert all the changes in this
file.

 
   // For testing.
   class TestApi {
@@ -115,6 +118,9 @@ class WebContentsModalDialogManager
   // All active dialogs.
   WebContentsModalDialogList child_dialogs_;
 
+  // All non clipped active dialogs
+  std::set<gfx::NativeWindow> nonclipped_child_dialogs_;
+
   // True while closing the dialogs on WebContents close.
   bool closing_all_dialogs_;