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

Unified Diff: chrome/browser/subresource_filter/chrome_subresource_filter_client.cc

Issue 2820933002: [subresource_filter] add //chrome level unit test harness (Closed)
Patch Set: remove dep 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: chrome/browser/subresource_filter/chrome_subresource_filter_client.cc
diff --git a/chrome/browser/subresource_filter/chrome_subresource_filter_client.cc b/chrome/browser/subresource_filter/chrome_subresource_filter_client.cc
index 8b564dca2154ed16ba98578d5faa79da45c07af3..91ecb68e27f4c320add3ffeab2408035f0c85665 100644
--- a/chrome/browser/subresource_filter/chrome_subresource_filter_client.cc
+++ b/chrome/browser/subresource_filter/chrome_subresource_filter_client.cc
@@ -21,7 +21,7 @@
ChromeSubresourceFilterClient::ChromeSubresourceFilterClient(
content::WebContents* web_contents)
- : web_contents_(web_contents), shown_for_navigation_(false) {
+ : web_contents_(web_contents), did_show_ui_for_navigation_(false) {
DCHECK(web_contents);
// Ensure the content settings manager is initialized.
SubresourceFilterContentSettingsManagerFactory::EnsureForProfile(
@@ -32,10 +32,10 @@ ChromeSubresourceFilterClient::~ChromeSubresourceFilterClient() {}
void ChromeSubresourceFilterClient::ToggleNotificationVisibility(
bool visibility) {
- if (shown_for_navigation_ && visibility)
+ if (did_show_ui_for_navigation_ && visibility)
return;
- shown_for_navigation_ = visibility;
+ did_show_ui_for_navigation_ = visibility;
TabSpecificContentSettings* content_settings =
TabSpecificContentSettings::FromWebContents(web_contents_);

Powered by Google App Engine
This is Rietveld 408576698