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

Unified Diff: components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc

Issue 2795053002: [subresource_filter] Implement the "Smart" UI on Android (Closed)
Patch Set: rebase on #468985 Created 3 years, 8 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: components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc
diff --git a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc
index e11a7e4d1a45764eb969df59768ff30f121ae72d..04d9ec48bd18600a569e9c57f00d19f25f54f2e1 100644
--- a/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc
+++ b/components/subresource_filter/content/browser/content_subresource_filter_driver_factory.cc
@@ -203,17 +203,19 @@ void ContentSubresourceFilterDriverFactory::WillProcessResponse(
DCHECK(activation_decision_ != ActivationDecision::UNKNOWN);
if (activation_decision_ != ActivationDecision::ACTIVATED) {
ResetActivationState();
- return;
+ } else {
+ activation_level_ = configurations->the_one_and_only().activation_level;
+ measure_performance_ =
+ activation_level_ != ActivationLevel::DISABLED &&
+ ShouldMeasurePerformanceForPageLoad(
+ configurations->the_one_and_only().performance_measurement_rate);
}
- activation_level_ = configurations->the_one_and_only().activation_level;
- measure_performance_ =
- activation_level_ != ActivationLevel::DISABLED &&
- ShouldMeasurePerformanceForPageLoad(
- configurations->the_one_and_only().performance_measurement_rate);
ActivationState state = ActivationState(activation_level_);
state.measure_performance = measure_performance_;
- throttle_manager_->NotifyPageActivationComputed(navigation_handle, state);
+
+ if (activation_level_ != ActivationLevel::DISABLED)
+ throttle_manager_->NotifyPageActivationComputed(navigation_handle, state);
}
void ContentSubresourceFilterDriverFactory::OnFirstSubresourceLoadDisallowed() {

Powered by Google App Engine
This is Rietveld 408576698