Commit 4ca897d2 authored by Brian White's avatar Brian White Committed by Commit Bot

Remove obsolete BackgroundIndependentMetrics experiment.

Bug: 882892
Change-Id: Id84d55aab6a947f3d36a785d01a705a15e2fed4a
Reviewed-on: https://chromium-review.googlesource.com/c/1482554
Commit-Queue: Brian White <bcwhite@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Auto-Submit: Brian White <bcwhite@chromium.org>
Reviewed-by: default avatarAlexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#634696}
parent 04575e6e
...@@ -36,9 +36,6 @@ namespace metrics { ...@@ -36,9 +36,6 @@ namespace metrics {
namespace { namespace {
const base::Feature kBackgroundIndependentMetrics = {
"BackgoundIndependentMetrics", base::FEATURE_ENABLED_BY_DEFAULT};
// These structures provide values used to define how files are opened and // These structures provide values used to define how files are opened and
// accessed. It obviates the need for multiple code-paths within several of // accessed. It obviates the need for multiple code-paths within several of
// the methods. // the methods.
...@@ -779,23 +776,15 @@ void FileMetricsProvider::ProvideIndependentMetrics( ...@@ -779,23 +776,15 @@ void FileMetricsProvider::ProvideIndependentMetrics(
SourceInfo* source_ptr = source.get(); SourceInfo* source_ptr = source.get();
DCHECK(source->allocator); DCHECK(source->allocator);
if (base::FeatureList::IsEnabled(kBackgroundIndependentMetrics)) { // Do the actual work as a background task.
// Do the actual work as a background task. base::PostTaskAndReplyWithResult(
base::PostTaskAndReplyWithResult( task_runner_.get(), FROM_HERE,
task_runner_.get(), FROM_HERE, base::BindOnce(
base::BindOnce( &FileMetricsProvider::ProvideIndependentMetricsOnTaskRunner,
&FileMetricsProvider::ProvideIndependentMetricsOnTaskRunner, source_ptr, system_profile_proto, snapshot_manager),
source_ptr, system_profile_proto, snapshot_manager), base::BindOnce(&FileMetricsProvider::ProvideIndependentMetricsCleanup,
base::BindOnce(&FileMetricsProvider::ProvideIndependentMetricsCleanup, weak_factory_.GetWeakPtr(), std::move(done_callback),
weak_factory_.GetWeakPtr(), std::move(done_callback), std::move(source)));
std::move(source)));
} else {
// Do the actual work now, inline (for performance comparisons).
bool success = ProvideIndependentMetricsOnTaskRunner(
source_ptr, system_profile_proto, snapshot_manager);
ProvideIndependentMetricsCleanup(std::move(done_callback),
std::move(source), success);
}
} }
void FileMetricsProvider::ProvideIndependentMetricsCleanup( void FileMetricsProvider::ProvideIndependentMetricsCleanup(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment