Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1416)

Unified Diff: chrome/browser/feedback/feedback_util.cc

Issue 225183018: Move feedback files into src/components (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase to latest Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/feedback/feedback_util.cc
diff --git a/chrome/browser/feedback/feedback_util.cc b/chrome/browser/feedback/feedback_util.cc
deleted file mode 100644
index 23e5406954cf61e82a13d98907b4c15fdf1dc8e6..0000000000000000000000000000000000000000
--- a/chrome/browser/feedback/feedback_util.cc
+++ /dev/null
@@ -1,275 +0,0 @@
-// 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_util.h"
-
-#include <sstream>
-#include <string>
-#include <vector>
-
-#include "base/bind.h"
-#include "base/command_line.h"
-#include "base/file_util.h"
-#include "base/file_version_info.h"
-#include "base/memory/singleton.h"
-#include "base/message_loop/message_loop.h"
-#include "base/strings/string_number_conversions.h"
-#include "base/strings/stringprintf.h"
-#include "base/strings/utf_string_conversions.h"
-#include "base/win/windows_version.h"
-#include "chrome/browser/browser_process.h"
-#include "chrome/browser/extensions/api/feedback_private/feedback_private_api.h"
-#include "chrome/browser/feedback/feedback_data.h"
-#include "chrome/browser/feedback/feedback_uploader.h"
-#include "chrome/browser/feedback/feedback_uploader_factory.h"
-#include "chrome/browser/metrics/variations/variations_http_header_provider.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/profiles/profile_manager.h"
-#include "chrome/browser/safe_browsing/safe_browsing_util.h"
-#include "chrome/browser/ui/browser_finder.h"
-#include "chrome/browser/ui/browser_list.h"
-#include "chrome/browser/ui/browser_window.h"
-#include "chrome/browser/ui/tabs/tab_strip_model.h"
-#include "chrome/common/chrome_content_client.h"
-#include "chrome/common/chrome_switches.h"
-#include "chrome/common/chrome_version_info.h"
-#include "components/metrics/metrics_log_manager.h"
-#include "content/public/browser/navigation_controller.h"
-#include "content/public/browser/web_contents.h"
-#include "grit/generated_resources.h"
-#include "grit/locale_settings.h"
-#include "grit/theme_resources.h"
-#include "net/base/load_flags.h"
-#include "net/http/http_request_headers.h"
-#include "net/url_request/url_fetcher.h"
-#include "net/url_request/url_fetcher_delegate.h"
-#include "net/url_request/url_request_status.h"
-#include "third_party/icu/source/common/unicode/locid.h"
-#include "third_party/zlib/google/zip.h"
-#include "ui/base/l10n/l10n_util.h"
-#include "url/gurl.h"
-
-namespace {
-
-GURL GetTargetTabUrl(int session_id, int index) {
- Browser* browser = chrome::FindBrowserWithID(session_id);
- // Sanity checks.
- if (!browser || index >= browser->tab_strip_model()->count())
- return GURL();
-
- if (index >= 0) {
- content::WebContents* target_tab =
- browser->tab_strip_model()->GetWebContentsAt(index);
- if (target_tab)
- return target_tab->GetURL();
- }
-
- return GURL();
-}
-
-const char kPngMimeType[] = "image/png";
-const char kArbitraryMimeType[] = "application/octet-stream";
-const char kHistogramsAttachmentName[] = "histograms.zip";
-const char kLogsAttachmentName[] = "system_logs.zip";
-
-#if defined(OS_CHROMEOS)
-const int kChromeOSProductId = 208;
-#else
-const int kChromeBrowserProductId = 237;
-#endif
-
-void AddFeedbackData(userfeedback::ExtensionSubmit* feedback_data,
- const std::string& key, const std::string& value) {
- // Don't bother with empty keys or values
- if (key == "" || value == "") return;
- // Create log_value object and add it to the web_data object
- userfeedback::ProductSpecificData log_value;
- log_value.set_key(key);
- log_value.set_value(value);
- userfeedback::WebData* web_data = feedback_data->mutable_web_data();
- *(web_data->add_product_specific_data()) = log_value;
-}
-
-// Adds data as an attachment to feedback_data if the data is non-empty.
-void AddAttachment(userfeedback::ExtensionSubmit* feedback_data,
- const char* name,
- std::string* data) {
- if (data == NULL || data->empty())
- return;
-
- userfeedback::ProductSpecificBinaryData* attachment =
- feedback_data->add_product_specific_binary_data();
- attachment->set_mime_type(kArbitraryMimeType);
- attachment->set_name(name);
- attachment->set_data(*data);
-}
-
-} // namespace
-
-namespace chrome {
-
-const char kAppLauncherCategoryTag[] = "AppLauncher";
-
-void ShowFeedbackPage(Browser* browser,
- const std::string& description_template,
- const std::string& category_tag) {
- GURL page_url;
- if (browser) {
- page_url = GetTargetTabUrl(browser->session_id().id(),
- browser->tab_strip_model()->active_index());
- }
-
- Profile* profile = NULL;
- if (browser) {
- profile = browser->profile();
- } else {
- profile = ProfileManager::GetLastUsedProfileAllowedByPolicy();
- }
- if (!profile) {
- LOG(ERROR) << "Cannot invoke feedback: No profile found!";
- return;
- }
-
- // We do not want to launch on an OTR profile.
- profile = profile->GetOriginalProfile();
- DCHECK(profile);
-
- extensions::FeedbackPrivateAPI* api =
- extensions::FeedbackPrivateAPI::GetFactoryInstance()->Get(profile);
-
- api->RequestFeedback(description_template,
- category_tag,
- page_url);
-}
-
-} // namespace chrome
-
-namespace feedback_util {
-
-void SendReport(scoped_refptr<FeedbackData> data) {
- if (!data.get()) {
- LOG(ERROR) << "SendReport called with NULL data!";
- NOTREACHED();
- return;
- }
-
- userfeedback::ExtensionSubmit feedback_data;
- // Unused field, needs to be 0 though.
- feedback_data.set_type_id(0);
-
- userfeedback::CommonData* common_data = feedback_data.mutable_common_data();
- // We're not using gaia ids, we're using the e-mail field instead.
- common_data->set_gaia_id(0);
- common_data->set_user_email(data->user_email());
- common_data->set_description(data->description());
-
- std::string chrome_locale = g_browser_process->GetApplicationLocale();
- common_data->set_source_description_language(chrome_locale);
-
- userfeedback::WebData* web_data = feedback_data.mutable_web_data();
- web_data->set_url(data->page_url());
- web_data->mutable_navigator()->set_user_agent(GetUserAgent());
-
- gfx::Rect screen_size;
- if (data->sys_info()) {
- for (FeedbackData::SystemLogsMap::const_iterator i =
- data->sys_info()->begin(); i != data->sys_info()->end(); ++i) {
- if (FeedbackData::BelowCompressionThreshold(i->second))
- AddFeedbackData(&feedback_data, i->first, i->second);
- }
-
- AddAttachment(&feedback_data, kLogsAttachmentName, data->compressed_logs());
- }
-
- if (data->histograms()) {
- AddAttachment(&feedback_data,
- kHistogramsAttachmentName,
- data->compressed_histograms());
- }
-
- if (!data->attached_filename().empty()) {
- // We need to use the UTF8Unsafe methods here to accomodate Windows, which
- // uses wide strings to store filepaths.
- std::string name = base::FilePath::FromUTF8Unsafe(
- data->attached_filename()).BaseName().AsUTF8Unsafe();
- AddAttachment(&feedback_data, name.c_str(), data->attached_filedata());
- }
-
- // NOTE: Screenshot needs to be processed after system info since we'll get
- // the screenshot dimensions from system info.
- if (data->image() && data->image()->size()) {
- userfeedback::PostedScreenshot screenshot;
- screenshot.set_mime_type(kPngMimeType);
-
- // Set that we 'have' dimensions of the screenshot. These dimensions are
- // ignored by the server but are a 'required' field in the protobuf.
- userfeedback::Dimensions dimensions;
- dimensions.set_width(0.0);
- dimensions.set_height(0.0);
-
- *(screenshot.mutable_dimensions()) = dimensions;
- screenshot.set_binary_content(*data->image());
-
- *(feedback_data.mutable_screenshot()) = screenshot;
- }
-
- if (data->category_tag().size())
- feedback_data.set_bucket(data->category_tag());
-
- // Set whether we're reporting from ChromeOS or Chrome on another platform.
- userfeedback::ChromeData chrome_data;
-#if defined(OS_CHROMEOS)
- chrome_data.set_chrome_platform(
- userfeedback::ChromeData_ChromePlatform_CHROME_OS);
- userfeedback::ChromeOsData chrome_os_data;
- chrome_os_data.set_category(
- userfeedback::ChromeOsData_ChromeOsCategory_OTHER);
- *(chrome_data.mutable_chrome_os_data()) = chrome_os_data;
- feedback_data.set_product_id(kChromeOSProductId);
-#else
- chrome_data.set_chrome_platform(
- userfeedback::ChromeData_ChromePlatform_CHROME_BROWSER);
- userfeedback::ChromeBrowserData chrome_browser_data;
- chrome_browser_data.set_category(
- userfeedback::ChromeBrowserData_ChromeBrowserCategory_OTHER);
- *(chrome_data.mutable_chrome_browser_data()) = chrome_browser_data;
- feedback_data.set_product_id(kChromeBrowserProductId);
-#endif
-
- *(feedback_data.mutable_chrome_data()) = chrome_data;
-
- // This pointer will eventually get deleted by the PostCleanup class, after
- // we've either managed to successfully upload the report or died trying.
- std::string post_body;
- feedback_data.SerializeToString(&post_body);
-
- feedback::FeedbackUploader *uploader =
- feedback::FeedbackUploaderFactory::GetForBrowserContext(data->profile());
- uploader->QueueReport(post_body);
-}
-
-bool ZipString(const base::FilePath& filename,
- const std::string& data, std::string* compressed_logs) {
- base::FilePath temp_path;
- base::FilePath zip_file;
-
- // Create a temporary directory, put the logs into a file in it. Create
- // another temporary file to receive the zip file in.
- if (!base::CreateNewTempDirectory(base::FilePath::StringType(), &temp_path))
- return false;
- if (base::WriteFile(temp_path.Append(filename), data.c_str(), data.size()) ==
- -1)
- return false;
-
- bool succeed = base::CreateTemporaryFile(&zip_file) &&
- zip::Zip(temp_path, zip_file, false) &&
- base::ReadFileToString(zip_file, compressed_logs);
-
- base::DeleteFile(temp_path, true);
- base::DeleteFile(zip_file, false);
-
- return succeed;
-}
-
-} // namespace feedback_util

Powered by Google App Engine
This is Rietveld 408576698