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

Commit 24c91ea1 authored by Automerger Merge Worker's avatar Automerger Merge Worker
Browse files

Merge "Merge "StateTracker handles active state changes" into rvc-dev am:...

Merge "Merge "StateTracker handles active state changes" into rvc-dev am: 0d455533 am: e806ea28" into rvc-d1-dev-plus-aosp am: e481191e

Change-Id: Ice40cdf347db450113db129a00732671b19c422d
parents d87f91e1 e481191e
Loading
Loading
Loading
Loading
+3 −2
Original line number Diff line number Diff line
@@ -187,8 +187,9 @@ void ValueMetricProducer::onStateChanged(int64_t eventTimeNs, int32_t atomId,
    VLOG("ValueMetric %lld onStateChanged time %lld, State %d, key %s, %d -> %d",
         (long long)mMetricId, (long long)eventTimeNs, atomId, primaryKey.toString().c_str(),
         oldState.mValue.int_value, newState.mValue.int_value);
    // If condition is not true, we do not need to pull for this state change.
    if (mCondition != ConditionState::kTrue) {
    // If condition is not true or metric is not active, we do not need to pull
    // for this state change.
    if (mCondition != ConditionState::kTrue || !mIsActive) {
        return;
    }

+4 −1
Original line number Diff line number Diff line
@@ -329,7 +329,10 @@ void OringDurationTracker::onConditionChanged(bool condition, const int64_t time

void OringDurationTracker::onStateChanged(const int64_t timestamp, const int32_t atomId,
                                          const FieldValue& newState) {
    // If no keys are being tracked, update the current state key and return.
    // Nothing needs to be done on a state change if we have not seen a start
    // event, the metric is currently not active, or condition is false.
    // For these cases, no keys are being tracked in mStarted, so update
    // the current state key and return.
    if (mStarted.empty()) {
        updateCurrentStateKey(atomId, newState);
        return;