Cleanup popup related browser navigation code.

chrome/browser/ui/webui/html_dialog_tab_contents_delegate.cc

 // Copyright (c) 2011 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/webui/html_dialog_tab_contents_delegate.h"
 
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_navigator.h"
@@ skipping 26 matching lines @@
     Browser* browser = NULL;
     browser::NavigateParams params(browser, url, transition);
     params.profile = profile_;
     params.referrer = referrer;
     if (source && source->is_crashed() && disposition == CURRENT_TAB &&
         transition == PageTransition::LINK)
       params.disposition = NEW_FOREGROUND_TAB;
     else
       params.disposition = disposition;
     params.window_action = browser::NavigateParams::SHOW_WINDOW;
+    params.user_gesture = true;
     browser::Navigate(&params);
   }
 }
 
 void HtmlDialogTabContentsDelegate::NavigationStateChanged(
     const TabContents* source, unsigned changed_flags) {
   // We shouldn't receive any NavigationStateChanged except the first
   // one, which we ignore because we're a dialog box.
 }
 
 void HtmlDialogTabContentsDelegate::AddNewContents(
     TabContents* source, TabContents* new_contents,
     WindowOpenDisposition disposition, const gfx::Rect& initial_pos,
     bool user_gesture) {
   if (profile_) {
     // Specify a NULL browser for navigation. This will cause Navigate()
     // to find a browser matching params.profile or create a new one.
     Browser* browser = NULL;
 
     TabContentsWrapper* wrapper = new TabContentsWrapper(new_contents);
     browser::NavigateParams params(browser, wrapper);
     params.profile = profile_;
     // TODO(pinkerton): no way to get a wrapper for this.
     // params.source_contents = source;
     params.disposition = disposition;
     params.window_bounds = initial_pos;
     params.window_action = browser::NavigateParams::SHOW_WINDOW;
+    params.user_gesture = true;
     browser::Navigate(&params);
   }
 }
 
 void HtmlDialogTabContentsDelegate::ActivateContents(TabContents* contents) {
   // We don't do anything here because there's only one TabContents in
   // this frame and we don't have a TabStripModel.
 }
 
 void HtmlDialogTabContentsDelegate::DeactivateContents(TabContents* contents) {
@@ skipping 19 matching lines @@
 void HtmlDialogTabContentsDelegate::UpdateTargetURL(TabContents* source,
                                                     const GURL& url) {
   // Ignored.
 }
 
 bool HtmlDialogTabContentsDelegate::ShouldAddNavigationToHistory(
     const history::HistoryAddPageArgs& add_page_args,
     NavigationType::Type navigation_type) {
   return false;
 }