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

Side by Side Diff: components/precache/core/precache_database_unittest.cc

Issue 2621413004: Add Precache.CacheStatus.Prefetch UMA. (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « components/precache/core/precache_database.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/precache/core/precache_database.h" 5 #include "components/precache/core/precache_database.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <map> 9 #include <map>
10 #include <memory> 10 #include <memory>
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
253 253
254 namespace { 254 namespace {
255 255
256 TEST_F(PrecacheDatabaseTest, PrecacheOverNetwork) { 256 TEST_F(PrecacheDatabaseTest, PrecacheOverNetwork) {
257 RecordPrecacheFromNetwork(kURL, kLatency, kFetchTime, kSize); 257 RecordPrecacheFromNetwork(kURL, kLatency, kFetchTime, kSize);
258 258
259 EXPECT_EQ(BuildURLTableMap(kURL, kFetchTime), GetActualURLTableMap()); 259 EXPECT_EQ(BuildURLTableMap(kURL, kFetchTime), GetActualURLTableMap());
260 260
261 ExpectNewSample("Precache.DownloadedPrecacheMotivated", kSize); 261 ExpectNewSample("Precache.DownloadedPrecacheMotivated", kSize);
262 ExpectNewSample("Precache.Latency.Prefetch", kLatency.InMilliseconds()); 262 ExpectNewSample("Precache.Latency.Prefetch", kLatency.InMilliseconds());
263 ExpectNewSample("Precache.CacheStatus.Prefetch", kFromNetwork);
263 ExpectNewSample("Precache.Freshness.Prefetch", kFreshnessBucket10K); 264 ExpectNewSample("Precache.Freshness.Prefetch", kFreshnessBucket10K);
264 ExpectNoOtherSamples(); 265 ExpectNoOtherSamples();
265 } 266 }
266 267
267 TEST_F(PrecacheDatabaseTest, PrecacheFromCacheWithURLTableEntry) { 268 TEST_F(PrecacheDatabaseTest, PrecacheFromCacheWithURLTableEntry) {
268 precache_url_table()->AddURL(kURL, kReferrerID, true, kOldFetchTime, false); 269 precache_url_table()->AddURL(kURL, kReferrerID, true, kOldFetchTime, false);
269 RecordPrecacheFromCache(kURL, kFetchTime, kSize); 270 RecordPrecacheFromCache(kURL, kFetchTime, kSize);
270 271
271 // The URL table entry should have been updated to have |kFetchTime| as the 272 // The URL table entry should have been updated to have |kFetchTime| as the
272 // timestamp. 273 // timestamp.
273 EXPECT_EQ(BuildURLTableMap(kURL, kFetchTime), GetActualURLTableMap()); 274 EXPECT_EQ(BuildURLTableMap(kURL, kFetchTime), GetActualURLTableMap());
274 275
275 ExpectNewSample("Precache.Latency.Prefetch", 0); 276 ExpectNewSample("Precache.Latency.Prefetch", 0);
277 ExpectNewSample("Precache.CacheStatus.Prefetch",
278 net::HttpResponseInfo::ENTRY_USED);
276 ExpectNewSample("Precache.Freshness.Prefetch", kFreshnessBucket10K); 279 ExpectNewSample("Precache.Freshness.Prefetch", kFreshnessBucket10K);
277 ExpectNoOtherSamples(); 280 ExpectNoOtherSamples();
278 } 281 }
279 282
280 TEST_F(PrecacheDatabaseTest, PrecacheFromCacheWithoutURLTableEntry) { 283 TEST_F(PrecacheDatabaseTest, PrecacheFromCacheWithoutURLTableEntry) {
281 RecordPrecacheFromCache(kURL, kFetchTime, kSize); 284 RecordPrecacheFromCache(kURL, kFetchTime, kSize);
282 285
283 EXPECT_TRUE(GetActualURLTableMap().empty()); 286 EXPECT_TRUE(GetActualURLTableMap().empty());
284 287
285 ExpectNewSample("Precache.Latency.Prefetch", 0); 288 ExpectNewSample("Precache.Latency.Prefetch", 0);
289 ExpectNewSample("Precache.CacheStatus.Prefetch",
290 net::HttpResponseInfo::ENTRY_USED);
286 ExpectNewSample("Precache.Freshness.Prefetch", kFreshnessBucket10K); 291 ExpectNewSample("Precache.Freshness.Prefetch", kFreshnessBucket10K);
287 ExpectNoOtherSamples(); 292 ExpectNoOtherSamples();
288 } 293 }
289 294
290 TEST_F(PrecacheDatabaseTest, FetchOverNetwork_NonCellular) { 295 TEST_F(PrecacheDatabaseTest, FetchOverNetwork_NonCellular) {
291 RecordFetchFromNetwork(kURL, kLatency, kFetchTime, kSize); 296 RecordFetchFromNetwork(kURL, kLatency, kFetchTime, kSize);
292 297
293 EXPECT_TRUE(GetActualURLTableMap().empty()); 298 EXPECT_TRUE(GetActualURLTableMap().empty());
294 299
295 ExpectNewSample("Precache.DownloadedNonPrecache", kSize); 300 ExpectNewSample("Precache.DownloadedNonPrecache", kSize);
(...skipping 390 matching lines...) Expand 10 before | Expand all | Expand 10 after
686 EXPECT_NE(PrecacheReferrerHostEntry::kInvalidId, referrer_id); 691 EXPECT_NE(PrecacheReferrerHostEntry::kInvalidId, referrer_id);
687 precache_database_->GetURLListForReferrerHost(referrer_id, &_, 692 precache_database_->GetURLListForReferrerHost(referrer_id, &_,
688 &actual_downloaded_urls); 693 &actual_downloaded_urls);
689 EXPECT_THAT(actual_downloaded_urls, 694 EXPECT_THAT(actual_downloaded_urls,
690 ElementsAre(GURL(already_reported_but_refetch))); 695 ElementsAre(GURL(already_reported_but_refetch)));
691 } 696 }
692 697
693 } // namespace 698 } // namespace
694 699
695 } // namespace precache 700 } // namespace precache
OLDNEW
« no previous file with comments | « components/precache/core/precache_database.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698