Pass the system url request context getter to the cast service so that it can be used by various sy…

chromecast/service/cast_service.h

Index: chromecast/service/cast_service.h
diff --git a/chromecast/service/cast_service.h b/chromecast/service/cast_service.h
index 606f8f7c87b6d89850fe6efccf72adfa223afeea..9514b5262c1de3f70c8dbd76fc3483e2fadcb53e 100644
--- a/chromecast/service/cast_service.h
+++ b/chromecast/service/cast_service.h
@@ -16,11 +16,23 @@ namespace content{
 class BrowserContext;
 }
 
+namespace net {
+class URLRequestContextGetter;
+}
+
 namespace chromecast {
 
 class CastService {
  public:
-  static CastService* Create(content::BrowserContext* browser_context);
+  // Create() takes a separate url request context getter because the request
+  // context getter obtained through the browser context might not be
+  // appropriate for the url requests made by the cast service/reciever.
+  // For example, on Chromecast, it is needed to pass in a system url request
+  // context getter that would set the request context for NSS, which the main
+  // getter doesn't do.
+  static CastService* Create(
+      content::BrowserContext* browser_context,
+      net::URLRequestContextGetter* request_context_getter);

[gunsch, 2014/09/25 15:50:02]

Hm, is there any case where we actually want to change this parameter? Since
this is called by the content embedder, I would expect it to continue to always
pass in the same argument.

[lcwu1, 2014/09/25 17:07:37]

There are two reasons to pass in an url request context getter to CastService:
(1) CastService doesn't hold a reference to url_request_context_factory and the
request context getter in the browser_context is not (always) what we want, (2)
I want to keep CastService request_context_getter agnostic. That is, it just
uses the request context getter passed in in the constructor without having to
know what this context getter is (and how it is constructed). Currently we would
like to pass in the system context getter to the cast service, but it might
change if later we want to simplify the number of url request context getters
and pass in the main one.

 
   virtual ~CastService();