[Android WebView] Call shouldInterceptRequest on a background thread

android_webview/browser/aw_request_handler.h

+// Copyright 2015 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 ANDROID_WEBVIEW_BROWSER_AW_REQUEST_HANDLER_H_
+#define ANDROID_WEBVIEW_BROWSER_AW_REQUEST_HANDLER_H_
+
+#include "android_webview/browser/net/android_stream_reader_url_request_job.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace net {
+class URLRequest;
+class URLRequestJob;
+class NetworkDelegate;
+}
+
+namespace android_webview {
+
+class AwWebResourceResponse;
+
+class AwRequestHandler : public base::RefCounted<AwRequestHandler> {

[mmenke, 2015/09/29 15:56:05]

This class really needs some documentation, as do its methods.

[mmenke, 2015/09/29 15:56:05]

Refcounting this seems like a bad idea, that's just going to end in bugs and
tears.  We really don't want this outliving a cancelled request.

[mnaganov (inactive), 2015/09/29 23:45:35]

Not relevant anymore.

[mnaganov (inactive), 2015/09/29 23:45:35]

It seems that we actually can do without it, removed :)

+ public:
+  AwRequestHandler();
+
+  AndroidStreamReaderURLRequestJob* MaybeInterceptRequest(
+      net::URLRequest* request,
+      net::NetworkDelegate* network_delegate);
+
+ private:
+  friend class base::RefCounted<AwRequestHandler>;
+
+  ~AwRequestHandler();

[mmenke, 2015/09/29 15:56:05]

+override

[mnaganov (inactive), 2015/09/29 23:45:35]

Not relevant anymore.

+  bool QueryForAwWebResourceResponse(net::URLRequest* request);
+  void QueryForAwWebResourceResponseDone(
+      scoped_ptr<AwWebResourceResponse> response);
+
+  scoped_refptr<AndroidStreamReaderURLRequestJob> job_;
+
+  DISALLOW_COPY_AND_ASSIGN(AwRequestHandler);
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_BROWSER_AW_REQUEST_HANDLER_H_