| 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
|
|
|