Refactor and fix feedback

chrome/browser/autofill/autofill_feedback_infobar_delegate.cc

Index: chrome/browser/autofill/autofill_feedback_infobar_delegate.cc
diff --git a/chrome/browser/autofill/autofill_feedback_infobar_delegate.cc b/chrome/browser/autofill/autofill_feedback_infobar_delegate.cc
index d51736b4487fade82a1b2d678d1e2c05e833cd82..f5a4350f4fedc37bbaea7906bd398f94cc0a6b75 100644
--- a/chrome/browser/autofill/autofill_feedback_infobar_delegate.cc
+++ b/chrome/browser/autofill/autofill_feedback_infobar_delegate.cc
@@ -7,12 +7,14 @@
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/infobars/infobar_tab_helper.h"
 #include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/webui/bug_report_ui.h"
-#include "chrome/browser/userfeedback/proto/extension.pb.h"
+#include "chrome/browser/ui/webui/feedback_ui.h"
+#include "chrome/browser/feedback/proto/extension.pb.h"
 #include "content/browser/tab_contents/navigation_details.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "googleurl/src/gurl.h"
 
+const char kCategoryTagAutofill[] = "Autofill";

[xiyuan, 2011/12/22 02:18:23]

"static" or anonymous namespace if we don't use it outside this file.

[tfarina, 2011/12/22 02:20:43]

const has internal linkage so it doesn't need either static or unnamed namespace
in C++. If still you want to go ahead, wrap it into an unnamed namespace.

namespace {
...
}  // namespace

[rkc, 2011/12/22 22:41:46]

Confirmed that the C++ standard does specify internal linkage for const; also
tested on gcc to confirm so leaving it as is.

+
 AutofillFeedbackInfoBarDelegate::AutofillFeedbackInfoBarDelegate(
     InfoBarTabHelper* infobar_helper,
     const string16& message,
@@ -41,17 +43,10 @@ string16 AutofillFeedbackInfoBarDelegate::GetLinkText() const {
 
 bool AutofillFeedbackInfoBarDelegate::LinkClicked(
     WindowOpenDisposition disposition) {
-#if defined(OS_CHROMEOS)
-  size_t issue_type = userfeedback::ChromeOsData_ChromeOsCategory_AUTOFILL;
-#else
-  size_t issue_type =
-      userfeedback::ChromeBrowserData_ChromeBrowserCategory_AUTOFILL;
-#endif
-
-  browser::ShowHtmlBugReportView(
+  browser::ShowHtmlFeedbackView(
       Browser::GetBrowserForController(&owner()->tab_contents()->controller(),
                                        NULL),
       feedback_message_,
-      issue_type);
+      std::string(kCategoryTagAutofill));
   return true;
 }