cashew: don't propagate inactive plans to clients

src/data_plan.h

Index: src/data_plan.h
diff --git a/src/data_plan.h b/src/data_plan.h
index f58d6f69b0f16a8873de491c61d6d569805e4b86..005bc46d80b116eaeb7da699d4138e3e22e8db8b 100644
--- a/src/data_plan.h
+++ b/src/data_plan.h
@@ -78,6 +78,9 @@ class DataPlan {
     // must be >= 0
     virtual void SetLocalBytesUsed(Bytes local_bytes_used);
 
+    // get total used bytes
+    virtual Bytes GetTotalBytesUsed() const;

[rtc, 2010/11/23 04:38:58]

Google style says that accessors should not include "Get", just
total_bytes_used() instead of GetTotalBytesUsed()

[Vince Laviano, 2010/11/23 05:11:04]

Added TODO.

+
     // is this an active plan?
     // we define active to mean that the plan is current and not exhausted
     virtual bool IsActive() const;
@@ -129,6 +132,11 @@ class DataPlan {
     // baseline from the carrier API
     Bytes local_bytes_used_;
 
+    // our best estimate of total bytes used for this plan
+    // carrier baseline (|data_bytes_used_|) +
+    // locally measured traffic (|local_bytes_used_|)
+    Bytes total_bytes_used_;
+
     // convert plan type enum value to string formatted for libcros
     const char* TypeToLibcrosString(Type type) const;