Index: chrome/browser/ui/toolbar/wrench_menu_model.cc |
diff --git a/chrome/browser/ui/toolbar/wrench_menu_model.cc b/chrome/browser/ui/toolbar/wrench_menu_model.cc |
index 37f0f5cba809654208bb7f7863ac098461a48ab7..c3bc26d65bb3c13422a375a03ab3698b7ae73cb5 100644 |
--- a/chrome/browser/ui/toolbar/wrench_menu_model.cc |
+++ b/chrome/browser/ui/toolbar/wrench_menu_model.cc |
@@ -30,6 +30,7 @@ |
#include "chrome/browser/ui/global_error/global_error.h" |
#include "chrome/browser/ui/global_error/global_error_service.h" |
#include "chrome/browser/ui/global_error/global_error_service_factory.h" |
+#include "chrome/browser/ui/send_feedback_experiment.h" |
#include "chrome/browser/ui/tabs/tab_strip_model.h" |
#include "chrome/browser/ui/toolbar/bookmark_sub_menu_model.h" |
#include "chrome/browser/ui/toolbar/encoding_menu_controller.h" |
@@ -85,6 +86,7 @@ string16 GetUpgradeDialogMenuItemName() { |
return l10n_util::GetStringUTF16(IDS_UPDATE_NOW); |
} |
} |
+ |
} // namespace |
//////////////////////////////////////////////////////////////////////////////// |
@@ -202,8 +204,17 @@ void ToolsMenuModel::Build(Browser* browser) { |
#if !defined(OS_CHROMEOS) |
// Show IDC_FEEDBACK in "Tools" menu for non-ChromeOS platforms. |
- AddItemWithStringId(IDC_FEEDBACK, IDS_FEEDBACK); |
- AddSeparator(ui::NORMAL_SEPARATOR); |
+ if (!chrome::UseAlternateSendFeedbackLocation()) { |
+ AddItemWithStringId(IDC_FEEDBACK, |
+ chrome::GetSendFeedbackMenuLabelID()); |
+ AddSeparator(ui::NORMAL_SEPARATOR); |
+ } |
+#else |
+ if (chrome::UseAlternateSendFeedbackLocation()) { |
+ AddItemWithStringId(IDC_FEEDBACK, |
+ chrome::GetSendFeedbackMenuLabelID()); |
+ AddSeparator(ui::NORMAL_SEPARATOR); |
+ } |
#endif |
encoding_menu_model_.reset(new EncodingMenuModel(browser)); |
@@ -610,8 +621,11 @@ void WrenchMenuModel::Build(bool is_new_menu, bool supports_new_separators) { |
} |
} |
- if (browser_defaults::kShowFeedbackMenuItem) |
- AddItemWithStringId(IDC_FEEDBACK, IDS_FEEDBACK); |
+ if (browser_defaults::kShowFeedbackMenuItem && |
+ !chrome::UseAlternateSendFeedbackLocation()) { |
+ AddItemWithStringId(IDC_FEEDBACK, |
+ chrome::GetSendFeedbackMenuLabelID()); |
+ } |
AddGlobalErrorMenuItems(); |
@@ -625,10 +639,20 @@ void WrenchMenuModel::Build(bool is_new_menu, bool supports_new_separators) { |
if (browser_->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_ASH) |
show_exit_menu = false; |
#endif |
- if (show_exit_menu) { |
+ if (show_exit_menu) |
+ AddSeparator(ui::NORMAL_SEPARATOR); |
+ |
+#if !defined(OS_CHROMEOS) |
+ // For Send Feedback Link experiment (crbug.com/169339). |
+ if (chrome::UseAlternateSendFeedbackLocation()) { |
+ AddItemWithStringId(IDC_FEEDBACK, |
+ chrome::GetSendFeedbackMenuLabelID()); |
AddSeparator(ui::NORMAL_SEPARATOR); |
- AddItemWithStringId(IDC_EXIT, IDS_EXIT); |
} |
+#endif |
+ |
+ if (show_exit_menu) |
+ AddItemWithStringId(IDC_EXIT, IDS_EXIT); |
if (is_new_menu && supports_new_separators && |
!ui::NativeTheme::IsNewMenuStyleEnabled()) { |