Add account_type into AccountId

chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_GAIA_SCREEN_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_GAIA_SCREEN_HANDLER_H_
 
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 159 matching lines @@
 
   SigninScreenHandlerDelegate* Delegate();
 
   // Returns temporary unused device Id.
   std::string GetTemporaryDeviceId();
 
   FrameState frame_state() const { return frame_state_; }
   net::Error frame_error() const { return frame_error_; }
 
   // Returns user canonical e-mail. Finds already used account alias, if
   // user has already signed in.

[Andrew T Wilson (Slow), 2016/11/29 11:05:36]

Document what account_type is.

[Roman Sorokin (ftl), 2016/12/02 11:13:10]

Switched to enum class

   AccountId GetAccountId(const std::string& authenticated_email,
-                         const std::string& gaia_id) const;
+                         const std::string& gaia_id,
+                         const std::string& account_type) const;

[Andrew T Wilson (Slow), 2016/11/29 11:05:36]

Seems like most places are just passing an empty string as account_type. Should
we define a separate GetAccountIdWithType() or something like that, and leave
GetAccountId() alone? Would greatly shrink this patch.

[Alexander Alekseev, 2016/11/30 04:18:42]

Yes, it will. But I believe that people should always supply all the known
information. So I'd prefer the longer version.

On the other hand, call to "GetAccountId" is temporary for most of this patch
and will go away after most of chrome code will use it as user identifier.

 
   bool offline_login_is_active() const { return offline_login_is_active_; }
   void set_offline_login_is_active(bool offline_login_is_active) {
     offline_login_is_active_ = offline_login_is_active;
   }
 
   // Current state of Gaia frame.
   FrameState frame_state_ = FRAME_STATE_UNKNOWN;
 
   // Latest Gaia frame error.
@@ skipping 58 matching lines @@
   bool auth_extension_being_loaded_ = false;
 
   base::WeakPtrFactory<GaiaScreenHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GaiaScreenHandler);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_GAIA_SCREEN_HANDLER_H_