Pass user session type to extension feature checks

extensions/common/features/base_feature_provider_unittest.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 "extensions/common/features/base_feature_provider.h"
 
 #include <algorithm>
 #include <set>
 #include <string>
 #include <utility>
@@ skipping 37 matching lines @@
       ExtensionBuilder()
           .SetManifest(DictionaryBuilder()
                            .Set("name", "test extension")
                            .Set("version", "1")
                            .Set("description", "hello there")
                            .Build())
           .Build();
   ASSERT_TRUE(extension.get());
 
   Feature* feature = provider->GetFeature("description");
-  EXPECT_EQ(Feature::IS_AVAILABLE,
-            feature->IsAvailableToContext(extension.get(),
-                                          Feature::UNSPECIFIED_CONTEXT,
-                                          GURL()).result());
+  EXPECT_EQ(
+      Feature::IS_AVAILABLE,
+      feature
+          ->IsAvailableToContext(extension.get(), Feature::UNSPECIFIED_CONTEXT,
+                                 Feature::SESSION_TYPE_UNSPECIFIED, GURL())
+          .result());
 
   // This is a generic extension, so an app-only feature isn't allowed.
   feature = provider->GetFeature("app.background");
   ASSERT_TRUE(feature);
-  EXPECT_EQ(Feature::INVALID_TYPE,
-            feature->IsAvailableToContext(extension.get(),
-                                          Feature::UNSPECIFIED_CONTEXT,
-                                          GURL()).result());
+  EXPECT_EQ(
+      Feature::INVALID_TYPE,
+      feature
+          ->IsAvailableToContext(extension.get(), Feature::UNSPECIFIED_CONTEXT,
+                                 Feature::SESSION_TYPE_UNSPECIFIED, GURL())
+          .result());
 
   // A feature not listed in the manifest isn't allowed.
   feature = provider->GetFeature("background");
   ASSERT_TRUE(feature);
-  EXPECT_EQ(Feature::NOT_PRESENT,
-            feature->IsAvailableToContext(extension.get(),
-                                          Feature::UNSPECIFIED_CONTEXT,
-                                          GURL()).result());
+  EXPECT_EQ(
+      Feature::NOT_PRESENT,
+      feature
+          ->IsAvailableToContext(extension.get(), Feature::UNSPECIFIED_CONTEXT,
+                                 Feature::SESSION_TYPE_UNSPECIFIED, GURL())
+          .result());
 }
 
 // Tests that a real permission feature is available for the correct types of
 // extensions and apps.
 TEST(BaseFeatureProviderTest, PermissionFeatureTypes) {
   // NOTE: This feature cannot have multiple rules, otherwise it is not a
   // SimpleFeature.
   const SimpleFeature* feature = static_cast<const SimpleFeature*>(
       BaseFeatureProvider::GetPermissionFeature("power"));
   ASSERT_TRUE(feature);
@@ skipping 29 matching lines @@
                   .Set("permissions", ListBuilder().Append("power").Build())
                   .Build())
           .Build();
   ASSERT_TRUE(app.get());
   ASSERT_TRUE(app->is_platform_app());
 
   // A permission requested in the manifest is available.
   Feature* feature = provider->GetFeature("power");
   EXPECT_EQ(
       Feature::IS_AVAILABLE,
-      feature->IsAvailableToContext(
-                   app.get(), Feature::UNSPECIFIED_CONTEXT, GURL()).result());
+      feature
+          ->IsAvailableToContext(app.get(), Feature::UNSPECIFIED_CONTEXT,
+                                 Feature::SESSION_TYPE_UNSPECIFIED, GURL())
+          .result());
 
   // A permission only available to whitelisted extensions returns availability
   // NOT_FOUND_IN_WHITELIST.
   feature = provider->GetFeature("bluetoothPrivate");
   ASSERT_TRUE(feature);
   EXPECT_EQ(
       Feature::NOT_FOUND_IN_WHITELIST,
-      feature->IsAvailableToContext(
-                   app.get(), Feature::UNSPECIFIED_CONTEXT, GURL()).result());
+      feature
+          ->IsAvailableToContext(app.get(), Feature::UNSPECIFIED_CONTEXT,
+                                 Feature::SESSION_TYPE_UNSPECIFIED, GURL())
+          .result());
 
   // A permission that isn't part of the manifest returns NOT_PRESENT.
   feature = provider->GetFeature("serial");
   ASSERT_TRUE(feature);
   EXPECT_EQ(
       Feature::NOT_PRESENT,
-      feature->IsAvailableToContext(
-                   app.get(), Feature::UNSPECIFIED_CONTEXT, GURL()).result());
+      feature
+          ->IsAvailableToContext(app.get(), Feature::UNSPECIFIED_CONTEXT,
+                                 Feature::SESSION_TYPE_UNSPECIFIED, GURL())
+          .result());
 }
 
 }  // namespace extensions