Remove ArcAuthService::GetAuthCodeDeprecated0() implementation.

chrome/browser/chromeos/arc/arc_auth_service.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/arc/arc_auth_service.h"
 
 #include <utility>
 
 #include "ash/common/shelf/shelf_delegate.h"
 #include "ash/common/wm_shell.h"
@@ skipping 335 matching lines @@
 std::string ArcAuthService::GetAndResetAuthCode() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   std::string auth_code;
   auth_code_.swap(auth_code);
   return auth_code;
 }
 
 void ArcAuthService::GetAuthCodeDeprecated0(
     const GetAuthCodeDeprecated0Callback& callback) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
-  DCHECK(!IsOptInVerificationDisabled());
-  // For robot account we must use RequestAccountInfo because it allows to
-  // specify account type.
-  DCHECK(!IsArcKioskMode());
-  callback.Run(GetAndResetAuthCode());
+  NOTREACHED() << "GetAuthCodeDeprecated0() should no longer be callable";
+  callback.Run(std::string());

[hidehiko, 2016/11/14 07:41:03]

Because this should never be called, we may be able to drop "just in case
callback.Run()" here?

[Luis Héctor Chávez, 2016/11/14 16:07:52]

yeah, we should drop it.

[hidehiko, 2016/11/14 17:54:51]

Done.

 }
 
 void ArcAuthService::GetAuthCodeDeprecated(
     const GetAuthCodeDeprecatedCallback& callback) {
   // For robot account we must use RequestAccountInfo because it allows
   // to specify account type.
   DCHECK(!IsArcKioskMode());
   RequestAccountInfoInternal(
       base::MakeUnique<ArcAuthService::AccountInfoNotifier>(callback));
 }
@@ skipping 775 matching lines @@
       return os << "FETCHING_CODE";
     case ArcAuthService::State::ACTIVE:
       return os << "ACTIVE";
     default:
       NOTREACHED();
       return os;
   }
 }
 
 }  // namespace arc