Index: chrome/browser/feedback/feedback_dialog_utils.cc |
diff --git a/chrome/browser/feedback/show_feedback_page.cc b/chrome/browser/feedback/feedback_dialog_utils.cc |
similarity index 52% |
copy from chrome/browser/feedback/show_feedback_page.cc |
copy to chrome/browser/feedback/feedback_dialog_utils.cc |
index 0c04809ebb0fd6ce89b219d38deaee5dbdc6da7e..56a71db50843b18b45b261ae2d889ddbd575fcf8 100644 |
--- a/chrome/browser/feedback/show_feedback_page.cc |
+++ b/chrome/browser/feedback/feedback_dialog_utils.cc |
@@ -1,26 +1,25 @@ |
-// Copyright 2014 The Chromium Authors. All rights reserved. |
+// Copyright 2016 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 <string> |
+#include "chrome/browser/feedback/feedback_dialog_utils.h" |
-#include "build/build_config.h" |
-#include "chrome/browser/browser_process.h" |
-#include "chrome/browser/extensions/api/feedback_private/feedback_private_api.h" |
#include "chrome/browser/profiles/profile.h" |
#include "chrome/browser/profiles/profile_manager.h" |
-#include "chrome/browser/ui/ash/multi_user/multi_user_util.h" |
-#include "chrome/browser/ui/ash/multi_user/multi_user_window_manager.h" |
+#include "chrome/browser/ui/browser.h" |
#include "chrome/browser/ui/browser_finder.h" |
-#include "chrome/browser/ui/browser_window.h" |
#include "chrome/browser/ui/tabs/tab_strip_model.h" |
-#include "chrome/browser/ui/webui/md_feedback/md_feedback_dialog_controller.h" |
-#include "chrome/common/chrome_switches.h" |
-#include "components/signin/core/account_id/account_id.h" |
#include "content/public/browser/web_contents.h" |
#include "url/gurl.h" |
-namespace { |
+#if defined(OS_CHROMEOS) |
+#include "chrome/browser/ui/ash/multi_user/multi_user_util.h" |
+#include "chrome/browser/ui/ash/multi_user/multi_user_window_manager.h" |
+#include "chrome/browser/ui/browser_window.h" |
+#include "components/signin/core/account_id/account_id.h" |
+#endif |
+ |
+namespace chrome { |
GURL GetTargetTabUrl(int session_id, int index) { |
Browser* browser = chrome::FindBrowserWithID(session_id); |
@@ -38,29 +37,12 @@ GURL GetTargetTabUrl(int session_id, int index) { |
return GURL(); |
} |
-} // namespace |
- |
-namespace chrome { |
- |
-void ShowFeedbackPage(Browser* browser, |
- const std::string& description_template, |
- const std::string& category_tag) { |
- GURL page_url; |
- if (browser) { |
- page_url = GetTargetTabUrl(browser->session_id().id(), |
- browser->tab_strip_model()->active_index()); |
- } |
- |
- Profile* profile = NULL; |
- if (browser) { |
- profile = browser->profile(); |
- } else { |
- profile = ProfileManager::GetLastUsedProfileAllowedByPolicy(); |
- } |
- if (!profile) { |
- LOG(ERROR) << "Cannot invoke feedback: No profile found!"; |
- return; |
- } |
+Profile* GetFeedbackProfile(Browser* browser) { |
+ Profile* profile = |
+ browser ? browser->profile() |
+ : ProfileManager::GetLastUsedProfileAllowedByPolicy(); |
+ if (!profile) |
+ return nullptr; |
// We do not want to launch on an OTR profile. |
profile = profile->GetOriginalProfile(); |
@@ -75,22 +57,10 @@ void ShowFeedbackPage(Browser* browser, |
? window_manager->GetUserPresentingWindow( |
browser->window()->GetNativeWindow()) |
: EmptyAccountId(); |
- profile = display_account_id.is_valid() |
- ? multi_user_util::GetProfileFromAccountId(display_account_id) |
- : profile; |
+ if (display_account_id.is_valid()) |
+ profile = multi_user_util::GetProfileFromAccountId(display_account_id); |
#endif |
- |
- if (::switches::MdFeedbackEnabled()) { |
- MdFeedbackDialogController::GetInstance()->Show(profile); |
- return; |
- } |
- |
- extensions::FeedbackPrivateAPI* api = |
- extensions::FeedbackPrivateAPI::GetFactoryInstance()->Get(profile); |
- |
- api->RequestFeedback(description_template, |
- category_tag, |
- page_url); |
+ return profile; |
} |
} // namespace chrome |