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

Unified Diff: chrome/renderer/content_settings_observer.cc

Issue 777103002: Add the names of plugins to the blocked plugin bubble. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use group_name instead of plugin.name Created 6 years 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/renderer/content_settings_observer.cc
diff --git a/chrome/renderer/content_settings_observer.cc b/chrome/renderer/content_settings_observer.cc
index 7c282fbd443324f6d202ca3eb19bded649931cdf..0f0eff8a6b54100522a5561cbe0947319b2b6b0c 100644
--- a/chrome/renderer/content_settings_observer.cc
+++ b/chrome/renderer/content_settings_observer.cc
@@ -200,9 +200,17 @@ bool ContentSettingsObserver::IsPluginTemporarilyAllowed(
void ContentSettingsObserver::DidBlockContentType(
ContentSettingsType settings_type) {
- if (!content_blocked_[settings_type]) {
+ DidBlockContentType(settings_type, base::string16());
+}
+
+void ContentSettingsObserver::DidBlockContentType(
+ ContentSettingsType settings_type,
+ const base::string16& details) {
+ // Send multiple ContentBlocked messages if details are provided.
+ if (!content_blocked_[settings_type] || !details.empty()) {
content_blocked_[settings_type] = true;
- Send(new ChromeViewHostMsg_ContentBlocked(routing_id(), settings_type));
+ Send(new ChromeViewHostMsg_ContentBlocked(routing_id(), settings_type,
+ details));
}
}
« no previous file with comments | « chrome/renderer/content_settings_observer.h ('k') | chrome/renderer/content_settings_observer_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698