Supervised Users: Add tests for PermissionRequestCreatorApiary.

chrome/browser/supervised_user/supervised_user_service.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 "chrome/browser/supervised_user/supervised_user_service.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/prefs/pref_service.h"
@@ skipping 746 matching lines @@
 
   GetSettingsService()->SetActive(active_);
 
 #if defined(ENABLE_EXTENSIONS)
   SetExtensionsActive();
 #endif
 
   if (active_) {
     if (CommandLine::ForCurrentProcess()->HasSwitch(
             switches::kPermissionRequestApiUrl)) {
-      scoped_ptr<PermissionRequestCreator> creator =
-          PermissionRequestCreatorApiary::CreateWithProfile(
-              profile_,
-              GURL(CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
-                  switches::kPermissionRequestApiUrl)));
-      permissions_creators_.push_back(creator.release());
+      GURL url(CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+          switches::kPermissionRequestApiUrl));
+      if (url.is_valid()) {
+        scoped_ptr<PermissionRequestCreator> creator =
+            PermissionRequestCreatorApiary::CreateWithProfile(profile_, url);
+        permissions_creators_.push_back(creator.release());
+      } else {
+        LOG(WARNING) << "Got invalid URL for "
+                     << switches::kPermissionRequestApiUrl;
+      }
     } else {
       permissions_creators_.push_back(new PermissionRequestCreatorSync(
           GetSettingsService(),
           SupervisedUserSharedSettingsServiceFactory::GetForBrowserContext(
               profile_),
           ProfileSyncServiceFactory::GetForProfile(profile_),
           GetSupervisedUserName(),
           profile_->GetPrefs()->GetString(prefs::kSupervisedUserId)));
     }
 
@@ skipping 179 matching lines @@
   // The active user can be NULL in unit tests.
   if (user_manager::UserManager::Get()->GetActiveUser()) {
     return UTF16ToUTF8(user_manager::UserManager::Get()->GetUserDisplayName(
         user_manager::UserManager::Get()->GetActiveUser()->GetUserID()));
   }
   return std::string();
 #else
   return profile_->GetPrefs()->GetString(prefs::kProfileName);
 #endif
 }