Don't display the file select dialog from invisible windows....

chrome/browser/blocked_popup_container.cc

 // Copyright (c) 2010 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/blocked_popup_container.h"
 
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "gfx/rect.h"
 
 // static
@@ skipping 17 matching lines @@
                                            const gfx::Rect& bounds) {
   if (blocked_popups_.size() == (kImpossibleNumberOfPopups - 1)) {
     delete tab_contents;
     LOG(INFO) << "Warning: Renderer is sending more popups to us than should "
         "be possible. Renderer compromised?";
     return;
   }
 
   blocked_popups_.push_back(BlockedPopup(tab_contents, bounds));
   tab_contents->set_delegate(this);
+  // Since the new tab_contents will not be showed, call WasHidden to change
+  // its status on both RenderViewHost and RenderView.
+  tab_contents->WasHidden();
   if (blocked_popups_.size() == 1)
     owner_->PopupNotificationVisibilityChanged(true);
 }
 
 void BlockedPopupContainer::LaunchPopupForContents(TabContents* tab_contents) {
   // Open the popup.
   for (BlockedPopups::iterator i(blocked_popups_.begin());
        i != blocked_popups_.end(); ++i) {
     if (i->tab_contents == tab_contents) {
       tab_contents->set_delegate(NULL);
+      // We needn't call WasRestored to change its status because the
+      // TabContents::AddNewContents will do it.
       owner_->AddNewContents(tab_contents, NEW_POPUP, i->bounds, true);
       blocked_popups_.erase(i);
       break;
     }
   }
 
   if (blocked_popups_.empty())
     Destroy();
 }
 
@@ skipping 64 matching lines @@
 }
 
 bool BlockedPopupContainer::IsPopup(const TabContents* source) const {
   return true;
 }
 
 TabContents* BlockedPopupContainer::GetConstrainingContents(
     TabContents* source) {
   return owner_;
 }