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

Unified Diff: chrome/browser/extensions/api/feedback_private/feedback_private_api.cc

Issue 55363003: Send compressed histograms with system logs when sending feedback. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase with grt's changes Created 7 years, 1 month 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/extensions/api/feedback_private/feedback_private_api.cc
diff --git a/chrome/browser/extensions/api/feedback_private/feedback_private_api.cc b/chrome/browser/extensions/api/feedback_private/feedback_private_api.cc
index ce46532a126a36e3a9e640ac6a78479ef5e8dcb5..1888ffb59787870511be964ba05a1067a54b2f61 100644
--- a/chrome/browser/extensions/api/feedback_private/feedback_private_api.cc
+++ b/chrome/browser/extensions/api/feedback_private/feedback_private_api.cc
@@ -107,7 +107,8 @@ bool FeedbackPrivateGetStringsFunction::RunImpl() {
SET_STRING("page-url", IDS_FEEDBACK_REPORT_URL_LABEL);
SET_STRING("screenshot", IDS_FEEDBACK_SCREENSHOT_LABEL);
SET_STRING("user-email", IDS_FEEDBACK_USER_EMAIL_LABEL);
- SET_STRING("sysinfo", IDS_FEEDBACK_INCLUDE_SYSTEM_INFORMATION_CHKBOX);
+ SET_STRING("sys-info",
+ IDS_FEEDBACK_INCLUDE_SYSTEM_INFORMATION_AND_METRICS_CHKBOX);
SET_STRING("attach-file-label", IDS_FEEDBACK_ATTACH_FILE_LABEL);
SET_STRING("attach-file-note", IDS_FEEDBACK_ATTACH_FILE_NOTE);
SET_STRING("attach-file-to-big", IDS_FEEDBACK_ATTACH_FILE_TO_BIG);
@@ -215,10 +216,19 @@ bool FeedbackPrivateSendFeedbackFunction::RunImpl() {
->GetForProfile(GetProfile())
->GetService();
DCHECK(service);
+
+ if (feedback_info.send_histograms) {
+ scoped_ptr<std::string> histograms(new std::string);
+ service->GetHistograms(histograms.get());
+ if (!histograms->empty())
+ feedback_data->SetAndCompressHistograms(histograms.Pass());
+ }
+
service->SendFeedback(
GetProfile(),
feedback_data,
base::Bind(&FeedbackPrivateSendFeedbackFunction::OnCompleted, this));
+
return true;
}
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/extensions/api/feedback_private/feedback_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698