Fix bug: AppShell: CHECK failure in PeerConnection init.

extensions/shell/browser/shell_url_request_context_getter.h

 // Copyright 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 EXTENSIONS_BROWSER_EXTENSION_URL_REQUEST_CONTEXT_GETTER_H_
-#define EXTENSIONS_BROWSER_EXTENSION_URL_REQUEST_CONTEXT_GETTER_H_
+#ifndef EXTENSIONS_SHELL_BROWSER_SHELL_URL_REQUEST_CONTEXT_GETTER_H_
+#define EXTENSIONS_SHELL_BROWSER_SHELL_URL_REQUEST_CONTEXT_GETTER_H_
 
 #include "base/files/file_path.h"
-#include "base/memory/ref_counted.h"
-#include "base/memory/scoped_ptr.h"
-#include "content/public/browser/content_browser_client.h"
-#include "net/url_request/url_request_context_getter.h"
-#include "net/url_request/url_request_job_factory.h"
+#include "content/shell/browser/shell_url_request_context_getter.h"
 
 namespace base {
 class MessageLoop;
 }
 
 namespace net {
 class NetworkDelegate;
 class NetLog;
-class ProxyConfigService;
-class URLRequestContextStorage;
 }
 
 namespace extensions {
 
 class InfoMap;
 
-class ExtensionURLRequestContextGetter : public net::URLRequestContextGetter {
+class ShellURLRequestContextGetter :
+  public content::ShellURLRequestContextGetter {
  public:
-  explicit ExtensionURLRequestContextGetter(
+  explicit ShellURLRequestContextGetter(

[James Cook, 2014/10/07 16:24:54]

no explicit

[Xi Han, 2014/10/07 19:29:54]

Done.

       void* browser_context,

[James Cook, 2014/10/07 16:24:54]

Does this have to be void* or can it be content::BrowserContext?

[Xi Han, 2014/10/07 19:29:54]

Done.

       bool ignore_certificate_errors,
       const base::FilePath& base_path,
       base::MessageLoop* io_loop,
       base::MessageLoop* file_loop,
       content::ProtocolHandlerMap* protocol_handlers,
       content::URLRequestInterceptorScopedVector request_interceptors,
       net::NetLog* net_log,
       InfoMap* extension_info_map);
 
-  // net::URLRequestContextGetter implementation.
-  virtual net::URLRequestContext* GetURLRequestContext() OVERRIDE;
-  virtual scoped_refptr<base::SingleThreadTaskRunner>
-      GetNetworkTaskRunner() const OVERRIDE;
+  // content::ShellURLRequestContextGetter implementation.
+  virtual net::NetworkDelegate* CreateNetworkDelegate() OVERRIDE;

[James Cook, 2014/10/07 16:24:54]

I think we prefer lower-case override now.

[Xi Han, 2014/10/07 19:29:54]

Done.

 
 protected:
-  virtual ~ExtensionURLRequestContextGetter();
+  virtual ~ShellURLRequestContextGetter();
+
 private:
   void* browser_context_;

[James Cook, 2014/10/07 16:24:54]

BrowserContext* please

[Xi Han, 2014/10/07 19:29:54]

Done.

   bool ignore_certificate_errors_;
-  base::FilePath base_path_;
-  base::MessageLoop* io_loop_;
-  base::MessageLoop* file_loop_;
-  net::NetLog* net_log_;
   InfoMap* extension_info_map_;
 
-  scoped_ptr<net::ProxyConfigService> proxy_config_service_;
-  scoped_ptr<net::NetworkDelegate> network_delegate_;
-  scoped_ptr<net::URLRequestContextStorage> storage_;
-  scoped_ptr<net::URLRequestContext> url_request_context_;
-  content::ProtocolHandlerMap protocol_handlers_;
-  content::URLRequestInterceptorScopedVector request_interceptors_;
-
 private:
-  DISALLOW_COPY_AND_ASSIGN(ExtensionURLRequestContextGetter);
+  DISALLOW_COPY_AND_ASSIGN(ShellURLRequestContextGetter);
 };
 
 }  // namespace extensions
 
-#endif  // EXTENSIONS_BROWSER_EXTENSION_URL_REQUEST_CONTEXT_GETTER_H_
+#endif  // EXTENSIONS_SHELL_BROWSER_SHELL_URL_REQUEST_CONTEXT_GETTER_H_