AU: Reduce instances of signalling the crash reporter.
We recently started notifying the crash reporter whenever we attempted
to report an error code to the update server. We now throttle that
back a little bit: we only notify for official builds when we were
successful in reporting the error to the update server.
TEST=unittest; tested on device
BUG=10805
Review URL: http://codereview.chromium.org/6154003
Change-Id: Iaff1d5a0b600663796e63ad308f408c57d6444da
diff --git a/omaha_request_action.cc b/omaha_request_action.cc
index b8495c8..d5b3241 100644
--- a/omaha_request_action.cc
+++ b/omaha_request_action.cc
@@ -348,6 +348,11 @@
// Events are best effort transactions -- assume they always succeed.
if (IsEvent()) {
CHECK(!HasOutputPipe()) << "No output pipe allowed for event requests.";
+ if (event_->result == OmahaEvent::kResultError && successful &&
+ utils::IsOfficialBuild()) {
+ LOG(INFO) << "Signalling Crash Reporter.";
+ utils::ScheduleCrashReporterUpload();
+ }
completer.set_code(kActionCodeSuccess);
return;
}