Attachment 349887 Details for Bug 189636 – Patch

[patch] Patch

bug-189636-20180917090152.patch (text/plain), 24.15 KB, created by Alex Christensen on 2018-09-17 09:01:53 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Christensen

Created: 2018-09-17 09:01:53 PDT

Size: 24.15 KB

patch

obsolete

>Index: Source/WebKit/ChangeLog
>===================================================================
>--- Source/WebKit/ChangeLog	(revision 236064)
>+++ Source/WebKit/ChangeLog	(working copy)
>@@ -1,3 +1,37 @@
>+2018-09-17  Alex Christensen  <achristensen@webkit.org>
>+
>+        Expose XPCServiceMain in a WebProcess header rather than WKProcessPool
>+        https://bugs.webkit.org/show_bug.cgi?id=189636
>+
>+        Reviewed by Dan Bernstein.
>+
>+        Fix a few build failures along the way.
>+
>+        * PluginProcess/mac/PluginControllerProxyMac.mm:
>+        (WebKit::PluginControllerProxy::windowAndViewFramesChanged):
>+        (WebKit::PluginControllerProxy::updateLayerHostingContext):
>+        * PluginProcess/mac/PluginProcessMac.mm:
>+        (WebKit::replacedNSWorkspace_launchApplicationAtURL_options_configuration_error):
>+        (WebKit::PluginProcess::platformInitializeProcess):
>+        * Shared/EntryPointUtilities/mac/XPCService/XPCServiceEntryPoint.h:
>+        * Shared/EntryPointUtilities/mac/XPCService/XPCServiceMain.mm:
>+        (WebKit::XPCServiceMain):
>+        (main):
>+        * SourcesCocoa.txt:
>+        * UIProcess/API/Cocoa/WKProcessPool.mm:
>+        (+[WKProcessPool _webContentProcessXPCMain]): Deleted.
>+        * UIProcess/API/Cocoa/WKProcessPoolPrivate.h:
>+        * WebKit.xcodeproj/project.pbxproj:
>+        * WebProcess/API: Added.
>+        * WebProcess/API/Cocoa: Added.
>+        * WebProcess/API/Cocoa/WKWebProcess.cpp: Added.
>+        (WKWebProcessMain):
>+        * WebProcess/API/Cocoa/WKWebProcess.h: Added.
>+        * WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm:
>+        (-[WKAccessibilityWebPageObject convertScreenPointToRootView:]):
>+        (-[WKAccessibilityWebPageObject accessibilityAttributeValue:]):
>+        (-[WKAccessibilityWebPageObject accessibilityHitTest:]):
>+
> 2018-09-17  Michael Catanzaro  <mcatanzaro@igalia.com>
> 
>         Unreviewed, fix incorrect WPE build fix made in r236009
>Index: Source/WebKit/SourcesCocoa.txt
>===================================================================
>--- Source/WebKit/SourcesCocoa.txt	(revision 236017)
>+++ Source/WebKit/SourcesCocoa.txt	(working copy)
>@@ -463,6 +463,8 @@ UIProcess/WebStorage/StorageManager.cpp
> 
> UIProcess/WebStorage/ios/LocalStorageDatabaseTrackerIOS.mm
> 
>+WebProcess/API/Cocoa/WKWebProcess.cpp
>+
> WebProcess/ApplePay/WebPaymentCoordinator.cpp
> 
> WebProcess/cocoa/PlaybackSessionManager.mm
>Index: Source/WebKit/PluginProcess/mac/PluginControllerProxyMac.mm
>===================================================================
>--- Source/WebKit/PluginProcess/mac/PluginControllerProxyMac.mm	(revision 236017)
>+++ Source/WebKit/PluginProcess/mac/PluginControllerProxyMac.mm	(working copy)
>@@ -37,8 +37,6 @@
> #import <QuartzCore/QuartzCore.h>
> #import <WebCore/GraphicsContextCG.h>
> 
>-using namespace WebCore;
>-
> namespace WebKit {
> 
> void PluginControllerProxy::pluginFocusOrWindowFocusChanged(bool pluginHasFocusAndWindowHasFocus)
>@@ -90,7 +88,7 @@ void PluginControllerProxy::platformGeom
>     [CATransaction commit];
> }
> 
>-void PluginControllerProxy::windowAndViewFramesChanged(const IntRect& windowFrameInScreenCoordinates, const IntRect& viewFrameInWindowCoordinates)
>+void PluginControllerProxy::windowAndViewFramesChanged(const WebCore::IntRect& windowFrameInScreenCoordinates, const WebCore::IntRect& viewFrameInWindowCoordinates)
> {
>     m_plugin->windowAndViewFramesChanged(windowFrameInScreenCoordinates, viewFrameInWindowCoordinates);
> }
>@@ -135,7 +133,7 @@ void PluginControllerProxy::updateLayerH
> #endif
>     }
> 
>-    m_layerHostingContext->setColorSpace(sRGBColorSpaceRef());
>+    m_layerHostingContext->setColorSpace(WebCore::sRGBColorSpaceRef());
>     m_layerHostingContext->setColorMatchUntaggedContent(true);
> 
>     m_layerHostingContext->setRootLayer(platformLayer);
>Index: Source/WebKit/PluginProcess/mac/PluginProcessMac.mm
>===================================================================
>--- Source/WebKit/PluginProcess/mac/PluginProcessMac.mm	(revision 236017)
>+++ Source/WebKit/PluginProcess/mac/PluginProcessMac.mm	(working copy)
>@@ -56,8 +56,6 @@
> #import <wtf/HashSet.h>
> #import <wtf/NeverDestroyed.h>
> 
>-using namespace WebCore;
>-
> const CFStringRef kLSPlugInBundleIdentifierKey = CFSTR("LSPlugInBundleIdentifierKey");
> 
> // These values were chosen to match default NSURLCache sizes at the time of this writing.
>@@ -401,7 +399,7 @@ static NSRunningApplication *replacedNSW
>         }
>     }
> 
>-    if (PluginProcess::singleton().launchApplicationAtURL(URL(url).string(), arguments)) {
>+    if (PluginProcess::singleton().launchApplicationAtURL(WebCore::URL(url).string(), arguments)) {
>         if (error)
>             *error = nil;
>         return nil;
>@@ -534,7 +532,7 @@ void PluginProcess::platformInitializePr
>     initializeCocoaOverrides();
> 
>     bool experimentalPlugInSandboxProfilesEnabled = parameters.extraInitializationData.get("experimental-sandbox-plugin") == "1";
>-    RuntimeEnabledFeatures::sharedFeatures().setExperimentalPlugInSandboxProfilesEnabled(experimentalPlugInSandboxProfilesEnabled);
>+    WebCore::RuntimeEnabledFeatures::sharedFeatures().setExperimentalPlugInSandboxProfilesEnabled(experimentalPlugInSandboxProfilesEnabled);
> 
>     // FIXME: It would be better to proxy SetCursor calls over to the UI process instead of
>     // allowing plug-ins to change the mouse cursor at any time.
>Index: Source/WebKit/Shared/EntryPointUtilities/mac/XPCService/XPCServiceEntryPoint.h
>===================================================================
>--- Source/WebKit/Shared/EntryPointUtilities/mac/XPCService/XPCServiceEntryPoint.h	(revision 236017)
>+++ Source/WebKit/Shared/EntryPointUtilities/mac/XPCService/XPCServiceEntryPoint.h	(working copy)
>@@ -117,7 +117,7 @@ void XPCServiceInitializer(OSObjectPtr<x
>     XPCServiceType::singleton().initialize(parameters);
> }
> 
>-int XPCServiceMain();
>+int XPCServiceMain(int, const char**);
> 
> void XPCServiceExit(OSObjectPtr<xpc_object_t>&& priorityBoostMessage);
> 
>Index: Source/WebKit/Shared/EntryPointUtilities/mac/XPCService/XPCServiceMain.mm
>===================================================================
>--- Source/WebKit/Shared/EntryPointUtilities/mac/XPCService/XPCServiceMain.mm	(revision 236017)
>+++ Source/WebKit/Shared/EntryPointUtilities/mac/XPCService/XPCServiceMain.mm	(working copy)
>@@ -92,7 +92,7 @@ static void XPCServiceEventHandler(xpc_c
>     xpc_connection_resume(peer);
> }
> 
>-int XPCServiceMain()
>+int XPCServiceMain(int, const char**)
> {
> #if defined(__i386__)
>     // FIXME: This should only be done for the 32-bit plug-in XPC service so we rely on the fact that
>@@ -160,7 +160,7 @@ int XPCServiceMain()
> 
> } // namespace WebKit
> 
>-int main(int argc, char** argv)
>+int main(int argc, const char** argv)
> {
>-    return WebKit::XPCServiceMain();
>+    return WebKit::XPCServiceMain(argc, argv);
> }
>Index: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm
>===================================================================
>--- Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm	(revision 236017)
>+++ Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm	(working copy)
>@@ -40,7 +40,6 @@
> #import "WebCookieManagerProxy.h"
> #import "WebProcessMessages.h"
> #import "WebProcessPool.h"
>-#import "XPCServiceEntryPoint.h"
> #import "_WKAutomationDelegate.h"
> #import "_WKAutomationSessionInternal.h"
> #import "_WKDownloadDelegate.h"
>@@ -182,11 +181,6 @@ + (NSURL *)_websiteDataURLForContainerWi
>     return [url URLByAppendingPathComponent:@"WebsiteData" isDirectory:YES];
> }
> 
>-+ (int)_webContentProcessXPCMain
>-{
>-    return WebKit::XPCServiceMain();
>-}
>-
> - (void)_setAllowsSpecificHTTPSCertificate:(NSArray *)certificateChain forHost:(NSString *)host
> {
>     _processPool->allowSpecificHTTPSCertificateForHost(WebKit::WebCertificateInfo::create(WebCore::CertificateInfo((__bridge CFArrayRef)certificateChain)).ptr(), host);
>Index: Source/WebKit/UIProcess/API/Cocoa/WKProcessPoolPrivate.h
>===================================================================
>--- Source/WebKit/UIProcess/API/Cocoa/WKProcessPoolPrivate.h	(revision 236017)
>+++ Source/WebKit/UIProcess/API/Cocoa/WKProcessPoolPrivate.h	(working copy)
>@@ -79,8 +79,6 @@
> 
> - (void)_registerURLSchemeAsCanDisplayOnlyIfCanRequest:(NSString *)scheme WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> 
>-+ (int)_webContentProcessXPCMain WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));
>-
> // Test only. Should be called only while no web content processes are running.
> - (void)_terminateStorageProcess;
> - (void)_terminateNetworkProcess;
>Index: Source/WebKit/WebKit.xcodeproj/project.pbxproj
>===================================================================
>--- Source/WebKit/WebKit.xcodeproj/project.pbxproj	(revision 236017)
>+++ Source/WebKit/WebKit.xcodeproj/project.pbxproj	(working copy)
>@@ -3428,6 +3428,8 @@
> 		5C62FDF81EFC263C00CE072E /* WKURLSchemeTaskPrivate.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = WKURLSchemeTaskPrivate.h; sourceTree = "<group>"; };
> 		5C6CE6D01F59BC460007C6CB /* PageClientImplCocoa.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = PageClientImplCocoa.mm; sourceTree = "<group>"; };
> 		5C6CE6D31F59EA350007C6CB /* PageClientImplCocoa.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PageClientImplCocoa.h; sourceTree = "<group>"; };
>+		5C74300E21500492004BFA17 /* WKWebProcess.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WKWebProcess.h; sourceTree = "<group>"; };
>+		5C74300F21500492004BFA17 /* WKWebProcess.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WKWebProcess.cpp; sourceTree = "<group>"; };
> 		5C7706731D111D8B0012700F /* WebSocketProvider.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = WebSocketProvider.cpp; path = Network/WebSocketProvider.cpp; sourceTree = "<group>"; };
> 		5C7C88DC1D0F41A0009D2F6D /* WebSocketProvider.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = WebSocketProvider.h; path = Network/WebSocketProvider.h; sourceTree = "<group>"; };
> 		5C84CF901F96AC4E00B6705A /* NetworkSessionCreationParameters.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = NetworkSessionCreationParameters.h; sourceTree = "<group>"; };
>@@ -6811,6 +6813,23 @@
> 			path = Cocoa;
> 			sourceTree = "<group>";
> 		};
>+		5C74300C21500434004BFA17 /* API */ = {
>+			isa = PBXGroup;
>+			children = (
>+				5C74300D21500442004BFA17 /* Cocoa */,
>+			);
>+			path = API;
>+			sourceTree = "<group>";
>+		};
>+		5C74300D21500442004BFA17 /* Cocoa */ = {
>+			isa = PBXGroup;
>+			children = (
>+				5C74300F21500492004BFA17 /* WKWebProcess.cpp */,
>+				5C74300E21500492004BFA17 /* WKWebProcess.h */,
>+			);
>+			path = Cocoa;
>+			sourceTree = "<group>";
>+		};
> 		6BE969BF1E54D452008B7483 /* ResourceLoadStatistics */ = {
> 			isa = PBXGroup;
> 			children = (
>@@ -7048,6 +7067,7 @@
> 		BC032D5C10F436D50058C15A /* WebProcess */ = {
> 			isa = PBXGroup;
> 			children = (
>+				5C74300C21500434004BFA17 /* API */,
> 				1AB1F7701D1B2F5D007C9BD1 /* ApplePay */,
> 				1C0A19431C8FF1A800FE0EBB /* Automation */,
> 				41D129D81F3D101400D15E47 /* Cache */,
>Index: Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp
>===================================================================
>--- Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp	(nonexistent)
>+++ Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp	(working copy)
>@@ -0,0 +1,34 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#include "config.h"
>+#include "WKWebProcess.h"
>+
>+#include "XPCServiceEntryPoint.h"
>+
>+int WKWebProcessMain(int argc, const char** argv)
>+{
>+    return WebKit::XPCServiceMain(argc, argv);
>+}
>Index: Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h
>===================================================================
>--- Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h	(nonexistent)
>+++ Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h	(working copy)
>@@ -0,0 +1,38 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#pragma once
>+
>+#include <WebKit/WKFoundation.h>
>+
>+#ifdef __cplusplus
>+extern "C" {
>+#endif
>+
>+WK_EXPORT int WKWebProcessMain(int argc, const char** argv) WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));
>+
>+#ifdef __cplusplus
>+}
>+#endif
>Index: Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp
>===================================================================
>--- Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp	(nonexistent)
>+++ Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp	(working copy)
>@@ -0,0 +1,34 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#include "config.h"
>+#include "WKWebProcess.h"
>+
>+#include "XPCServiceEntryPoint.h"
>+
>+int WKWebProcessMain(int argc, const char** argv)
>+{
>+    return WebKit::XPCServiceMain(argc, argv);
>+}
>Index: Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h
>===================================================================
>--- Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h	(nonexistent)
>+++ Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h	(working copy)
>@@ -0,0 +1,38 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#pragma once
>+
>+#include <WebKit/WKFoundation.h>
>+
>+#ifdef __cplusplus
>+extern "C" {
>+#endif
>+
>+WK_EXPORT int WKWebProcessMain(int argc, const char** argv) WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));
>+
>+#ifdef __cplusplus
>+}
>+#endif
>Index: Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp
>===================================================================
>--- Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp	(nonexistent)
>+++ Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.cpp	(working copy)
>@@ -0,0 +1,34 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#include "config.h"
>+#include "WKWebProcess.h"
>+
>+#include "XPCServiceEntryPoint.h"
>+
>+int WKWebProcessMain(int argc, const char** argv)
>+{
>+    return WebKit::XPCServiceMain(argc, argv);
>+}
>Index: Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h
>===================================================================
>--- Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h	(nonexistent)
>+++ Source/WebKit/WebProcess/API/Cocoa/WKWebProcess.h	(working copy)
>@@ -0,0 +1,38 @@
>+/*
>+ * Copyright (C) 2018 Apple Inc. All rights reserved.
>+ *
>+ * Redistribution and use in source and binary forms, with or without
>+ * modification, are permitted provided that the following conditions
>+ * are met:
>+ * 1. Redistributions of source code must retain the above copyright
>+ *    notice, this list of conditions and the following disclaimer.
>+ * 2. Redistributions in binary form must reproduce the above copyright
>+ *    notice, this list of conditions and the following disclaimer in the
>+ *    documentation and/or other materials provided with the distribution.
>+ *
>+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
>+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
>+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
>+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
>+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
>+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
>+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
>+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
>+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
>+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
>+ * THE POSSIBILITY OF SUCH DAMAGE.
>+ */
>+
>+#pragma once
>+
>+#include <WebKit/WKFoundation.h>
>+
>+#ifdef __cplusplus
>+extern "C" {
>+#endif
>+
>+WK_EXPORT int WKWebProcessMain(int argc, const char** argv) WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));
>+
>+#ifdef __cplusplus
>+}
>+#endif
>Index: Source/WebKit/WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm
>===================================================================
>--- Source/WebKit/WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm	(revision 236017)
>+++ Source/WebKit/WebProcess/WebPage/mac/WKAccessibilityWebPageObjectMac.mm	(working copy)
>@@ -48,9 +48,6 @@
> #import <pal/spi/mac/NSAccessibilitySPI.h>
> #import <wtf/ObjcRuntimeExtras.h>
> 
>-using namespace WebCore;
>-using namespace WebKit;
>-
> @implementation WKAccessibilityWebPageObject
> 
> - (void)dealloc
>@@ -100,7 +97,7 @@ - (void)accessibilitySetValue:(id)value 
> 
> - (NSPoint)convertScreenPointToRootView:(NSPoint)point
> {
>-    return m_page->screenToRootView(IntPoint(point.x, point.y));
>+    return m_page->screenToRootView(WebCore::IntPoint(point.x, point.y));
> }
> 
> - (NSArray *)accessibilityActionNames
>@@ -152,8 +149,8 @@ - (id)accessibilityAttributeValue:(NSStr
>         return [[self accessibilityRootObjectWrapper] accessibilityAttributeValue:attribute];
>     
>     if ([attribute isEqualToString:NSAccessibilitySizeAttribute]) {
>-        const IntSize& s = m_page->size();
>-        return [NSValue valueWithSize:NSMakeSize(s.width(), s.height())];
>+        const auto& size = m_page->size();
>+        return [NSValue valueWithSize:NSMakeSize(size.width(), size.height())];
>     }
>     
>     if ([attribute isEqualToString:NSAccessibilityChildrenAttribute])
>@@ -198,11 +195,11 @@ - (id)accessibilityHitTest:(NSPoint)poin
>     if (!m_page)
>         return nil;
> 
>-    IntPoint convertedPoint = m_page->screenToRootView(IntPoint(point));
>+    auto convertedPoint = m_page->screenToRootView(WebCore::IntPoint(point));
>     
>     // Some plugins may be able to figure out the scroll position and inset on their own.
>     bool applyContentOffset = true;
>-    if (auto pluginView = WebPage::pluginViewForFrame(m_page->mainFrame()))
>+    if (auto pluginView = WebKit::WebPage::pluginViewForFrame(m_page->mainFrame()))
>         applyContentOffset = !pluginView->plugin()->pluginHandlesContentOffsetForAccessibilityHitTest();
> 
>     if (applyContentOffset) {

Actions: View | Formatted Diff | Diff

Attachments on bug 189636: 349814 | 349832 | 349852 | 349864 | 349887 | 349894