Refactor and fix feedback

chrome/browser/ui/webui/bidi_checker_web_ui_test.cc

 // Copyright (c) 2012 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/webui/bidi_checker_web_ui_test.h"
 
 #include "base/base_paths.h"
 #include "base/i18n/rtl.h"
 #include "base/path_service.h"
 #include "base/utf_string_conversions.h"
@@ skipping 119 matching lines @@
 //==============================
 
 // This page isn't localized to an RTL language so we test it only in English.
 IN_PROC_BROWSER_TEST_F(WebUIBidiCheckerBrowserTestLTR, TestAboutPage) {
   RunBidiCheckerOnPage(chrome::kChromeUIAboutURL);
 }
 
 //==============================
 // chrome://bugreport
 //==============================
-// TestBugReportPage fails on chromeos Debug build. crbug.com/105631
+// TestFeedbackPage fails on chromeos Debug build. crbug.com/105631
 #if defined(OS_CHROMEOS)
-#define MAYBE_TestBugReportPage DISABLED_TestBugReportPage
+#define MAYBE_TestFeedbackPage DISABLED_TestFeedbackPage
 #else
-#define MAYBE_TestBugReportPage TestBugReportPage
+#define MAYBE_TestFeedbackPage TestFeedbackPage
 #endif
 
-IN_PROC_BROWSER_TEST_F(WebUIBidiCheckerBrowserTestLTR, MAYBE_TestBugReportPage)
-{
+IN_PROC_BROWSER_TEST_F(WebUIBidiCheckerBrowserTestLTR,
+                       MAYBE_TestFeedbackPage) {
   // The bugreport page receives its contents as GET arguments. Here we provide
   // a custom, Hebrew typed, description message.
   RunBidiCheckerOnPage(
-      "chrome://bugreport#0?description=%D7%91%D7%93%D7%99%D7%A7%D7%94"
-      "&issueType=1");
+      "chrome://feedback#0?description=%D7%91%D7%93%D7%99%D7%A7%D7%94");
 }
 
 IN_PROC_BROWSER_TEST_F(WebUIBidiCheckerBrowserTestRTL,
-                       MAYBE_TestBugReportPage) {
-  RunBidiCheckerOnPage("chrome://bugreport#0?description=test&issueType=1");
+                       MAYBE_TestFeedbackPage) {
+  RunBidiCheckerOnPage("chrome://feedback#0?description=test");
 }
 
 //==============================
 // chrome://crashes
 //==============================
 
 IN_PROC_BROWSER_TEST_F(WebUIBidiCheckerBrowserTestLTR, TestCrashesPage) {
   RunBidiCheckerOnPage(chrome::kChromeUICrashesURL);
 }
 
@@ skipping 256 matching lines @@
   url += std::string(chrome::kSearchEnginesSubPage);
   RunBidiCheckerOnPage(url.c_str());
 }
 
 IN_PROC_BROWSER_TEST_F(WebUIBidiCheckerBrowserTestRTL,
                        TestSettingsSearchEnginesOptionsPage) {
   std::string url(chrome::kChromeUISettingsURL);
   url += std::string(chrome::kSearchEnginesSubPage);
   RunBidiCheckerOnPage(url.c_str());
 }