[Android WebView] Only send extra headers for the main page

android_webview/browser/aw_resource_context.h

+// Copyright (c) 2013 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_RESOURCE_CONTEXT_H_
+#define ANDROID_WEBVIEW_BROWSER_AW_RESOURCE_CONTEXT_H_
+
+#include <map>
+#include <string>
+
+#include "base/synchronization/lock.h"
+#include "content/public/browser/resource_context.h"
+
+namespace net {
+class URLRequestContextGetter;
+}
+
+namespace android_webview {
+
+class AwResourceContext : public content::ResourceContext {
+ public:
+  explicit AwResourceContext(net::URLRequestContextGetter* getter);
+  virtual ~AwResourceContext();
+
+  // content::ResourceContext implementation.
+  virtual net::HostResolver* GetHostResolver() OVERRIDE;
+  virtual net::URLRequestContext* GetRequestContext() OVERRIDE;
+  virtual bool AllowMicAccess(const GURL& origin) OVERRIDE;
+  virtual bool AllowCameraAccess(const GURL& origin) OVERRIDE;
+
+  void SetExtraHeaders(const GURL& url, std::string& headers);

[bulach, 2013/12/10 15:16:57]

nit: const for headers?

also, I think non-virtuals should be listed before the virtuals above.

[mnaganov (inactive), 2013/12/10 16:00:37]

Changed to const.

There is no explicit style guide on placing non-virtuals vs. virtuals. But as
AwBrowserContext puts non-virtuals first, made the same here for consistency.

+  std::string GetExtraHeaders(const GURL& url);
+
+ private:
+  net::URLRequestContextGetter* getter_;
+  base::Lock lock_;

[bulach, 2013/12/10 15:16:57]

nit: it'd be nice to say what this is protecting..

[mnaganov (inactive), 2013/12/10 16:00:37]

Renamed to extra_headers_lock_.

+  std::map<std::string, std::string> extra_headers_;
+
+  DISALLOW_COPY_AND_ASSIGN(AwResourceContext);
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_BROWSER_AW_RESOURCE_CONTEXT_H_