Support for identifying traffic type in data use ascriber

components/data_use_measurement/core/data_use.cc

 // Copyright 2016 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 "components/data_use_measurement/core/data_use.h"
 
 namespace data_use_measurement {
 
-DataUse::DataUse() : total_bytes_sent_(0), total_bytes_received_(0) {}
+DataUse::DataUse()
+    : traffic_type_(UNKNOWN), total_bytes_sent_(0), total_bytes_received_(0) {}
 
 DataUse::DataUse(const DataUse& other) :
     total_bytes_sent_(other.total_bytes_sent_),
     total_bytes_received_(other.total_bytes_received_) {}

[tbansal1, 2017/05/09 00:59:08]

Add |traffic_type_| here.

[rajendrant, 2017/05/11 21:16:08]

Done.

 
 DataUse::~DataUse() {}
 
 void DataUse::MergeFrom(const DataUse& other) {
   total_bytes_sent_ += other.total_bytes_sent_;
   total_bytes_received_ += other.total_bytes_received_;

[tbansal1, 2017/05/09 00:59:08]

May be DCHECK that |traffic_type_| matches |other.traffic_type_|?

[rajendrant, 2017/05/11 21:16:08]

Need not be same while merging. When render frame is created before its main
frame request is sent, type will be UNKNOWN and later merged.

https://cs.chromium.org/chromium/src/chrome/browser/data_use_measurement/chro...

[tbansal1, 2017/05/12 15:59:58]

May be add that as a comment here?

[rajendrant, 2017/05/16 00:50:08]

Done.

 }
 
 }  // namespace data_use_measurement