mirror of
				https://bitbucket.org/chromiumembedded/cef
				synced 2025-06-05 21:39:12 +02:00 
			
		
		
		
	
		
			
				
	
	
		
			229 lines
		
	
	
		
			9.5 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			229 lines
		
	
	
		
			9.5 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
diff --git chrome/browser/plugins/plugin_info_host_impl.cc chrome/browser/plugins/plugin_info_host_impl.cc
 | 
						|
index 484e07af5a98..0e62e20095c7 100644
 | 
						|
--- chrome/browser/plugins/plugin_info_host_impl.cc
 | 
						|
+++ chrome/browser/plugins/plugin_info_host_impl.cc
 | 
						|
@@ -16,6 +16,7 @@
 | 
						|
 #include "base/task_runner_util.h"
 | 
						|
 #include "base/threading/thread_task_runner_handle.h"
 | 
						|
 #include "build/build_config.h"
 | 
						|
+#include "cef/libcef/features/features.h"
 | 
						|
 #include "chrome/browser/browser_process.h"
 | 
						|
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 | 
						|
 #include "chrome/browser/plugins/chrome_plugin_service_filter.h"
 | 
						|
@@ -53,6 +54,11 @@
 | 
						|
 #include "url/gurl.h"
 | 
						|
 #include "url/origin.h"
 | 
						|
 
 | 
						|
+#if BUILDFLAG(ENABLE_CEF)
 | 
						|
+#include "cef/libcef/browser/plugins/plugin_service_filter.h"
 | 
						|
+#include "cef/libcef/common/extensions/extensions_util.h"
 | 
						|
+#endif
 | 
						|
+
 | 
						|
 #if BUILDFLAG(ENABLE_EXTENSIONS)
 | 
						|
 #include "components/guest_view/browser/guest_view_base.h"
 | 
						|
 #include "extensions/browser/extension_registry.h"
 | 
						|
