Reporting of policy errors via host-offline-reason: part 3

remoting/host/dns_blackhole_checker.h

Index: remoting/host/dns_blackhole_checker.h
diff --git a/remoting/host/dns_blackhole_checker.h b/remoting/host/dns_blackhole_checker.h
index 49d73ca78ae4a5d08e4e1c47fa72aa1711aba18d..6a5875c1b5af13fb6fcf000fef62fd6695bcd445 100644
--- a/remoting/host/dns_blackhole_checker.h
+++ b/remoting/host/dns_blackhole_checker.h
@@ -22,9 +22,9 @@ extern const char kDefaultHostTalkGadgetPrefix[];
 
 class DnsBlackholeChecker : public net::URLFetcherDelegate {
  public:
-  DnsBlackholeChecker(
-      scoped_refptr<net::URLRequestContextGetter> url_request_context_getter,
-      std::string talkgadget_prefix);
+  DnsBlackholeChecker(const scoped_refptr<net::URLRequestContextGetter>&
+                          url_request_context_getter,

[Wez, 2015/01/08 01:49:50]

What's the rationale for making this const scoped_refptr&?

[Łukasz Anforowicz, 2015/01/08 23:50:31]

When following your suggestion to pass network_task_runner +
url_request_context_getter directly (rather than wrapped in chromoting_context),
I've introduced parameters to HostSignalingManager::Create with const
scoped_refptr<X>& declaration.  This declaration avoids unnecessary
addref/release and ref-count churn (I am not sure now where exactly I read the
advise to pass scoped_refptr by const-ref rather than by value).

When fixing usage of these parameters in HostSignalingManager::Create, I've
noticed that we can also change the declarations in here (and a few other
places).  I've checked that in all of those cases the parameter (now changed to
const scoped_refptr<X>&) is just used in the constructor to initialize a field
of scoped_refptr<X> type, so this change seemed straightforward and safe.

[Wez, 2015/01/09 02:55:45]

Acknowledged.

+                      std::string talkgadget_prefix);

[Wez, 2015/01/08 01:49:50]

Where did this weird wrapping come from?

[Łukasz Anforowicz, 2015/01/08 23:50:31]

git cl format

The thing is that the 1st parameter won't fit on a single line even when
indented by 4 spaces (rather than continuing right after the opening paren).

I've changed "url_request_context_getter" to "request_context_getter" (this name
is already used by xmpp_signal_strategy.h) and now it fits in 80 columns.

[Wez, 2015/01/09 02:55:45]

Acknowledged.

   ~DnsBlackholeChecker() override;
 
   // net::URLFetcherDelegate interface.