PS - Adjusting webRequest API for use in Public Sessions

extensions/browser/api/web_request/web_request_permissions.h

Index: extensions/browser/api/web_request/web_request_permissions.h
diff --git a/extensions/browser/api/web_request/web_request_permissions.h b/extensions/browser/api/web_request/web_request_permissions.h
index 02c98ca4f14f92da76587d4df893ad0d4e35cc64..32597d78ca10ee748f00156a2e18153658812286 100644
--- a/extensions/browser/api/web_request/web_request_permissions.h
+++ b/extensions/browser/api/web_request/web_request_permissions.h
@@ -49,6 +49,10 @@ class WebRequestPermissions {
       HostPermissionsCheck host_permissions_check);
 
  private:
+  FRIEND_TEST_ALL_PREFIXES(ExtensionWebRequestApiTest, WebRequestPublicSession);
+
+  static bool allow_all_extensions_in_public_sessions_;

[Ivan Šandrk, 2016/11/14 20:16:13]

Added the runtime bool like this, but I'm unable to access it from the apitest
(... is a private member of WebRequestPermissions). Also tried using a private
setter function to no avail. Help?

[Devlin, 2016/11/15 01:19:37]

Hmm... I can't think of why that would be - it seems like this should work. 
I'll try patching it in locally tomorrow to take a look.

[Ivan Šandrk, 2016/11/15 13:49:58]

Got it working in a bit different way (suggestion by Drew). It seems that
functions ending with "ForTesting" generate presubmit warnings if called from
non-testing code :-)

+
   DISALLOW_IMPLICIT_CONSTRUCTORS(WebRequestPermissions);
 };