Refactor and fix feedback

This CL does the following things,

.) Removes user-facing categories from the UI completely.
.) Changes categories that are still used (Autofill for example) use category tags instead of the category enum (which has been deprecated)
.) Updates the protocol buffers
.) Refactors the code to change bug report to feedback and re-organizes the file naming/layout
.) Cleans up the unused feedback strings, files and code
.) Cleans up the code at a few places
.) Makes a few UI changes to get closer to the mock given in chromium:98738

This seems like a really huge CL but in reality it just just a couple of hundred lines of 'code' changes; a lot of this is just code being deleted

BUG=chromium:98738
TEST=Tested sending reports.

Review URL: http://codereview.chromium.org/9006003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116598 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 118 insertions, 0 deletions

diff --git a/chrome/browser/feedback/feedback_data.cc b/chrome/browser/feedback/feedback_data.cc
new file mode 100644
index 0000000..60903c7
--- /dev/null
+++ b/chrome/browser/feedback/feedback_data.cc
@@ -0,0 +1,118 @@
+// 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/feedback/feedback_data.h"
+
+#include "chrome/browser/feedback/feedback_util.h"
+#include "content/public/browser/browser_thread.h"
+
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/notifications/system_notification.h"
+#endif
+
+using content::BrowserThread;
+
+FeedbackData::FeedbackData()
+    : profile_(NULL)
+#if defined(OS_CHROMEOS)
+    , sys_info_(NULL)
+    , zip_content_(NULL)
+    , sent_report_(false)
+    , send_sys_info_(false)
+#endif
+{
+}
+
+FeedbackData::~FeedbackData() {}
+
+void FeedbackData::UpdateData(Profile* profile,
+                               const std::string& target_tab_url,
+                               const std::string& category_tag,
+                               const std::string& page_url,
+                               const std::string& description,
+                               ScreenshotDataPtr image
+#if defined(OS_CHROMEOS)
+                               , const std::string& user_email
+                               , const bool send_sys_info
+                               , const bool sent_report
+#endif
+                               ) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  profile_ = profile;
+  target_tab_url_ = target_tab_url;
+  category_tag_ = category_tag;
+  page_url_ = page_url;
+  description_ = description;
+  image_ = image;
+#if defined(OS_CHROMEOS)
+  user_email_ = user_email;
+  send_sys_info_ = send_sys_info;
+  sent_report_ = sent_report;
+#endif
+}
+
+void FeedbackData::SendReport() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+#if defined(OS_CHROMEOS)
+  if (sent_report_)
+    return;  // We already received the syslogs and sent the report.
+
+  // Set send_report_ to ensure that we only send one report.
+  sent_report_ = true;
+#endif
+
+  gfx::Rect& screen_size = FeedbackUtil::GetScreenshotSize();
+  FeedbackUtil::SendReport(profile_
+                            , category_tag_
+                            , page_url_
+                            , description_
+                            , image_
+                            , screen_size.width()
+                            , screen_size.height()
+#if defined(OS_CHROMEOS)
+                            , user_email_
+                            , zip_content_ ? zip_content_->c_str() : NULL
+                            , zip_content_ ? zip_content_->length() : 0
+                            , send_sys_info_ ? sys_info_ : NULL
+#endif
+                          );
+
+#if defined(OS_CHROMEOS)
+  if (sys_info_) {
+    delete sys_info_;
+    sys_info_ = NULL;
+  }
+  if (zip_content_) {
+    delete zip_content_;
+    zip_content_ = NULL;
+  }
+#endif
+
+  // Delete this object once the report has been sent.
+  delete this;
+}
+
+#if defined(OS_CHROMEOS)
+// SyslogsComplete may be called before UpdateData, in which case, we do not
+// want to delete the logs that were gathered, and we do not want to send the
+// report either. Instead simply populate |sys_info_| and |zip_content_|.
+void FeedbackData::SyslogsComplete(chromeos::system::LogDictionaryType* logs,
+                                    std::string* zip_content) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  if (sent_report_) {
+    // We already sent the report, just delete the data.
+    if (logs)
+      delete logs;
+    if (zip_content)
+      delete zip_content;
+  } else {
+    zip_content_ = zip_content;
+    sys_info_ = logs;  // Will get deleted when SendReport() is called.
+    if (send_sys_info_) {
+      // We already prepared the report, send it now.
+      this->SendReport();
+    }
+  }
+}
+#endif