Pass user session type to extension feature checks

extensions/renderer/script_context.h

Index: extensions/renderer/script_context.h
diff --git a/extensions/renderer/script_context.h b/extensions/renderer/script_context.h
index dd97ac90dc3871d8b7c8d095aec42e33eec09c66..ed786c1b1783f275284ce62352e8d151fe553cb6 100644
--- a/extensions/renderer/script_context.h
+++ b/extensions/renderer/script_context.h
@@ -50,6 +50,7 @@ class ScriptContext : public RequestSender::Source {
                 blink::WebLocalFrame* frame,
                 const Extension* extension,
                 Feature::Context context_type,
+                Feature::SessionType session_type,
                 const Extension* effective_extension,
                 Feature::Context effective_context_type);
   ~ScriptContext() override;
@@ -229,6 +230,7 @@ class ScriptContext : public RequestSender::Source {
 
   // The type of context.
   Feature::Context context_type_;
+  Feature::SessionType session_type_;

[Devlin, 2016/08/17 17:07:35]

This isn't really related to the context - it's global to the process.  I'd
rather we treat this more similarly to channel, rather than trying to curry it
around everywhere.

[tbarzic, 2016/08/17 19:49:01]

yeah, makes sense, I probably should have noticed that before :/

 
   // The effective extension associated with this context, or NULL if there is
   // none. This is different from the above extension if this context is in an