Title: [218206] trunk/Source/WebKit2
Revision
218206
Author
[email protected]
Date
2017-06-13 14:58:36 -0700 (Tue, 13 Jun 2017)

Log Message

[WK2][macOS] Allow AppleEvents in Citrix View Plugin
https://bugs.webkit.org/show_bug.cgi?id=173280
<rdar://problem/31913624>

Reviewed by Alexey Proskuryakov.

A slightly better fix: Allow AppleEvents for non-sandboxed plugins. This should
avoid inadvertently breaking custom plugins, etc.

* PluginProcess/mac/PluginProcessMac.mm:
(WebKit::PluginProcess::platformInitializeProcess): Switch check from a specific Citrix
test, to just allowing non-sandboxed plugins to use AppleEvents.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (218205 => 218206)


--- trunk/Source/WebKit2/ChangeLog	2017-06-13 21:54:57 UTC (rev 218205)
+++ trunk/Source/WebKit2/ChangeLog	2017-06-13 21:58:36 UTC (rev 218206)
@@ -1,3 +1,18 @@
+2017-06-13  Brent Fulgham  <[email protected]>
+
+        [WK2][macOS] Allow AppleEvents in Citrix View Plugin
+        https://bugs.webkit.org/show_bug.cgi?id=173280
+        <rdar://problem/31913624>
+
+        Reviewed by Alexey Proskuryakov.
+
+        A slightly better fix: Allow AppleEvents for non-sandboxed plugins. This should
+        avoid inadvertently breaking custom plugins, etc.
+
+        * PluginProcess/mac/PluginProcessMac.mm:
+        (WebKit::PluginProcess::platformInitializeProcess): Switch check from a specific Citrix
+        test, to just allowing non-sandboxed plugins to use AppleEvents.
+
 2017-06-13  Chris Dumez  <[email protected]>
 
         Include PID in some UIProcess-side release logging that relates to a particular WebProcess

Modified: trunk/Source/WebKit2/PluginProcess/mac/PluginProcessMac.mm (218205 => 218206)


--- trunk/Source/WebKit2/PluginProcess/mac/PluginProcessMac.mm	2017-06-13 21:54:57 UTC (rev 218205)
+++ trunk/Source/WebKit2/PluginProcess/mac/PluginProcessMac.mm	2017-06-13 21:58:36 UTC (rev 218206)
@@ -607,7 +607,7 @@
     if (m_pluginBundleIdentifier == "com.oracle.java.JavaAppletPlugin")
         (new UserActivity("com.oracle.java.JavaAppletPlugin"))->start();
     
-    if (m_pluginBundleIdentifier == "com.citrix.citrixicaclientplugIn") {
+    if (!pluginHasSandboxProfile(m_pluginBundleIdentifier)) {
         // Allow Apple Events from Citrix plugin. This can be removed when <rdar://problem/14012823> is fixed.
         setenv("__APPLEEVENTSSERVICENAME", "com.apple.coreservices.appleevents", 1);
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to