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

Side by Side Diff: chrome/browser/browsing_data/browsing_data_remover_unittest.cc

Issue 491753003: Domain Reliability: Don't upload when metrics reporting is off. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix a last couple of nits Created 6 years, 3 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 | Annotate | Revision Log
« no previous file with comments | « base/prefs/pref_member.cc ('k') | chrome/browser/profiles/profile_impl.h » ('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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/browsing_data/browsing_data_remover.h" 5 #include "chrome/browser/browsing_data/browsing_data_remover.h"
6 6
7 #include <set> 7 #include <set>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 583 matching lines...) Expand 10 before | Expand all | Expand 10 after
594 DISALLOW_COPY_AND_ASSIGN(RemoveLocalStorageTester); 594 DISALLOW_COPY_AND_ASSIGN(RemoveLocalStorageTester);
595 }; 595 };
596 596
597 class MockDomainReliabilityService : public DomainReliabilityService { 597 class MockDomainReliabilityService : public DomainReliabilityService {
598 public: 598 public:
599 MockDomainReliabilityService() : clear_count_(0) {} 599 MockDomainReliabilityService() : clear_count_(0) {}
600 600
601 virtual ~MockDomainReliabilityService() {} 601 virtual ~MockDomainReliabilityService() {}
602 602
603 virtual scoped_ptr<DomainReliabilityMonitor> CreateMonitor( 603 virtual scoped_ptr<DomainReliabilityMonitor> CreateMonitor(
604 scoped_refptr<base::SequencedTaskRunner> network_task_runner) OVERRIDE { 604 scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
605 PrefService* local_state_pref_service,
606 const char* reporting_pref_name) OVERRIDE {
605 NOTREACHED(); 607 NOTREACHED();
606 return scoped_ptr<DomainReliabilityMonitor>(); 608 return scoped_ptr<DomainReliabilityMonitor>();
607 } 609 }
608 610
609 virtual void ClearBrowsingData(DomainReliabilityClearMode clear_mode, 611 virtual void ClearBrowsingData(DomainReliabilityClearMode clear_mode,
610 const base::Closure& callback) OVERRIDE { 612 const base::Closure& callback) OVERRIDE {
611 clear_count_++; 613 clear_count_++;
612 last_clear_mode_ = clear_mode; 614 last_clear_mode_ = clear_mode;
613 callback.Run(); 615 callback.Run();
614 } 616 }
(...skipping 1215 matching lines...) Expand 10 before | Expand all | Expand 10 after
1830 EXPECT_EQ(1u, tester.clear_count()); 1832 EXPECT_EQ(1u, tester.clear_count());
1831 EXPECT_EQ(CLEAR_CONTEXTS, tester.last_clear_mode()); 1833 EXPECT_EQ(CLEAR_CONTEXTS, tester.last_clear_mode());
1832 } 1834 }
1833 1835
1834 TEST_F(BrowsingDataRemoverTest, DomainReliability_NoMonitor) { 1836 TEST_F(BrowsingDataRemoverTest, DomainReliability_NoMonitor) {
1835 BlockUntilBrowsingDataRemoved( 1837 BlockUntilBrowsingDataRemoved(
1836 BrowsingDataRemover::EVERYTHING, 1838 BrowsingDataRemover::EVERYTHING,
1837 BrowsingDataRemover::REMOVE_HISTORY | 1839 BrowsingDataRemover::REMOVE_HISTORY |
1838 BrowsingDataRemover::REMOVE_COOKIES, false); 1840 BrowsingDataRemover::REMOVE_COOKIES, false);
1839 } 1841 }
OLDNEW
« no previous file with comments | « base/prefs/pref_member.cc ('k') | chrome/browser/profiles/profile_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698