Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1834)

Unified Diff: net/base/host_resolver_impl.cc

Issue 1057001: Correct readability items from previous checkin... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | net/base/net_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/host_resolver_impl.cc
===================================================================
--- net/base/host_resolver_impl.cc (revision 41747)
+++ net/base/host_resolver_impl.cc (working copy)
@@ -435,9 +435,9 @@
void Start() {
DCHECK(IsOnOriginThread());
- const bool IS_SLOW = true;
+ const bool kIsSlow = true;
WorkerPool::PostTask(
- FROM_HERE, NewRunnableMethod(this, &IPv6ProbeJob::DoProbe), IS_SLOW);
+ FROM_HERE, NewRunnableMethod(this, &IPv6ProbeJob::DoProbe), kIsSlow);
}
// Cancels the current job.
@@ -1095,11 +1095,12 @@
AddressFamily address_family) {
DCHECK(address_family == ADDRESS_FAMILY_UNSPECIFIED ||
address_family == ADDRESS_FAMILY_IPV4);
- if (default_address_family_ != address_family)
- LOG(INFO) << "IPv6Probe forced AddressFamily setting to "
- << ((address_family == ADDRESS_FAMILY_UNSPECIFIED)
- ? "ADDRESS_FAMILY_UNSPECIFIED"
- : "ADDRESS_FAMILY_IPV4");
+ if (default_address_family_ != address_family) {
+ LOG(INFO) << "IPv6Probe forced AddressFamily setting to "
+ << ((address_family == ADDRESS_FAMILY_UNSPECIFIED)
+ ? "ADDRESS_FAMILY_UNSPECIFIED"
+ : "ADDRESS_FAMILY_IPV4");
+ }
default_address_family_ = address_family;
// Drop reference since the job has called us back.
DiscardIPv6ProbeJob();
« no previous file with comments | « no previous file | net/base/net_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698