Refactor and fix declarative webRequest API permissions

chrome/browser/extensions/api/web_request/web_request_permissions.h

+// Copyright (c) 2012 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_API_WEB_REQUEST_WEB_REQUEST_PERMISSIONS_H_
+#define CHROME_BROWSER_EXTENSIONS_API_WEB_REQUEST_WEB_REQUEST_PERMISSIONS_H_
+
+#include <map>
+#include <string>
+
+class ExtensionInfoMap;
+class GURL;
+
+namespace net {
+class URLRequest;
+}
+
+// This class is used to test whether extensions may modify web requests.
+class WebRequestPermissions {
+ public:
+  WebRequestPermissions();
+  ~WebRequestPermissions();
+
+  // Returns true if the request shall not be reported to extensions.
+  static bool HideRequest(const net::URLRequest* request);
+
+  // Called when an incognito profile is created or destroyed.
+  void OnOTRProfileCreated(void* original_profile,
+                           void* otr_profile);
+  void OnOTRProfileDestroyed(void* original_profile,
+                             void* otr_profile);
+
+  void* GetCrossProfile(void* profile) const;
+
+  bool CanExtensionAccessURL(const ExtensionInfoMap* extension_info_map,
+                             const std::string& extension_id,
+                             const GURL& url,
+                             bool crosses_incognito,
+                             bool enforce_host_permissions) const;
+ private:
+  typedef std::map<void*, void*> CrossProfileMap;
+
+  // A map of original profile -> corresponding incognito profile (and vice
+  // versa).
+  CrossProfileMap cross_profile_map_;

[Matt Perry, 2012/07/25 21:27:56]

If you move this back to the event router, then this class can just be static
methods, and you won't need to pass an instance around.

[battre, 2012/07/26 16:38:43]

Done.

I have left this class rather than moving everything back into the helpers
namespace and into web_request_api_helpers.cc because I think that the
separation of permissions is quite good for readability.
web_request_api_helpers.cc is pretty large already.

[Matt Perry, 2012/07/26 18:42:41]

Agreed - I only meant to move cross_profile_map_. This class looks good.

+};
+
+#endif  // CHROME_BROWSER_EXTENSIONS_API_WEB_REQUEST_WEB_REQUEST_PERMISSIONS_H_