[HBD] Merge two identical ShouldUseJavaScriptSettingForPlugin impls

chrome/common/plugin_utils.cc

Index: chrome/common/plugin_utils.cc
diff --git a/chrome/common/plugin_utils.cc b/chrome/common/plugin_utils.cc
new file mode 100644
index 0000000000000000000000000000000000000000..5b812b230f6cb1db80009a1d7dcda3944581e53e
--- /dev/null
+++ b/chrome/common/plugin_utils.cc
@@ -0,0 +1,39 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/common/plugin_utils.h"
+
+#include "base/strings/utf_string_conversions.h"
+#include "content/public/common/webplugininfo.h"
+#include "widevine_cdm_version.h"  // In SHARED_INTERMEDIATE_DIR.
+
+#if !defined(DISABLE_NACL)
+#include "components/nacl/common/nacl_constants.h"
+#endif
+
+// For certain sandboxed Pepper plugins, use the JavaScript Content Settings.

[Lei Zhang, 2016/09/23 05:00:51]

Put this in the header?

[tommycli, 2016/09/23 16:46:45]

Done.

+bool ShouldUseJavaScriptSettingForPlugin(const content::WebPluginInfo& plugin) {
+  if (plugin.type != content::WebPluginInfo::PLUGIN_TYPE_PEPPER_IN_PROCESS &&
+      plugin.type !=
+          content::WebPluginInfo::PLUGIN_TYPE_PEPPER_OUT_OF_PROCESS) {
+    return false;
+  }
+
+#if !defined(DISABLE_NACL)
+  // Treat Native Client invocations like JavaScript.
+  if (plugin.name == base::ASCIIToUTF16(nacl::kNaClPluginName))
+    return true;
+#endif
+
+#if defined(WIDEVINE_CDM_AVAILABLE) && defined(ENABLE_PEPPER_CDMS)
+  // Treat CDM invocations like JavaScript.
+  if (plugin.name == base::ASCIIToUTF16(kWidevineCdmDisplayName)) {
+    DCHECK_EQ(content::WebPluginInfo::PLUGIN_TYPE_PEPPER_OUT_OF_PROCESS,
+              plugin.type);
+    return true;
+  }
+#endif  // defined(WIDEVINE_CDM_AVAILABLE) && defined(ENABLE_PEPPER_CDMS)
+
+  return false;
+}