Added incident report for variations seed signature mismatch.

chrome/browser/metrics/variations/variations_service.cc

 // 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/metrics/variations/variations_service.h"
 
 #include <set>
 
 #include "base/build_time.h"
 #include "base/command_line.h"
@@ skipping 589 matching lines @@
 }
 
 void VariationsService::RecordLastFetchTime() {
   // local_state_ is NULL in tests, so check it first.
   if (local_state_) {
     local_state_->SetInt64(prefs::kVariationsLastFetchTime,
                            base::Time::Now().ToInternalValue());
   }
 }
 
+void VariationsService::OnDelayedSendIncidentReport(

[grt (UTC plus 2), 2014/10/10 17:45:42]

rather than embed a dependency on safe_browsing here in the variations service,
i think it makes more sense to add an accessor to VariationsService so that the
analyzer in safe_browsing can ask it if the signature was invalid and, if so,
ask it for the value.

[Georges Khalil, 2014/10/10 20:19:31]

Agreed, modified the code to reflect this change.

+    const safe_browsing::AddIncidentCallback& callback) {
+  if (!seed_store_.GetBadSignature().empty()) {
+    scoped_ptr<safe_browsing::ClientIncidentReport_IncidentData> incident_data(
+        new safe_browsing::ClientIncidentReport_IncidentData());
+    safe_browsing::
+        ClientIncidentReport_IncidentData_FinchConfigSignatureIncident*
+            finch_config_signature =
+                incident_data->mutable_finch_config_signature();
+    finch_config_signature->set_signature_hash(seed_store_.GetBadSignature());
+    callback.Run(incident_data.Pass());
+  }
+}
+
 }  // namespace chrome_variations