@@ -74,12 +80,10 @@ namespace {
 | 
						|
 class PluginInfoHostImplShutdownNotifierFactory
 | 
						|
     : public BrowserContextKeyedServiceShutdownNotifierFactory {
 | 
						|
  public:
 | 
						|
-  static PluginInfoHostImplShutdownNotifierFactory* GetInstance() {
 | 
						|
-    return base::Singleton<PluginInfoHostImplShutdownNotifierFactory>::get();
 | 
						|
-  }
 | 
						|
+  static PluginInfoHostImplShutdownNotifierFactory* GetInstance();
 | 
						|
 
 | 
						|
  private:
 | 
						|
-  friend struct base::DefaultSingletonTraits<
 | 
						|
+  friend struct base::LazyInstanceTraitsBase<
 | 
						|
       PluginInfoHostImplShutdownNotifierFactory>;
 | 
						|
 
 | 
						|
   PluginInfoHostImplShutdownNotifierFactory()
 | 
						|
@@ -91,6 +95,16 @@ class PluginInfoHostImplShutdownNotifierFactory
 | 
						|
   DISALLOW_COPY_AND_ASSIGN(PluginInfoHostImplShutdownNotifierFactory);
 | 
						|
 };
 | 
						|
 
 | 
						|
+base::LazyInstance<PluginInfoHostImplShutdownNotifierFactory>::Leaky
 | 
						|
+    g_plugin_info_host_impl_shutdown_notifier_factory =
 | 
						|
+        LAZY_INSTANCE_INITIALIZER;
 | 
						|
+
 | 
						|
+// static
 | 
						|
+PluginInfoHostImplShutdownNotifierFactory*
 | 
						|
+PluginInfoHostImplShutdownNotifierFactory::GetInstance() {
 | 
						|
+  return g_plugin_info_host_impl_shutdown_notifier_factory.Pointer();
 | 
						|
+}
 | 
						|
+
 | 
						|
 #if BUILDFLAG(ENABLE_EXTENSIONS)
 | 
						|
 // Returns whether a request from a plugin to load |resource| from a renderer
 | 
						|
 // with process id |process_id| is a request for an internal resource by an app
 | 
						|
@@ -99,6 +113,9 @@ bool IsPluginLoadingAccessibleResourceInWebView(
 | 
						|
     extensions::ExtensionRegistry* extension_registry,
 | 
						|
     int process_id,
 | 
						|
     const GURL& resource) {
 | 
						|
+  if (!extension_registry)
 | 
						|
+    return false;
 | 
						|
+
 | 
						|
   extensions::WebViewRendererState* renderer_state =
 | 
						|
       extensions::WebViewRendererState::GetInstance();
 | 
						|
   std::string partition_id;
 | 
						|
@@ -128,12 +145,16 @@ bool IsPluginLoadingAccessibleResourceInWebView(
 | 
						|
 PluginInfoHostImpl::Context::Context(int render_process_id, Profile* profile)
 | 
						|
     : render_process_id_(render_process_id),
 | 
						|
       resource_context_(profile->GetResourceContext()),
 | 
						|
-#if BUILDFLAG(ENABLE_EXTENSIONS)
 | 
						|
-      extension_registry_(extensions::ExtensionRegistry::Get(profile)),
 | 
						|
-#endif
 | 
						|
       host_content_settings_map_(
 | 
						|
           HostContentSettingsMapFactory::GetForProfile(profile)),
 | 
						|
       plugin_prefs_(PluginPrefs::GetForProfile(profile)) {
 | 
						|
+#if BUILDFLAG(ENABLE_EXTENSIONS)
 | 
						|
+#if BUILDFLAG(ENABLE_CEF)
 | 
						|
+  if (extensions::ExtensionsEnabled())
 | 
						|
+#endif
 | 
						|
+    extension_registry_ = extensions::ExtensionRegistry::Get(profile);
 | 
						|
+#endif
 | 
						|
+
 | 
						|
   allow_outdated_plugins_.Init(prefs::kPluginsAllowOutdated,
 | 
						|
                                profile->GetPrefs());
 | 
						|
   allow_outdated_plugins_.MoveToThread(
 | 
						|
@@ -232,6 +253,7 @@ void PluginInfoHostImpl::PluginsLoaded(
 | 
						|
         plugin_metadata->identifier(), &output->status);
 | 
						|
   }
 | 
						|
 
 | 
						|
+#if !BUILDFLAG(ENABLE_CEF)
 | 
						|
   if (output->status == chrome::mojom::PluginStatus::kNotFound) {
 | 
						|
     // Check to see if the component updater can fetch an implementation.
 | 
						|
     base::PostTaskAndReplyWithResult(
 | 
						|
@@ -243,7 +265,9 @@ void PluginInfoHostImpl::PluginsLoaded(
 | 
						|
         base::BindOnce(&PluginInfoHostImpl::ComponentPluginLookupDone, this,
 | 
						|
                        params, std::move(output), std::move(callback),
 | 
						|
                        std::move(plugin_metadata)));
 | 
						|
-  } else {
 | 
						|
+  } else
 | 
						|
+#endif  // !BUILDFLAG(ENABLE_CEF)
 | 
						|
+  {
 | 
						|
     GetPluginInfoFinish(params, std::move(output), std::move(callback),
 | 
						|
                         std::move(plugin_metadata));
 | 
						|
   }
 | 
						|
@@ -256,6 +280,14 @@ void PluginInfoHostImpl::Context::DecidePluginStatus(
 | 
						|
     PluginMetadata::SecurityStatus security_status,
 | 
						|
     const std::string& plugin_identifier,
 | 
						|
     chrome::mojom::PluginStatus* status) const {
 | 
						|
+#if BUILDFLAG(ENABLE_CEF)
 | 
						|
+  // Don't override the user decision.
 | 
						|
+  if (*status == chrome::mojom::PluginStatus::kBlocked ||
 | 
						|
+      *status == chrome::mojom::PluginStatus::kDisabled) {
 | 
						|
+    return;
 | 
						|
+  }
 | 
						|
+#endif
 | 
						|
+
 | 
						|
   if (security_status == PluginMetadata::SECURITY_STATUS_FULLY_TRUSTED) {
 | 
						|
     *status = chrome::mojom::PluginStatus::kAllowed;
 | 
						|
     return;
 | 
						|
@@ -380,16 +412,36 @@ bool PluginInfoHostImpl::Context::FindEnabledPlugin(
 | 
						|
     return false;
 | 
						|
   }
 | 
						|
 
 | 
						|
+  const bool is_main_frame =
 | 
						|
+      main_frame_origin.IsSameOriginWith(url::Origin::Create(url));
 | 
						|
+
 | 
						|
+#if BUILDFLAG(ENABLE_CEF)
 | 
						|
+  CefPluginServiceFilter* filter = static_cast<CefPluginServiceFilter*>(
 | 
						|
+      PluginService::GetInstance()->GetFilter());
 | 
						|
+  DCHECK(filter);
 | 
						|
+
 | 
						|
+  size_t i = 0;
 | 
						|
+  for (; i < matching_plugins.size(); ++i) {
 | 
						|
+    if (filter->IsPluginAvailable(render_process_id_, render_frame_id,
 | 
						|
+                                  resource_context_, url, is_main_frame,
 | 
						|
+                                  main_frame_origin, &matching_plugins[i],
 | 
						|
+                                  status)) {
 | 
						|
+      break;
 | 
						|
+    }
 | 
						|
+  }
 | 
						|
+#else  // !BUILDFLAG(ENABLE_CEF)
 | 
						|
   content::PluginServiceFilter* filter =
 | 
						|
       PluginService::GetInstance()->GetFilter();
 | 
						|
   size_t i = 0;
 | 
						|
   for (; i < matching_plugins.size(); ++i) {
 | 
						|
     if (!filter || filter->IsPluginAvailable(
 | 
						|
                        render_process_id_, render_frame_id, resource_context_,
 | 
						|
-                       url, main_frame_origin, &matching_plugins[i])) {
 | 
						|
+                       url, is_main_frame, main_frame_origin,
 | 
						|
+                       &matching_plugins[i])) {
 | 
						|
       break;
 | 
						|
     }
 | 
						|
   }
 | 
						|
+#endif  // !BUILDFLAG(ENABLE_CEF)
 | 
						|
 
 | 
						|
   // If we broke out of the loop, we have found an enabled plugin.
 | 
						|
   bool enabled = i < matching_plugins.size();
 | 
						|
diff --git chrome/renderer/chrome_content_renderer_client.cc chrome/renderer/chrome_content_renderer_client.cc
 | 
						|
index 6a15716bb62f..60883d8e1031 100644
 | 
						|
--- chrome/renderer/chrome_content_renderer_client.cc
 | 
						|
+++ chrome/renderer/chrome_content_renderer_client.cc
 | 
						|
@@ -761,6 +761,7 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
 | 
						|
 
 | 
						|
     if ((status == chrome::mojom::PluginStatus::kUnauthorized ||
 | 
						|
          status == chrome::mojom::PluginStatus::kBlocked) &&
 | 
						|
+        observer &&
 | 
						|
         observer->IsPluginTemporarilyAllowed(identifier)) {
 | 
						|
       status = chrome::mojom::PluginStatus::kAllowed;
 | 
						|
     }
 | 
						|
@@ -948,7 +949,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
 | 
						|
         render_frame->GetRemoteAssociatedInterfaces()->GetInterface(
 | 
						|
             &plugin_auth_host);
 | 
						|
         plugin_auth_host->BlockedUnauthorizedPlugin(group_name, identifier);
 | 
						|
-        observer->DidBlockContentType(content_type, group_name);
 | 
						|
+        if (observer)
 | 
						|
+          observer->DidBlockContentType(content_type, group_name);
 | 
						|
         break;
 | 
						|
       }
 | 
						|
       case chrome::mojom::PluginStatus::kBlocked: {
 | 
						|
@@ -957,7 +959,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
 | 
						|
             l10n_util::GetStringFUTF16(IDS_PLUGIN_BLOCKED, group_name));
 | 
						|
         placeholder->AllowLoading();
 | 
						|
         RenderThread::Get()->RecordAction(UserMetricsAction("Plugin_Blocked"));
 | 
						|
-        observer->DidBlockContentType(content_type, group_name);
 | 
						|
+        if (observer)
 | 
						|
+          observer->DidBlockContentType(content_type, group_name);
 | 
						|
         break;
 | 
						|
       }
 | 
						|
       case chrome::mojom::PluginStatus::kBlockedByPolicy: {
 | 
						|
@@ -967,7 +970,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
 | 
						|
                                        group_name));
 | 
						|
         RenderThread::Get()->RecordAction(
 | 
						|
             UserMetricsAction("Plugin_BlockedByPolicy"));
 | 
						|
-        observer->DidBlockContentType(content_type, group_name);
 | 
						|
+        if (observer)
 | 
						|
+          observer->DidBlockContentType(content_type, group_name);
 | 
						|
         break;
 | 
						|
       }
 | 
						|
       case chrome::mojom::PluginStatus::kBlockedNoLoading: {
 | 
						|
@@ -975,7 +979,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
 | 
						|
             IDR_BLOCKED_PLUGIN_HTML,
 | 
						|
             l10n_util::GetStringFUTF16(IDS_PLUGIN_BLOCKED_NO_LOADING,
 | 
						|
                                        group_name));
 | 
						|
-        observer->DidBlockContentType(content_type, group_name);
 | 
						|
+        if (observer)
 | 
						|
+          observer->DidBlockContentType(content_type, group_name);
 | 
						|
         break;
 | 
						|
       }
 | 
						|
       case chrome::mojom::PluginStatus::kComponentUpdateRequired: {
 | 
						|
diff --git chrome/renderer/plugins/chrome_plugin_placeholder.cc chrome/renderer/plugins/chrome_plugin_placeholder.cc
 | 
						|
index 7052b1d68246..ba05a591ebf5 100644
 | 
						|
--- chrome/renderer/plugins/chrome_plugin_placeholder.cc
 | 
						|
+++ chrome/renderer/plugins/chrome_plugin_placeholder.cc
 | 
						|
@@ -350,8 +350,11 @@ void ChromePluginPlaceholder::OnBlockedContent(
 | 
						|
 
 | 
						|
   if (status ==
 | 
						|
       content::RenderFrame::PeripheralContentStatus::CONTENT_STATUS_TINY) {
 | 
						|
-    ContentSettingsObserver::Get(render_frame())
 | 
						|
-        ->DidBlockContentType(CONTENT_SETTINGS_TYPE_PLUGINS, title_);
 | 
						|
+    ContentSettingsObserver* observer =
 | 
						|
+        ContentSettingsObserver::Get(render_frame());
 | 
						|
+    if (observer) {
 | 
						|
+      observer->DidBlockContentType(CONTENT_SETTINGS_TYPE_PLUGINS, title_);
 | 
						|
+    }
 | 
						|
   }
 | 
						|
 
 | 
						|
   std::string message = base::StringPrintf(
 |