Introduce WebRequestEventRouterDelegate.

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

Index: chrome/browser/extensions/api/web_request/chrome_extension_web_request_event_router_delegate.h
diff --git a/chrome/browser/extensions/api/web_request/chrome_extension_web_request_event_router_delegate.h b/chrome/browser/extensions/api/web_request/chrome_extension_web_request_event_router_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..61006208846d056328fd71504d953ef058fb7d9e
--- /dev/null
+++ b/chrome/browser/extensions/api/web_request/chrome_extension_web_request_event_router_delegate.h
@@ -0,0 +1,41 @@
+// 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_API_WEB_REQUEST_CHROME_EXTENSION_WEB_REQUEST_EVENT_ROUTER_DELEGATE_H_
+#define CHROME_BROWSER_EXTENSIONS_API_WEB_REQUEST_CHROME_EXTENSION_WEB_REQUEST_EVENT_ROUTER_DELEGATE_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
+#include "extensions/browser/api/web_request/web_request_event_router_delegate.h"
+
+class ChromeExtensionWebRequestEventRouterDelegate :
+  public extensions::WebRequestEventRouterDelegate {
+ public:
+  ChromeExtensionWebRequestEventRouterDelegate();
+  virtual ~ChromeExtensionWebRequestEventRouterDelegate();
+
+  // WebRequestEventRouterDelegate implementation.
+  virtual void ExtractExtraRequestDetails(
+    net::URLRequest* request, base::DictionaryValue* out) OVERRIDE;
+  virtual bool OnGetMachingListernersImplCheck(
+    int tab_id, int window_id, net::URLRequest* request) OVERRIDE;
+  virtual void LogExtensionActivity(
+    void* profile_id,
+    bool is_incognito,
+    const std::string& extension_id,
+    const GURL& url,
+    const std::string& api_calli,
+    const extension_web_request_api_helpers::EventResponseDelta& delta)
+      OVERRIDE;
+ private:
+  void LogExtensionActivityInternal(
+    void* profile_id,

[Ken Rockot(use gerrit already), 2014/09/19 23:08:34]

Could you change the name to browser_context_id here as well?

Also I'm still wondering why we are using void* browser_context_id instead of
BrowserContext* browser_context. Is it because of some threading policy?

[Xi Han, 2014/09/22 14:34:15]

I talked with Battre@ about this, and he said void* was used to avoid
accidentally access profile on IO thread, but we could cast the pointer to a
BrowserContext* if we were certain to be on the UI thread. I look at the
implementation of this function again, and find out this function is not only
visited on UI thread. So I believe we'd better to use void* here.


On 2014/09/19 23:08:34, Ken Rockot wrote:

+    bool is_incognito,
+    const std::string& extension_id,
+    const GURL& url,
+    const std::string& api_calli,
+    scoped_ptr<base::DictionaryValue> details);
+};
+
+#endif  // CHROME_BROWSER_EXTENSIONS_API_WEB_REQUEST_CHROME_EXTENSION_WEB_REQUEST_EVENT_ROUTER_DELEGATE_H_