Eliminate chrome::IsTrustedWindowWithScheme

chrome/browser/ui/settings_window_manager.cc

 // Copyright (c) 2014 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/settings_window_manager.h"
 
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_navigator.h"
@@ skipping 36 matching lines @@
     }
     NavigateParams params(browser, gurl,
                           ui::PAGE_TRANSITION_AUTO_BOOKMARK);
     params.window_action = NavigateParams::SHOW_WINDOW;
     params.user_gesture = true;
     chrome::Navigate(&params);
     return;
   }
 
   // No existing browser window, create one.
-  NavigateParams params(profile, gurl, ui::PAGE_TRANSITION_AUTO_BOOKMARK);
-  params.disposition = NEW_POPUP;
-  params.trusted_source = true;
-  params.window_action = NavigateParams::SHOW_WINDOW;
-  params.user_gesture = true;
-  params.path_behavior = NavigateParams::IGNORE_AND_NAVIGATE;
-  chrome::Navigate(&params);
-  settings_session_map_[profile] = params.browser->session_id().id();
+  navigate_params_.reset(
+      new NavigateParams(profile, gurl, ui::PAGE_TRANSITION_AUTO_BOOKMARK));
+  navigate_params_->disposition = NEW_POPUP;
+  navigate_params_->trusted_source = true;
+  navigate_params_->window_action = NavigateParams::SHOW_WINDOW;
+  navigate_params_->user_gesture = true;
+  navigate_params_->path_behavior = NavigateParams::IGNORE_AND_NAVIGATE;
+  chrome::Navigate(navigate_params_.get());
+  settings_session_map_[profile] = navigate_params_->browser->session_id().id();
 
   FOR_EACH_OBSERVER(SettingsWindowManagerObserver,
-                    observers_, OnNewSettingsWindow(params.browser));
+                    observers_, OnNewSettingsWindow(navigate_params_->browser));

[Mr4D (OOO till 08-26), 2014/09/25 21:18:34]

At this time you could destroy the navigate_params_ again.

[stevenjb, 2014/09/25 21:36:58]

Good suggestion now that I think about it, will do.

 }
 
 Browser* SettingsWindowManager::FindBrowserForProfile(Profile* profile) {
   ProfileSessionMap::iterator iter = settings_session_map_.find(profile);
   if (iter != settings_session_map_.end())
     return chrome::FindBrowserWithID(iter->second);
   return NULL;
 }
 
 bool SettingsWindowManager::IsSettingsBrowser(Browser* browser) const {
+  // This may get called from chrome::Navigate, so first test to see if this
+  // is the browser we most recently navigate to.

[Mr4D (OOO till 08-26), 2014/09/25 21:18:34]

.. most recently navigated to.

[stevenjb, 2014/09/25 21:36:58]

Done.

+  if (navigate_params_ && navigate_params_->browser == browser)

[Mr4D (OOO till 08-26), 2014/09/25 21:18:34]

Don't really like this, but I do not see a good other way to do this either. [A
window property wouldn't help here, and looking for the requested GURL isn't
better either].

[stevenjb, 2014/09/25 21:36:58]

Right, pretty much the same conclusion I came to. I do like resetting
navigate_params_ after Navigate completes and settings_session_map_ is set
however.

+    return true;
   ProfileSessionMap::const_iterator iter =
       settings_session_map_.find(browser->profile());
   return (iter != settings_session_map_.end() &&
           iter->second == browser->session_id().id());
 }
 
 SettingsWindowManager::SettingsWindowManager() {
 }
 
 SettingsWindowManager::~SettingsWindowManager() {
 }
 
 }  // namespace chrome