[HBD] Only use Plugin Content Settings for Flash.

content/public/test/ppapi_test_utils.h

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.
 bool RegisterPowerSaverTestPlugin(base::CommandLine* command_line)
     WARN_UNUSED_RESULT;

[xhwang, 2016/09/26 23:44:51]

When we call this in various tests, readers will have no clue that the "power
saver" plugin is registered to "application/x-ppapi-tests" mime type, with a
"Flash" name. I feel this is too tricky and hard to read, until people search
for RegisterPowerSaverTestPlugin() and find the comments here.

Can we somehow rename something (e.g. the function name here) to make things
easier to understand?

[tommycli, 2016/09/27 18:51:07]

Done.