Title: [261479] trunk

Diff

Modified: trunk/Source/WebCore/ChangeLog (261478 => 261479)


--- trunk/Source/WebCore/ChangeLog	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebCore/ChangeLog	2020-05-11 15:43:00 UTC (rev 261479)
@@ -1,3 +1,16 @@
+2020-05-11  Per Arne Vollan  <pvol...@apple.com>
+
+        Unreviewed, reverting r261296.
+
+        Rolling r260769 back in, since this was not causing a
+        regression.
+
+        Reverted changeset:
+
+        "Unreviewed, reverting r260769."
+        https://bugs.webkit.org/show_bug.cgi?id=211578
+        https://trac.webkit.org/changeset/261296
+
 2020-05-11  Zalan Bujtas  <za...@apple.com>
 
         [LFC][TFC] Introduce GridSpace

Modified: trunk/Source/WebCore/platform/MIMETypeRegistry.cpp (261478 => 261479)


--- trunk/Source/WebCore/platform/MIMETypeRegistry.cpp	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebCore/platform/MIMETypeRegistry.cpp	2020-05-11 15:43:00 UTC (rev 261479)
@@ -270,12 +270,6 @@
     return unsupportedTextMIMETypes;
 }
 
-Optional<HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>>& overriddenMimeTypesMap()
-{
-    static NeverDestroyed<Optional<HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>>> map;
-    return map;
-}
-
 const std::initializer_list<TypeExtensionPair>& commonMediaTypes()
 {
     // A table of common media MIME types and file extensions used when a platform's
@@ -362,7 +356,7 @@
     return commonMediaTypes;
 }
 
