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

Unified Diff: chrome/browser/signin/token_service_unittest.h

Issue 71723002: This is the second CL of several that will eventually replace TokenService with (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 7 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/signin/signin_tracker.h ('k') | chrome/browser/signin/token_service_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/signin/token_service_unittest.h
diff --git a/chrome/browser/signin/token_service_unittest.h b/chrome/browser/signin/token_service_unittest.h
index 8e892923f956f8a3f705e0e466bf1b8cd63fc8aa..a12634f8196a9813ef5a181265cb4494db97f65e 100644
--- a/chrome/browser/signin/token_service_unittest.h
+++ b/chrome/browser/signin/token_service_unittest.h
@@ -20,43 +20,6 @@
class FakeSigninManagerBase;
-// TestNotificationTracker doesn't do a deep copy on the notification details.
-// We have to in order to read it out, or we have a bad ptr, since the details
-// are a reference on the stack.
-class TokenAvailableTracker : public content::TestNotificationTracker {
- public:
- TokenAvailableTracker();
- virtual ~TokenAvailableTracker();
-
- const TokenService::TokenAvailableDetails& details() {
- return details_;
- }
-
- private:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) OVERRIDE;
-
- TokenService::TokenAvailableDetails details_;
-};
-
-class TokenFailedTracker : public content::TestNotificationTracker {
- public:
- TokenFailedTracker();
- virtual ~TokenFailedTracker();
-
- const TokenService::TokenRequestFailedDetails& details() {
- return details_;
- }
-
- private:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) OVERRIDE;
-
- TokenService::TokenRequestFailedDetails details_;
-};
-
class TokenServiceTestHarness : public testing::Test {
protected:
TokenServiceTestHarness();
@@ -72,8 +35,6 @@ class TokenServiceTestHarness : public testing::Test {
const GaiaAuthConsumer::ClientLoginResult& credentials() const {
return credentials_;
}
- TokenAvailableTracker* success_tracker() { return &success_tracker_; }
- TokenFailedTracker* failure_tracker() { return &failure_tracker_; }
void CreateSigninManager(const std::string& username);
@@ -82,8 +43,6 @@ class TokenServiceTestHarness : public testing::Test {
FakeSigninManagerBase* signin_manager_;
TokenService* service_;
- TokenAvailableTracker success_tracker_;
- TokenFailedTracker failure_tracker_;
GaiaAuthConsumer::ClientLoginResult credentials_;
std::string oauth_token_;
std::string oauth_secret_;
« no previous file with comments | « chrome/browser/signin/signin_tracker.h ('k') | chrome/browser/signin/token_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698