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

Unified Diff: chrome/browser/io_thread.cc

Issue 9150016: Move creation and ownership of ResourceDispatcherHost and PluginService to content. This gives a ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix chromeos ui_tests Created 8 years, 11 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
Index: chrome/browser/io_thread.cc
===================================================================
--- chrome/browser/io_thread.cc (revision 117096)
+++ chrome/browser/io_thread.cc (working copy)
@@ -11,7 +11,6 @@
#include "base/bind_helpers.h"
#include "base/debug/leak_tracker.h"
#include "base/logging.h"
-#include "base/metrics/field_trial.h"
#include "base/stl_util.h"
#include "base/string_number_conversions.h"
#include "base/string_split.h"
@@ -121,43 +120,6 @@
} else {
LOG(ERROR) << "Invalid switch for host resolver parallelism: " << s;
}
- } else {
- // Set up a field trial to see what impact the total number of concurrent
- // resolutions have on DNS resolutions.
- base::FieldTrial::Probability kDivisor = 1000;
- // For each option (i.e., non-default), we have a fixed probability.
- base::FieldTrial::Probability kProbabilityPerGroup = 100; // 10%.
-
- // After June 30, 2011 builds, it will always be in default group
- // (parallel_default).
- scoped_refptr<base::FieldTrial> trial(
- new base::FieldTrial(
- "DnsParallelism", kDivisor, "parallel_default", 2011, 6, 30));
-
- // List options with different counts.
- // Firefox limits total to 8 in parallel, and default is currently 50.
- int parallel_6 = trial->AppendGroup("parallel_6", kProbabilityPerGroup);
- int parallel_7 = trial->AppendGroup("parallel_7", kProbabilityPerGroup);
- int parallel_8 = trial->AppendGroup("parallel_8", kProbabilityPerGroup);
- int parallel_9 = trial->AppendGroup("parallel_9", kProbabilityPerGroup);
- int parallel_10 = trial->AppendGroup("parallel_10", kProbabilityPerGroup);
- int parallel_14 = trial->AppendGroup("parallel_14", kProbabilityPerGroup);
- int parallel_20 = trial->AppendGroup("parallel_20", kProbabilityPerGroup);
-
- if (trial->group() == parallel_6)
- parallelism = 6;
- else if (trial->group() == parallel_7)
- parallelism = 7;
- else if (trial->group() == parallel_8)
- parallelism = 8;
- else if (trial->group() == parallel_9)
- parallelism = 9;
- else if (trial->group() == parallel_10)
- parallelism = 10;
- else if (trial->group() == parallel_14)
- parallelism = 14;
- else if (trial->group() == parallel_20)
- parallelism = 20;
}
size_t retry_attempts = net::HostResolver::kDefaultRetryAttempts;

Powered by Google App Engine
This is Rietveld 408576698