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

components/web_modal/web_contents_modal_dialog_manager.h

 // Copyright 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 COMPONENTS_WEB_MODAL_WEB_CONTENTS_MODAL_DIALOG_MANAGER_H_
 #define COMPONENTS_WEB_MODAL_WEB_CONTENTS_MODAL_DIALOG_MANAGER_H_
 
 #include <deque>
 #include <memory>
+#include <set>
 
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "components/web_modal/single_web_contents_dialog_manager.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
 #include "ui/gfx/native_widget_types.h"
 
 namespace web_modal {
 
@@ skipping 27 matching lines @@
   // Returns true if any dialogs are active and not closed.
   bool IsDialogActive() const;
 
   // Focus the topmost modal dialog.  IsDialogActive() must be true when calling
   // this function.
   void FocusTopmostDialog() const;
 
   // 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 {
    public:
     explicit TestApi(WebContentsModalDialogManager* manager)
         : manager_(manager) {}
 
     void CloseAllDialogs() { manager_->CloseAllDialogs(); }
     void DidAttachInterstitialPage() { manager_->DidAttachInterstitialPage(); }
     void WebContentsWasShown() { manager_->WasShown(); }
@@ skipping 41 matching lines @@
   void WasHidden() override;
   void WebContentsDestroyed() override;
   void DidAttachInterstitialPage() override;
 
   // Delegate for notifying our owner about stuff. Not owned by us.
   WebContentsModalDialogManagerDelegate* delegate_;
 
   // 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_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsModalDialogManager);
 };
 
 }  // namespace web_modal
 
 #endif  // COMPONENTS_WEB_MODAL_WEB_CONTENTS_MODAL_DIALOG_MANAGER_H_