Files.app: Fixed the problem that dialog opened by Ctrl+O cannot be expand.

chrome/browser/ui/views/extensions/extension_dialog.cc

 // 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.
 
 #include "chrome/browser/ui/views/extensions/extension_dialog.h"
 
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 39 matching lines @@
 }
 
 // static
 ExtensionDialog* ExtensionDialog::Show(
     const GURL& url,
     BaseWindow* base_window,
     Profile* profile,
     WebContents* web_contents,
     int width,
     int height,
+    int minWidth,

[sky, 2013/05/08 16:14:42]

min_width, min_height

How do we know we can trust these values? What happens if they are ginormous or
too small?

[hirono, 2013/05/09 02:23:16]

Actually these arguments are added just for calling
NativeViewHost::SetPreferredSize. So how to deal with these values is
depending on each NativeViewHost. Currently ExtensionDialog::Show
is only used from SelectFileDialogExtension and constant values are
passed as these arguments there.

[sky, 2013/05/09 15:51:57]

I was assuming these values are coming from script on a page. If the values
aren't coming from the page then I agree with you.

+    int minHeight,
     const string16& title,
     ExtensionDialogObserver* observer) {
   extensions::ExtensionHost* host = CreateExtensionHost(url, profile);
+  // Preferred size must be set before views::Widget::CreateWindowWithParent
+  // is called because CreateWindowWithParent refers the result of CanResize().
+  host->view()->SetPreferredSize(gfx::Size(minWidth, minHeight));
   if (!host)
     return NULL;
   host->SetAssociatedWebContents(web_contents);
 
   return ExtensionDialog::ShowInternal(url, base_window, host, width, height,
                                        false, title, observer);
 }
 
 #if defined(USE_AURA)
 // static
@@ skipping 209 matching lines @@
       if (content::Details<extensions::ExtensionHost>(host()) != details)
         return;
       if (observer_)
         observer_->ExtensionTerminated(this);
       break;
     default:
       NOTREACHED() << L"Received unexpected notification";
       break;
   }
 }