Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(492)

Side by Side Diff: components/data_use_measurement/core/data_use_network_delegate_unittest.cc

Issue 2756373003: Record the data use of service worker initiated requests (Closed)
Patch Set: Fix unittest Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/data_use_measurement/core/data_use_network_delegate.h" 5 #include "components/data_use_measurement/core/data_use_network_delegate.h"
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "base/test/histogram_tester.h" 10 #include "base/test/histogram_tester.h"
(...skipping 21 matching lines...) Expand all
32 32
33 static void MarkAsUserRequest(net::URLRequest* request) { 33 static void MarkAsUserRequest(net::URLRequest* request) {
34 request->SetUserData(kUserDataKey, new TestURLRequestClassifier()); 34 request->SetUserData(kUserDataKey, new TestURLRequestClassifier());
35 } 35 }
36 36
37 DataUseUserData::DataUseContentType GetContentType( 37 DataUseUserData::DataUseContentType GetContentType(
38 const net::URLRequest& request, 38 const net::URLRequest& request,
39 const net::HttpResponseHeaders& response_headers) const override { 39 const net::HttpResponseHeaders& response_headers) const override {
40 return DataUseUserData::OTHER; 40 return DataUseUserData::OTHER;
41 } 41 }
42
43 bool IsServiceWorkerRequest(const net::URLRequest& request) const override {
44 return false;
45 }
42 }; 46 };
43 47
44 class TestDataUseAscriber : public DataUseAscriber { 48 class TestDataUseAscriber : public DataUseAscriber {
45 public: 49 public:
46 TestDataUseAscriber() {} 50 TestDataUseAscriber() {}
47 51
48 DataUseRecorder* GetOrCreateDataUseRecorder( 52 DataUseRecorder* GetOrCreateDataUseRecorder(
49 net::URLRequest* request) override { 53 net::URLRequest* request) override {
50 return nullptr; 54 return nullptr;
51 } 55 }
(...skipping 181 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 histogram_tester.ExpectTotalCount( 237 histogram_tester.ExpectTotalCount(
234 "DataUse.TrafficSize.System.Downstream.Foreground.NotCellular", 0); 238 "DataUse.TrafficSize.System.Downstream.Foreground.NotCellular", 0);
235 histogram_tester.ExpectTotalCount( 239 histogram_tester.ExpectTotalCount(
236 "DataUse.TrafficSize.System.Upstream.Foreground.NotCellular", 0); 240 "DataUse.TrafficSize.System.Upstream.Foreground.NotCellular", 0);
237 histogram_tester.ExpectTotalCount("DataUse.MessageSize.Suggestions", 0); 241 histogram_tester.ExpectTotalCount("DataUse.MessageSize.Suggestions", 0);
238 } 242 }
239 243
240 } // namespace 244 } // namespace
241 245
242 } // namespace data_use_measurement 246 } // namespace data_use_measurement
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698