Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a3ae129f authored by Alex Vakulenko's avatar Alex Vakulenko Committed by Gerrit Code Review
Browse files

Merge "metricsd: Update weave command APIs"

parents 55793400 35f89638
Loading
Loading
Loading
Loading
+8 −6
Original line number Diff line number Diff line
@@ -326,7 +326,7 @@ void MetricsDaemon::OnShutdown(int* return_code) {
}

void MetricsDaemon::OnWeaveCommand(CommandProxy* command) {
  if (command->category() != "metrics" || command->status() != "queued") {
  if (command->state() != "queued") {
    return;
  }

@@ -342,24 +342,26 @@ void MetricsDaemon::OnEnableMetrics(CommandProxy* command) {
  if (base::WriteFile(metrics_directory_.Append(metrics::kConsentFileName),
                      "", 0) != 0) {
    PLOG(ERROR) << "Could not create the consent file.";
    command->Abort(nullptr);
    command->Abort("metrics_error", "Could not create the consent file",
                   nullptr);
    return;
  }

  NotifyStateChanged();
  command->Done(nullptr);
  command->Complete({}, nullptr);
}

void MetricsDaemon::OnDisableMetrics(CommandProxy* command) {
  if (!base::DeleteFile(metrics_directory_.Append(metrics::kConsentFileName),
                        false)) {
    PLOG(ERROR) << "Cound not delete the consent file.";
    command->Abort(nullptr);
    PLOG(ERROR) << "Could not delete the consent file.";
    command->Abort("metrics_error", "Could not delete the consent file",
                   nullptr);
    return;
  }

  NotifyStateChanged();
  command->Done(nullptr);
  command->Complete({}, nullptr);
}

void MetricsDaemon::NotifyStateChanged() {