Fix bug: AppShell: CHECK failure in PeerConnection init.

content/shell/browser/shell_browser_context.h

Index: content/shell/browser/shell_browser_context.h
diff --git a/content/shell/browser/shell_browser_context.h b/content/shell/browser/shell_browser_context.h
index f015344063cae5c5b2c9d062c6078db6f905b4e6..b366e6b00097524c569e081cb94232f24e5866c5 100644
--- a/content/shell/browser/shell_browser_context.h
+++ b/content/shell/browser/shell_browser_context.h
@@ -34,6 +34,9 @@ class ShellBrowserContext : public BrowserContext {
     guest_manager_ = guest_manager;
   }
 
+  void set_url_request_context_getter(ShellURLRequestContextGetter* getter);

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

I would make these methods protected to help emphasize that only subclasses
should call them.

Also, since they are not inlined:
SetURLRequestContextGetter();
GetURLRequestContextGetter();

And the first one in particular needs a comment about why it exists.

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

Good idea. Make these methods protected, but leave them as inline functions
here. Update the set_url_request_context_getter and make it just work as a
setter.


On 2014/10/07 16:24:54, James Cook wrote:

+  ShellURLRequestContextGetter* get_url_request_context_getter();
+
   // BrowserContext implementation.
   virtual base::FilePath GetPath() const override;
   virtual bool IsOffTheRecord() const override;