Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6805)

Unified Diff: chrome/browser/download/download_ui_controller.cc

Issue 1659203002: Remove HostDesktopType from FindLastActive[WithProfile] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mac adl Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/download/download_ui_controller.cc
diff --git a/chrome/browser/download/download_ui_controller.cc b/chrome/browser/download/download_ui_controller.cc
index 5b45568db437f388da4e5e7522aeaa11f197b6bb..bc7f57860dbc54664675076a5d8bafe74fd4c05f 100644
--- a/chrome/browser/download/download_ui_controller.cc
+++ b/chrome/browser/download/download_ui_controller.cc
@@ -87,10 +87,8 @@ void DownloadShelfUIControllerDelegate::OnNewDownloadReady(
// As a last resort, use the last active browser for this profile. Not ideal,
// but better than not showing the download at all.
- if (browser == NULL) {
- browser = chrome::FindLastActiveWithProfile(profile_,
- chrome::GetActiveDesktop());
- }
+ if (browser == NULL)
sky 2016/02/02 22:33:10 nit: nullptr
scottmg 2016/02/02 23:08:43 Done.
+ browser = chrome::FindLastActiveWithProfile(profile_);
if (browser && browser->window() &&
DownloadItemModel(item).ShouldShowInShelf()) {

Powered by Google App Engine
This is Rietveld 408576698