Refactor and fix feedback

chrome/test/data/webui/bidichecker_tests.js

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
 // The following test runs the BiDi Checker on the current page with LTR
 // base direction. Any bad mixture or RTL and LTR will be reported back
 // by this test.
 
 function filtersForPage(pageName, isRTL) {
   // Suppression filters should be added to the object below. You may omit
   // either RTL or LTR filters, or may not include filters for your page at all.
@@ skipping 44 matching lines @@
         bidichecker.FilterFactory.locationClass('plugin-text')
       ]
     },
     "chrome://newtab" : {
       "RTL" : [
         // BUG: http://crbug.com/93339
         bidichecker.FilterFactory.atText("Chrome Web Store"),
         bidichecker.FilterFactory.atText("File Manager")
       ]
     },
-"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" :
     {
       "LTR" : [
         // BUG: http://crbug.com/90835
         bidichecker.FilterFactory.atText("בדיקה")
       ]
     },
-    "chrome://bugreport#0?description=test&issueType=1" : {
+    "chrome://feedback#0?description=test" : {
       "RTL" : [
         // BUG: http://crbug.com/90835
         bidichecker.FilterFactory.atText("test"),
         bidichecker.FilterFactory.atText("stub-user@example.com")
       ]
     },
     "chrome://settings/browser" : {
       "LTR" : [
         // BUG: http://crbug.com/93702
         bidichecker.FilterFactory.atText(
@@ skipping 56 matching lines @@
     prettyErrors += bidiErrors[idx].toString();
   }
   prettyErrors += '\n\n';
   return prettyErrors;
 }
 
 function runBidiChecker(pageName, isRTL) {
   var filters = filtersForPage(pageName, isRTL);
   var bidiErrors = bidichecker.checkPage(isRTL, top.document.body, filters);
   assertTrue(bidiErrors.length == 0, buildPrettyErrors(bidiErrors));
-}
+}