Move is_under_advanced_protection to AccountInfo

The plan is to have client code keep instance of CoreAccountInfo
instead of keeping only one of the three components (account_id,
gaia or email) and then querying the IdentityManager for the
other.

As is_under_advanced_protection is not a valid key for an account,
it does not fit in CoreAccountInfo, so move it back to AccountInfo.
The few locations that need it during the initialization will have
to fetch the extended account information from IdentityManager.

Bug: 926204
Change-Id: I4a36e2464098ef18760507acc5d5cf3ce51c2fc6
Reviewed-on: https://chromium-review.googlesource.com/c/1448321
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#628268}

components/signin/core/browser/account_info.h

@@ -32,8 +32,6 @@ struct CoreAccountInfo {
   std::string account_id;
   std::string gaia;
   std::string email;
-
-  bool is_under_advanced_protection = false;
 };
 
 // Stores all the information known about an account. Part of the information
@@ -55,6 +53,7 @@ struct AccountInfo : public CoreAccountInfo {
   std::string picture_url;
   gfx::Image account_image;
   bool is_child_account = false;
+  bool is_under_advanced_protection = false;
 
   // Returns true if all fields in the account info are empty.
   bool IsEmpty() const;