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

Unified Diff: components/signin/core/browser/test_signin_client.cc

Issue 964553002: Remove support for special "sign in process" state. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ocl3
Patch Set: rebased Created 5 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 side-by-side diff with in-line comments
Download patch
Index: components/signin/core/browser/test_signin_client.cc
diff --git a/components/signin/core/browser/test_signin_client.cc b/components/signin/core/browser/test_signin_client.cc
index f2ca618228d321bc88067487fcc13d10f0abf536..536f137a28602d51ce8fbd1fbaf7f7543ebe9b56 100644
--- a/components/signin/core/browser/test_signin_client.cc
+++ b/components/signin/core/browser/test_signin_client.cc
@@ -14,12 +14,6 @@
#include "ios/public/test/fake_profile_oauth2_token_service_ios_provider.h"
#endif
-namespace {
-
-// Helper for testing.
-const int kInvalidProcessId = -1;
-}
-
TestSigninClient::TestSigninClient()
: request_context_(new net::TestURLRequestContextGetter(
base::MessageLoopProxy::current())),
@@ -110,26 +104,6 @@ TestSigninClient::GetIOSProviderAsFake() {
}
#endif
-void TestSigninClient::SetSigninProcess(int process_id) {
- if (process_id == signin_host_id_)
- return;
- DLOG_IF(WARNING, signin_host_id_ != kInvalidProcessId)
- << "Replacing in-use signin process.";
- signin_host_id_ = process_id;
-}
-
-void TestSigninClient::ClearSigninProcess() {
- signin_host_id_ = kInvalidProcessId;
-}
-
-bool TestSigninClient::IsSigninProcess(int process_id) const {
- return process_id == signin_host_id_;
-}
-
-bool TestSigninClient::HasSigninProcess() const {
- return signin_host_id_ != kInvalidProcessId;
-}
-
bool TestSigninClient::IsFirstRun() const {
return false;
}
« no previous file with comments | « components/signin/core/browser/test_signin_client.h ('k') | extensions/browser/api/web_request/web_request_permissions.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698