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

Side by Side Diff: chrome/browser/android/data_usage/external_data_use_observer.h

Issue 2505343002: Restrict sending GWS ID only to signed in users (Closed)
Patch Set: Rebased 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef CHROME_BROWSER_ANDROID_DATA_USAGE_EXTERNAL_DATA_USE_OBSERVER_H_ 5 #ifndef CHROME_BROWSER_ANDROID_DATA_USAGE_EXTERNAL_DATA_USE_OBSERVER_H_
6 #define CHROME_BROWSER_ANDROID_DATA_USAGE_EXTERNAL_DATA_USE_OBSERVER_H_ 6 #define CHROME_BROWSER_ANDROID_DATA_USAGE_EXTERNAL_DATA_USE_OBSERVER_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <memory> 9 #include <memory>
10 10
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 // to move this, ExternalDataUseObserverBridge needs to hold a pointer to 71 // to move this, ExternalDataUseObserverBridge needs to hold a pointer to
72 // ExternalDataUseReporter. This is currently not doable, since this creates 72 // ExternalDataUseReporter. This is currently not doable, since this creates
73 // a circular dependency between ExternalDataUseReporter, DataUseTabModel and 73 // a circular dependency between ExternalDataUseReporter, DataUseTabModel and
74 // ExternalDataUseObserverBridge, which creates issues during desctruction. 74 // ExternalDataUseObserverBridge, which creates issues during desctruction.
75 void OnReportDataUseDone(bool success); 75 void OnReportDataUseDone(bool success);
76 76
77 ExternalDataUseReporter* GetExternalDataUseReporterForTesting() const { 77 ExternalDataUseReporter* GetExternalDataUseReporterForTesting() const {
78 return external_data_use_reporter_; 78 return external_data_use_reporter_;
79 } 79 }
80 80
81 void SetRegisterGoogleVariationID(bool register_google_variation_id);
82
81 private: 83 private:
82 friend class DataUseTabModelTest; 84 friend class DataUseTabModelTest;
83 friend class DataUseUITabModelTest; 85 friend class DataUseUITabModelTest;
84 friend class ExternalDataUseObserverTest; 86 friend class ExternalDataUseObserverTest;
85 FRIEND_TEST_ALL_PREFIXES(ExternalDataUseObserverTest, 87 FRIEND_TEST_ALL_PREFIXES(ExternalDataUseObserverTest,
86 MatchingRuleFetchOnControlAppInstall); 88 MatchingRuleFetchOnControlAppInstall);
87 FRIEND_TEST_ALL_PREFIXES(ExternalDataUseObserverTest, 89 FRIEND_TEST_ALL_PREFIXES(ExternalDataUseObserverTest,
88 PeriodicFetchMatchingRules); 90 PeriodicFetchMatchingRules);
89 FRIEND_TEST_ALL_PREFIXES(ExternalDataUseObserverTest, 91 FRIEND_TEST_ALL_PREFIXES(ExternalDataUseObserverTest,
90 RegisteredAsDataUseObserver); 92 RegisteredAsDataUseObserver);
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 base::WeakPtrFactory<ExternalDataUseObserver> weak_factory_; 140 base::WeakPtrFactory<ExternalDataUseObserver> weak_factory_;
139 141
140 DISALLOW_COPY_AND_ASSIGN(ExternalDataUseObserver); 142 DISALLOW_COPY_AND_ASSIGN(ExternalDataUseObserver);
141 }; 143 };
142 144
143 } // namespace android 145 } // namespace android
144 146
145 } // namespace chrome 147 } // namespace chrome
146 148
147 #endif // CHROME_BROWSER_ANDROID_DATA_USAGE_EXTERNAL_DATA_USE_OBSERVER_H_ 149 #endif // CHROME_BROWSER_ANDROID_DATA_USAGE_EXTERNAL_DATA_USE_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698