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

Side by Side Diff: chrome/common/plugin_utils.cc

Issue 2367553002: [HBD] Merge two identical ShouldUseJavaScriptSettingForPlugin impls (Closed)
Patch Set: update comment 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 unified diff | Download patch
« no previous file with comments | « chrome/common/plugin_utils.h ('k') | chrome/renderer/chrome_content_renderer_client.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/common/plugin_utils.h"
6
7 #include "base/strings/utf_string_conversions.h"
8 #include "content/public/common/webplugininfo.h"
9 #include "widevine_cdm_version.h" // In SHARED_INTERMEDIATE_DIR.
10
11 #if !defined(DISABLE_NACL)
12 #include "components/nacl/common/nacl_constants.h"
13 #endif
14
15 bool ShouldUseJavaScriptSettingForPlugin(const content::WebPluginInfo& plugin) {
16 if (plugin.type != content::WebPluginInfo::PLUGIN_TYPE_PEPPER_IN_PROCESS &&
17 plugin.type !=
18 content::WebPluginInfo::PLUGIN_TYPE_PEPPER_OUT_OF_PROCESS) {
19 return false;
20 }
21
22 #if !defined(DISABLE_NACL)
23 // Treat Native Client invocations like JavaScript.
24 if (plugin.name == base::ASCIIToUTF16(nacl::kNaClPluginName))
25 return true;
26 #endif
27
28 #if defined(WIDEVINE_CDM_AVAILABLE) && defined(ENABLE_PEPPER_CDMS)
29 // Treat CDM invocations like JavaScript.
30 if (plugin.name == base::ASCIIToUTF16(kWidevineCdmDisplayName)) {
31 DCHECK_EQ(content::WebPluginInfo::PLUGIN_TYPE_PEPPER_OUT_OF_PROCESS,
32 plugin.type);
33 return true;
34 }
35 #endif // defined(WIDEVINE_CDM_AVAILABLE) && defined(ENABLE_PEPPER_CDMS)
36
37 return false;
38 }
OLDNEW
« no previous file with comments | « chrome/common/plugin_utils.h ('k') | chrome/renderer/chrome_content_renderer_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698