Disable some API calls in networkingPrivate for non-primary user

chromeos/login/login_state.cc

 // Copyright (c) 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.
 
 #include "chromeos/login/login_state.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/sys_info.h"
 #include "chromeos/chromeos_switches.h"
@@ skipping 39 matching lines @@
 }
 
 void LoginState::AddObserver(Observer* observer) {
   observer_list_.AddObserver(observer);
 }
 
 void LoginState::RemoveObserver(Observer* observer) {
   observer_list_.RemoveObserver(observer);
 }
 
+void LoginState::SetLoggedInStateAndPrimaryUser(
+    LoggedInState state,
+    LoggedInUserType type,
+    const std::string& primary_user_hash) {
+  DCHECK(type != LOGGED_IN_USER_NONE);
+  primary_user_hash_ = primary_user_hash;
+  VLOG(1) << "LoggedInStateUser: " << primary_user_hash;
+  SetLoggedInState(state, type);
+}
+
 void LoginState::SetLoggedInState(LoggedInState state,
                                   LoggedInUserType type) {
   if (state == logged_in_state_ && type == logged_in_user_type_)
     return;
   VLOG(1) << "LoggedInState: " << state << " UserType: " << type;
   logged_in_state_ = state;
   logged_in_user_type_ = type;
+  if (type == LOGGED_IN_USER_NONE)

[pneubeck (no reviews), 2014/06/13 08:51:12]

The concept of logging out does not exist in ChromeOS!
Please CHECK or DCHECK but don't handle it (also don't test this). This only
conveys the false information that ChromeOS does handle logouts.

[stevenjb, 2014/06/13 16:53:24]

I don't want to put a DCHECK in because I don't know what the behavior is during
shutdown, especially in tests. I'll remove the clearing, that shouldn't hurt
anything, but since this class merely reflects state controlled elsewhere
(UserManager) I don't think it should be checking that state.

+    primary_user_hash_.clear();
   NotifyObservers();
 }
 
 LoginState::LoggedInUserType LoginState::GetLoggedInUserType() const {
   return logged_in_user_type_;
 }
 
 bool LoginState::IsUserLoggedIn() const {
   if (always_logged_in_)
     return true;
@@ skipping 55 matching lines @@
 
 LoginState::~LoginState() {
 }
 
 void LoginState::NotifyObservers() {
   FOR_EACH_OBSERVER(LoginState::Observer, observer_list_,
                     LoggedInStateChanged());
 }
 
 }  // namespace chromeos