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

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: one more nit 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..daf16a33df40f657d2eb3d2a76b4a216c0aa1359 100644
--- a/chrome/renderer/content_settings_observer.cc
+++ b/chrome/renderer/content_settings_observer.cc
@@ -200,9 +200,16 @@ bool ContentSettingsObserver::IsPluginTemporarilyAllowed(
void ContentSettingsObserver::DidBlockContentType(
ContentSettingsType settings_type) {
+ DidBlockContentType(settings_type, base::string16());
+}
+
+void ContentSettingsObserver::DidBlockContentType(
+ ContentSettingsType settings_type,
+ const base::string16& details) {
if (!content_blocked_[settings_type]) {
Bernhard Bauer 2014/12/08 09:21:44 This check means you are only going to send this m
Will Harris 2014/12/08 21:11:46 send this IPC multiple times if details is not emp
content_blocked_[settings_type] = true;
- Send(new ChromeViewHostMsg_ContentBlocked(routing_id(), settings_type));
+ Send(new ChromeViewHostMsg_ContentBlocked(routing_id(), settings_type,
+ details));
}
}

Powered by Google App Engine
This is Rietveld 408576698