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

Unified Diff: content/public/test/ppapi_test_utils.h

Issue 2356053002: [HBD] Only use Plugin Content Settings for Flash. (Closed)
Patch Set: fix Created 4 years, 3 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: content/public/test/ppapi_test_utils.h
diff --git a/content/public/test/ppapi_test_utils.h b/content/public/test/ppapi_test_utils.h
index 103e0e09cecf3529c63a59d695f67e46dc069289..7678570a874a0d1c8b6180e42c9f0a67ccbc6ae4 100644
--- a/content/public/test/ppapi_test_utils.h
+++ b/content/public/test/ppapi_test_utils.h
@@ -28,7 +28,8 @@ bool RegisterTestPluginWithExtraParameters(
WARN_UNUSED_RESULT;
// Registers the Power Saver test plugin to application/x-ppapi-tests.
-// Returns true on success, and false otherwise.
+// Returns true on success, and false otherwise. This plugin is registered
+// with the Flash name, so the Plugins Content Settings applies to it.
raymes 2016/09/26 08:18:30 I might just modify this function to register this
tommycli 2016/09/27 18:51:07 No problem with me! I stole this idea from your CL
bool RegisterPowerSaverTestPlugin(base::CommandLine* command_line)
WARN_UNUSED_RESULT;

Powered by Google App Engine
This is Rietveld 408576698