Use DCHECKs when observing only a single notification.

chrome/browser/policy/cloud/user_policy_signin_service_base.cc

 // 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.
 
 #include "chrome/browser/policy/cloud/user_policy_signin_service_base.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 64 matching lines @@
 
 void UserPolicySigninServiceBase::GoogleSignedOut(const std::string& account_id,
                                                   const std::string& username) {
   ShutdownUserCloudPolicyManager();
 }
 
 void UserPolicySigninServiceBase::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
-  switch (type) {
-    case chrome::NOTIFICATION_PROFILE_ADDED:
-      // A new profile has been loaded - if it's signed in, then initialize the
-      // UCPM, otherwise shut down the UCPM (which deletes any cached policy
-      // data). This must be done here instead of at constructor time because
-      // the Profile is not fully initialized when this object is constructed
-      // (DoFinalInit() has not yet been called, so ProfileIOData and
-      // SSLConfigServiceManager have not been created yet).
-      // TODO(atwilson): Switch to using a timer instead, to avoid contention
-      // with other services at startup (http://crbug.com/165468).
-      InitializeOnProfileReady(content::Source<Profile>(source).ptr());
-      break;
-    default:
-      NOTREACHED();
-  }
+  DCHECK_EQ(chrome::NOTIFICATION_PROFILE_ADDED, type);
+
+  // A new profile has been loaded - if it's signed in, then initialize the
+  // UCPM, otherwise shut down the UCPM (which deletes any cached policy
+  // data). This must be done here instead of at constructor time because
+  // the Profile is not fully initialized when this object is constructed
+  // (DoFinalInit() has not yet been called, so ProfileIOData and
+  // SSLConfigServiceManager have not been created yet).
+  // TODO(atwilson): Switch to using a timer instead, to avoid contention
+  // with other services at startup (http://crbug.com/165468).
+  InitializeOnProfileReady(content::Source<Profile>(source).ptr());
 }
 
 void UserPolicySigninServiceBase::OnInitializationCompleted(
     CloudPolicyService* service) {
   // This is meant to be overridden by subclasses. Starting and stopping to
   // observe the CloudPolicyService from this base class avoids the need for
   // more virtuals.
 }
 
 void UserPolicySigninServiceBase::OnPolicyFetched(CloudPolicyClient* client) {}
@@ skipping 144 matching lines @@
     manager->DisconnectAndRemovePolicy();
 }
 
 scoped_refptr<net::URLRequestContextGetter>
 UserPolicySigninServiceBase::CreateSystemRequestContext() {
   return new SystemPolicyRequestContext(
       system_request_context(), GetUserAgent());
 }
 
 }  // namespace policy