Include tab IDs when reporting data use accounting.

components/data_usage/core/data_use.h

Index: components/data_usage/core/data_use.h
diff --git a/components/data_usage/core/data_use.h b/components/data_usage/core/data_use.h
index ac005b3ccb22ee7b87e468572d652d11fc93c446..e04d04cdd48280b389a3278c86c70496c30c32b1 100644
--- a/components/data_usage/core/data_use.h
+++ b/components/data_usage/core/data_use.h
@@ -27,6 +27,8 @@ struct DataUse {
 
   ~DataUse();
 
+  bool operator==(const DataUse& other) const;

[bengr, 2015/11/05 00:00:38]

One of your tests had a method to check if two DataUse objects were equal, but
didn't use this operator.

[sclittle, 2015/11/05 01:22:48]

I think you mean the ExpectDataUse function in tab_id_annotator_unittest.cc,
which does actually use the "==" operator. It calls EXPECT_EQ(*expected,
*actual).

+
   // Returns true if |this| and |other| are identical except for byte counts.
   bool CanCombineWith(const DataUse& other) const;