Pass user session type to extension feature checks

extensions/browser/renderer_startup_helper.cc

Index: extensions/browser/renderer_startup_helper.cc
diff --git a/extensions/browser/renderer_startup_helper.cc b/extensions/browser/renderer_startup_helper.cc
index 452c816963ddd1b509e6b12ea30c540b9b7ce566..c200c27a2dd843ec4e6fa83d24ef74f875f9c21a 100644
--- a/extensions/browser/renderer_startup_helper.cc
+++ b/extensions/browser/renderer_startup_helper.cc
@@ -17,6 +17,7 @@
 #include "extensions/common/extension_set.h"
 #include "extensions/common/extensions_client.h"
 #include "extensions/common/features/feature_channel.h"
+#include "extensions/common/features/feature_session_type.h"
 #include "ui/base/webui/web_ui_util.h"
 
 using content::BrowserContext;
@@ -66,9 +67,11 @@ void RendererStartupHelper::InitializeProcess(
         new ExtensionMsg_SetActivityLoggingEnabled(activity_logging_enabled));
   }
 
-  // Extensions need to know the channel for API restrictions. Send the channel
-  // to all renderers, as the non-extension renderers may have content scripts.
-  process->Send(new ExtensionMsg_SetChannel(GetCurrentChannel()));
+  // Extensions need to know the channel and the session type for API
+  // restrictions. The values are sent to all renderers, as the non-extension
+  // renderers may have content scripts.
+  process->Send(new ExtensionMsg_SetSessionInfo(
+      GetCurrentChannel(), GetCurrentFeatureSessionType()));

[Devlin, 2016/08/19 00:43:28]

I'm not familiar with where the login flow might happen, but is it possible that
it wouldn't be properly set by the time a renderer is created?

[tbarzic, 2016/08/19 17:23:32]

It's done during profile creation, so it should be done by the time renderers
are created.

 
   // Platform apps need to know the system font.
   // TODO(dbeam): this is not the system font in all cases.