-const HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>& commonMimeTypesMap()
+static const HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>& commonMimeTypesMap()
 {
     ASSERT(isMainThread());
     static NeverDestroyed<HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>> mimeTypesMap = [] {
@@ -387,12 +381,6 @@
 
 static const Vector<String>* typesForCommonExtension(const String& extension)
 {
-    if (overriddenMimeTypesMap().hasValue()) {
-        auto mapEntry = overriddenMimeTypesMap()->find(extension);
-        if (mapEntry == overriddenMimeTypesMap()->end())
-            return nullptr;
-        return &mapEntry->value;
-    }
     auto mapEntry = commonMimeTypesMap().find(extension);
     if (mapEntry == commonMimeTypesMap().end())
         return nullptr;

Modified: trunk/Source/WebCore/platform/MIMETypeRegistry.h (261478 => 261479)


--- trunk/Source/WebCore/platform/MIMETypeRegistry.h	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebCore/platform/MIMETypeRegistry.h	2020-05-11 15:43:00 UTC (rev 261479)
@@ -31,9 +31,6 @@
 
 namespace WebCore {
 
-WEBCORE_EXPORT Optional<HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>>& overriddenMimeTypesMap();
-WEBCORE_EXPORT const HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>& commonMimeTypesMap();
-
 struct TypeExtensionPair {
     ASCIILiteral type;
     ASCIILiteral extension;

Modified: trunk/Source/WebCore/testing/Internals.cpp (261478 => 261479)


--- trunk/Source/WebCore/testing/Internals.cpp	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebCore/testing/Internals.cpp	2020-05-11 15:43:00 UTC (rev 261479)
@@ -119,7 +119,6 @@
 #include "LibWebRTCProvider.h"
 #include "LoaderStrategy.h"
 #include "Location.h"
-#include "MIMETypeRegistry.h"
 #include "MallocStatistics.h"
 #include "MediaDevices.h"
 #include "MediaEngineConfigurationFactory.h"
@@ -5667,11 +5666,6 @@
 }
 #endif
 
-String Internals::mediaMIMETypeForExtension(const String& extension)
-{
-    return MIMETypeRegistry::getMediaMIMETypeForExtension(extension);
-}
-
 bool Internals::supportsPictureInPicture()
 {
     return WebCore::supportsPictureInPicture();

Modified: trunk/Source/WebCore/testing/Internals.h (261478 => 261479)


--- trunk/Source/WebCore/testing/Internals.h	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebCore/testing/Internals.h	2020-05-11 15:43:00 UTC (rev 261479)
@@ -1001,8 +1001,6 @@
     int readPreferenceInteger(const String& domain, const String& key);
     String encodedPreferenceValue(const String& domain, const String& key);
 
-    String mediaMIMETypeForExtension(const String& extension);
-
     String getUTIFromTag(const String& tagClass, const String& tag, const String& conformingToUTI);
 
     bool supportsPictureInPicture();

Modified: trunk/Source/WebCore/testing/Internals.idl (261478 => 261479)


--- trunk/Source/WebCore/testing/Internals.idl	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebCore/testing/Internals.idl	2020-05-11 15:43:00 UTC (rev 261479)
@@ -900,8 +900,6 @@
     long readPreferenceInteger(DOMString domain, DOMString key);
     DOMString encodedPreferenceValue(DOMString domain, DOMString key);
 
-    DOMString mediaMIMETypeForExtension(DOMString extension);
-
     DOMString getUTIFromTag(DOMString tagClass, DOMString tag, DOMString conformingToUTI);
 
     boolean supportsPictureInPicture();

Modified: trunk/Source/WebKit/ChangeLog (261478 => 261479)


--- trunk/Source/WebKit/ChangeLog	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebKit/ChangeLog	2020-05-11 15:43:00 UTC (rev 261479)
@@ -1,3 +1,16 @@
+2020-05-11  Per Arne Vollan  <pvol...@apple.com>
+
+        Unreviewed, reverting r261296.
+
+        Rolling r260769 back in, since this was not causing a
+        regression.
+
+        Reverted changeset:
+
+        "Unreviewed, reverting r260769."
+        https://bugs.webkit.org/show_bug.cgi?id=211578
+        https://trac.webkit.org/changeset/261296
+
 2020-05-11  Wenson Hsieh  <wenson_hs...@apple.com>
 
         Unreviewed, fix the watchOS build after r261457

Modified: trunk/Source/WebKit/Shared/WebProcessCreationParameters.cpp (261478 => 261479)


--- trunk/Source/WebKit/Shared/WebProcessCreationParameters.cpp	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebKit/Shared/WebProcessCreationParameters.cpp	2020-05-11 15:43:00 UTC (rev 261479)
@@ -173,7 +173,6 @@
     encoder << neSessionManagerExtensionHandle;
     encoder << mapDBExtensionHandle;
     encoder << systemHasBattery;
-    encoder << mimeTypesMap;
 #endif
 
 #if PLATFORM(IOS_FAMILY)
@@ -488,12 +487,6 @@
     if (!systemHasBattery)
         return false;
     parameters.systemHasBattery = WTFMove(*systemHasBattery);
-
-    Optional<Optional<HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>>> mimeTypesMap;
-    decoder >> mimeTypesMap;
-    if (!mimeTypesMap)
-        return false;
-    parameters.mimeTypesMap = WTFMove(*mimeTypesMap);
 #endif
 
 #if PLATFORM(IOS_FAMILY)

Modified: trunk/Source/WebKit/Shared/WebProcessCreationParameters.h (261478 => 261479)


--- trunk/Source/WebKit/Shared/WebProcessCreationParameters.h	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebKit/Shared/WebProcessCreationParameters.h	2020-05-11 15:43:00 UTC (rev 261479)
@@ -213,7 +213,6 @@
     Optional<SandboxExtension::Handle> neSessionManagerExtensionHandle;
     Optional<SandboxExtension::Handle> mapDBExtensionHandle;
     bool systemHasBattery { false };
-    Optional<HashMap<String, Vector<String>, ASCIICaseInsensitiveHash>> mimeTypesMap;
 #endif
 
 #if PLATFORM(IOS_FAMILY)

Modified: trunk/Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm (261478 => 261479)


--- trunk/Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm	2020-05-11 15:43:00 UTC (rev 261479)
@@ -51,7 +51,6 @@
 #import <WebCore/AGXCompilerService.h>
 #import <WebCore/Color.h>
 #import <WebCore/LocalizedDeviceModel.h>
-#import <WebCore/MIMETypeRegistry.h>
 #import <WebCore/NetworkStorageSession.h>
 #import <WebCore/NotImplemented.h>
 #import <WebCore/PictureInPictureSupport.h>
@@ -414,7 +413,6 @@
         parameters.neSessionManagerExtensionHandle = WTFMove(managerHandle);
     }
     parameters.systemHasBattery = systemHasBattery();
-    parameters.mimeTypesMap = commonMimeTypesMap();
 
     SandboxExtension::Handle mapDBHandle;
     if (SandboxExtension::createHandleForMachLookup("com.apple.lsd.mapdb", WTF::nullopt, mapDBHandle, SandboxExtension::Flags::NoReport))

Modified: trunk/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm (261478 => 261479)


--- trunk/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm	2020-05-11 15:43:00 UTC (rev 261479)
@@ -61,7 +61,6 @@
 #import <WebCore/LocalizedDeviceModel.h>
 #import <WebCore/LocalizedStrings.h>
 #import <WebCore/LogInitialization.h>
-#import <WebCore/MIMETypeRegistry.h>
 #import <WebCore/MemoryRelease.h>
 #import <WebCore/NSScrollerImpDetails.h>
 #import <WebCore/NetworkExtensionContentFilter.h>
@@ -349,9 +348,6 @@
 
     setSystemHasBattery(parameters.systemHasBattery);
 
-    if (parameters.mimeTypesMap)
-        overriddenMimeTypesMap() = WTFMove(parameters.mimeTypesMap);
-
 #if PLATFORM(IOS_FAMILY)
     RenderThemeIOS::setCSSValueToSystemColorMap(WTFMove(parameters.cssValueToSystemColorMap));
     RenderThemeIOS::setFocusRingColor(parameters.focusRingColor);

Modified: trunk/Tools/ChangeLog (261478 => 261479)


--- trunk/Tools/ChangeLog	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Tools/ChangeLog	2020-05-11 15:43:00 UTC (rev 261479)
@@ -1,3 +1,16 @@
+2020-05-11  Per Arne Vollan  <pvol...@apple.com>
+
+        Unreviewed, reverting r261296.
+
+        Rolling r260769 back in, since this was not causing a
+        regression.
+
+        Reverted changeset:
+
+        "Unreviewed, reverting r260769."
+        https://bugs.webkit.org/show_bug.cgi?id=211578
+        https://trac.webkit.org/changeset/261296
+
 2020-05-11  Philippe Normand  <pnorm...@igalia.com>
 
         [GTK] Media and audio permission request unit tests are timing out

Modified: trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj (261478 => 261479)


--- trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj	2020-05-11 15:43:00 UTC (rev 261479)
@@ -898,7 +898,6 @@
 		C0BD669F131D3CFF00E18F2A /* ResponsivenessTimerDoesntFireEarly_Bundle.cpp in Sources */ = {isa = PBXBuildFile; fileRef = C0BD669E131D3CFF00E18F2A /* ResponsivenessTimerDoesntFireEarly_Bundle.cpp */; };
 		C0C5D3C61459912900A802A6 /* GetBackingScaleFactor_Bundle.mm in Sources */ = {isa = PBXBuildFile; fileRef = C0C5D3BD14598B6F00A802A6 /* GetBackingScaleFactor_Bundle.mm */; };
 		C13D82D92416F13200A62793 /* EnableAccessibility.mm in Sources */ = {isa = PBXBuildFile; fileRef = C13D82D82416F13200A62793 /* EnableAccessibility.mm */; };
-		C145CC0C23DA5A1F003A5EEB /* MimeTypes.mm in Sources */ = {isa = PBXBuildFile; fileRef = C145CC0B23DA5A0F003A5EEB /* MimeTypes.mm */; };
 		C149D550242E98DF003EBB12 /* SleepDisabler.mm in Sources */ = {isa = PBXBuildFile; fileRef = C149D54F242E9844003EBB12 /* SleepDisabler.mm */; };
 		C15CBB3023F1FF1A00300CC7 /* BacklightLevelNotification.mm in Sources */ = {isa = PBXBuildFile; fileRef = C15CBB2F23F1FF1A00300CC7 /* BacklightLevelNotification.mm */; };
 		C15CBB3F23FB177A00300CC7 /* PreferenceChanges.mm in Sources */ = {isa = PBXBuildFile; fileRef = C15CBB3E23FB177A00300CC7 /* PreferenceChanges.mm */; };
@@ -2512,7 +2511,6 @@
 		C0C5D3BC14598B6F00A802A6 /* GetBackingScaleFactor.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = GetBackingScaleFactor.mm; sourceTree = "<group>"; };
 		C0C5D3BD14598B6F00A802A6 /* GetBackingScaleFactor_Bundle.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = GetBackingScaleFactor_Bundle.mm; sourceTree = "<group>"; };
 		C13D82D82416F13200A62793 /* EnableAccessibility.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = EnableAccessibility.mm; sourceTree = "<group>"; };
-		C145CC0B23DA5A0F003A5EEB /* MimeTypes.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = MimeTypes.mm; sourceTree = "<group>"; };
 		C149D54F242E9844003EBB12 /* SleepDisabler.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = SleepDisabler.mm; sourceTree = "<group>"; };
 		C15CBB2F23F1FF1A00300CC7 /* BacklightLevelNotification.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = BacklightLevelNotification.mm; sourceTree = "<group>"; };
 		C15CBB3E23FB177A00300CC7 /* PreferenceChanges.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = PreferenceChanges.mm; sourceTree = "<group>"; };
@@ -2986,7 +2984,6 @@
 				E394AE6E23F2303E005B4936 /* GrantAccessToMobileAssets.mm */,
 				E34A6D182412DE390012AB6E /* GrantAccessToPreferencesService.mm */,
 				E35B908123F60DD0000011FF /* LocalizedDeviceModel.mm */,
-				C145CC0B23DA5A0F003A5EEB /* MimeTypes.mm */,
 				E325C90623E3870200BC7D3B /* PictureInPictureSupport.mm */,
 				C15CBB3E23FB177A00300CC7 /* PreferenceChanges.mm */,
 				C149D54F242E9844003EBB12 /* SleepDisabler.mm */,
@@ -5037,7 +5034,6 @@
 				5C0BF8941DD599C900B00328 /* MenuTypesForMouseEvents.mm in Sources */,
 				5165FE04201EE620009F7EC3 /* MessagePortProviders.mm in Sources */,
 				A5B149DE1F5A19EA00C6DAFF /* MIMETypeRegistry.cpp in Sources */,
-				C145CC0C23DA5A1F003A5EEB /* MimeTypes.mm in Sources */,
 				51CD1C6C1B38CE4300142CA5 /* ModalAlerts.mm in Sources */,
 				7C83E0B61D0A64B300FEBCF3 /* ModalAlertsSPI.cpp in Sources */,
 				7CCE7F011A411AE600447C4C /* MouseMoveAfterCrash.cpp in Sources */,

Deleted: trunk/Tools/TestWebKitAPI/Tests/WebKit/MimeTypes.mm (261478 => 261479)


--- trunk/Tools/TestWebKitAPI/Tests/WebKit/MimeTypes.mm	2020-05-11 14:59:28 UTC (rev 261478)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit/MimeTypes.mm	2020-05-11 15:43:00 UTC (rev 261479)
@@ -1,52 +0,0 @@
-/*
- * Copyright (C) 2020 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.
- */
-
-#import "config.h"
-
-#if WK_HAVE_C_SPI
-
-#import "PlatformUtilities.h"
-#import "TestWKWebView.h"
-#import <WebCore/MIMETypeRegistry.h>
-
-TEST(WebKit, MimeTypes)
-{
-    auto configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
-    WKRetainPtr<WKContextRef> context = adoptWK(TestWebKitAPI::Util::createContextForInjectedBundleTest("InternalsInjectedBundleTest"));
-    configuration.get().processPool = (WKProcessPool *)context.get();
-    auto webView = adoptNS([[TestWKWebView alloc] initWithFrame:NSMakeRect(0, 0, 300, 300) configuration:configuration.get() addToWindow:YES]);
-
-    for (auto& pair : WebCore::commonMediaTypes()) {
-        ASCIILiteral extension = pair.extension;
-
-        auto js = [NSString stringWithFormat:@"window.internals.mediaMIMETypeForExtension(\"%s\")", extension.characters()];
-
-        auto mimeType = [webView stringByEvaluatingJavaScript:js];
-
-        ASSERT_TRUE(WebCore::MIMETypeRegistry::getMediaMIMETypeForExtension(extension) == String(mimeType));
-    }
-}
-
-#endif // WK_HAVE_C_SPI
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to