ash: Add checks for supervised users.

components/user_manager/user.cc

 // Copyright 2014 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 "components/user_manager/user.h"
 
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_restrictions.h"
@@ skipping 10 matching lines @@
 std::string GetUserName(const std::string& email) {
   std::string::size_type i = email.find('@');
   if (i == 0 || i == std::string::npos) {
     return email;
   }
   return email.substr(0, i);
 }
 
 }  // namespace
 
+bool User::IsSupervised() const {
+  return false;
+}
+
+void User::SetIsSupervised(bool is_supervised) {
+  VLOG(1) << "Ignoring SetIsSupervised call with param " << is_supervised;
+}
+
 class RegularUser : public User {
  public:
   explicit RegularUser(const std::string& email);
   virtual ~RegularUser();
 
   // Overridden from User:
   virtual UserType GetType() const OVERRIDE;
   virtual bool CanSyncImage() const OVERRIDE;
+  virtual void SetIsSupervised(bool is_supervised) OVERRIDE {
+    VLOG(1) << "Setting user is supervised to " << is_supervised;
+    is_supervised_ = is_supervised;
+  }
+  virtual bool IsSupervised() const OVERRIDE {
+    return is_supervised_;
+  }
 
  private:
+  bool is_supervised_;
+
   DISALLOW_COPY_AND_ASSIGN(RegularUser);
 };
 
 class GuestUser : public User {
  public:
   GuestUser();
   virtual ~GuestUser();
 
   // Overridden from User:
   virtual UserType GetType() const OVERRIDE;
@@ skipping 14 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(KioskAppUser);
 };
 
 class SupervisedUser : public User {
  public:
   explicit SupervisedUser(const std::string& username);
   virtual ~SupervisedUser();
 
   // Overridden from User:
   virtual UserType GetType() const OVERRIDE;
+  virtual bool IsSupervised() const OVERRIDE;
   virtual std::string display_email() const OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SupervisedUser);
 };
 
 class RetailModeUser : public User {
  public:
   RetailModeUser();
   virtual ~RetailModeUser();
@@ skipping 132 matching lines @@
 
 void User::SetStubImage(const UserImage& stub_user_image,
                         int image_index,
                         bool is_loading) {
   user_image_ = stub_user_image;
   image_index_ = image_index;
   image_is_stub_ = true;
   image_is_loading_ = is_loading;
 }
 
 RegularUser::RegularUser(const std::string& email) : User(email) {

[Nikita (slow), 2014/09/17 18:25:26]

You need to initialize is_supervised_(false) in ctor.

[merkulova, 2014/09/18 08:51:29]

Done.

   set_can_lock(true);
   set_display_email(email);
 }
 
 RegularUser::~RegularUser() {
 }
 
 UserType RegularUser::GetType() const {
   return user_manager::USER_TYPE_REGULAR;
 }
@@ skipping 33 matching lines @@
 }
 
 UserType SupervisedUser::GetType() const {
   return user_manager::USER_TYPE_SUPERVISED;
 }
 
 std::string SupervisedUser::display_email() const {
   return base::UTF16ToUTF8(display_name());
 }
 
+bool SupervisedUser::IsSupervised() const {
+  return true;
+}
+
 RetailModeUser::RetailModeUser() : User(chromeos::login::kRetailModeUserName) {
   set_display_email(std::string());
 }
 
 RetailModeUser::~RetailModeUser() {
 }
 
 UserType RetailModeUser::GetType() const {
   return user_manager::USER_TYPE_RETAIL_MODE;
 }
@@ skipping 19 matching lines @@
     case user_manager::USER_TYPE_SUPERVISED:
     case user_manager::USER_TYPE_KIOSK_APP:
       return false;
     default:
       NOTREACHED();
   }
   return false;
 }
 
 }  // namespace user_manager