Attachment 370174 Details for Bug 198010 – Patch

[patch] Patch

bug-198010-20190517182228.patch (text/plain), 9.78 KB, created by Myles C. Maxfield on 2019-05-17 18:22:29 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Myles C. Maxfield

Created: 2019-05-17 18:22:29 PDT

Size: 9.78 KB

patch

obsolete

>Subversion Revision: 245483
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 8f32ef9a20a1a6e6225a1222254193c42986cfa4..70c326125bca301ff3c6e5b80e109047107df9c7 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-05-17  Myles C. Maxfield  <mmaxfield@apple.com>
>+
>+        [Mac] Add API test to make sure the system font is drawn just like in native code
>+        https://bugs.webkit.org/show_bug.cgi?id=198010
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        We can compare the width of using font-family:system-ui to a CTLine.
>+
>+        * TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
>+        * TestWebKitAPI/Tests/WebKitCocoa/TextWidth.html: Added.
>+        * TestWebKitAPI/Tests/WebKitCocoa/TextWidth.mm: Added.
>+        (TEST):
>+
> 2019-05-17  Alex Christensen  <achristensen@webkit.org>
> 
>         Add SPI to set a list of hosts to which to send custom header fields cross-origin
>diff --git a/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj b/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
>index df27f125c293d390c47703960b243293f7f6f055..00464ba2322e76d1a728c2ce3f6b1b94b0449686 100644
>--- a/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
>+++ b/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj
>@@ -758,6 +758,8 @@
> 		C0ADBE9612FCA79B00D2C129 /* simple-form.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = C0ADBE8412FCA6B600D2C129 /* simple-form.html */; };
> 		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 */; };
>+		C22FA32B228F8708009D7988 /* TextWidth.mm in Sources */ = {isa = PBXBuildFile; fileRef = C22FA32A228F8708009D7988 /* TextWidth.mm */; };
>+		C22FA32D228F8AEB009D7988 /* TextWidth.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = C22FA32C228F877A009D7988 /* TextWidth.html */; };
> 		C25CCA061E51380B0026CB8A /* LineBreaking.mm in Sources */ = {isa = PBXBuildFile; fileRef = C25CCA051E51380B0026CB8A /* LineBreaking.mm */; };
> 		C25CCA0B1E5140C10026CB8A /* LineBreaking.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = C25CCA0A1E513F490026CB8A /* LineBreaking.html */; };
> 		C25CCA0D1E5141840026CB8A /* AllAhem.svg in Copy Resources */ = {isa = PBXBuildFile; fileRef = C25CCA0C1E5140E50026CB8A /* AllAhem.svg */; };
>@@ -1315,6 +1317,7 @@
> 				F4CD74C620FDACFA00DE3794 /* text-with-async-script.html in Copy Resources */,
> 				F44C7A0520FAAE3C0014478C /* text-with-deferred-script.html in Copy Resources */,
> 				F41AB9AA1EF4696B0083FA08 /* textarea-to-input.html in Copy Resources */,
>+				C22FA32D228F8AEB009D7988 /* TextWidth.html in Copy Resources */,
> 				F4451C761EB8FD890020C5DA /* two-paragraph-contenteditable.html in Copy Resources */,
> 				C540F784152E5A9A00A40C8C /* verboseMarkup.html in Copy Resources */,
> 				CD57779D211CE91F001B371E /* video-with-audio-and-web-audio.html in Copy Resources */,
>@@ -2108,6 +2111,8 @@
> 		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>"; };
> 		C1D8EE212028E8E3008EB141 /* WebProcessTerminate.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebProcessTerminate.mm; sourceTree = "<group>"; };
>+		C22FA32A228F8708009D7988 /* TextWidth.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = TextWidth.mm; sourceTree = "<group>"; };
>+		C22FA32C228F877A009D7988 /* TextWidth.html */ = {isa = PBXFileReference; lastKnownFileType = text.html; path = TextWidth.html; sourceTree = "<group>"; };
> 		C25CCA051E51380B0026CB8A /* LineBreaking.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; name = LineBreaking.mm; path = ../WebCore/LineBreaking.mm; sourceTree = "<group>"; };
> 		C25CCA0A1E513F490026CB8A /* LineBreaking.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = LineBreaking.html; sourceTree = "<group>"; };
> 		C25CCA0C1E5140E50026CB8A /* AllAhem.svg */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = AllAhem.svg; sourceTree = "<group>"; };
>@@ -2697,6 +2702,7 @@
> 				5774AA6721FBBF7800AF2A1B /* TestLoadOptimizer.mm */,
> 				F4CD74C720FDB49600DE3794 /* TestURLSchemeHandler.h */,
> 				F4CD74C820FDB49600DE3794 /* TestURLSchemeHandler.mm */,
>+				C22FA32A228F8708009D7988 /* TextWidth.mm */,
> 				5CB40B4D1F4B98BE007DC7B9 /* UIDelegate.mm */,
> 				7CC3E1FA197E234100BE6252 /* UserContentController.mm */,
> 				7C882E031C80C624006BF731 /* UserContentWorld.mm */,
>@@ -3091,6 +3097,7 @@
> 				F4CD74C520FDACF500DE3794 /* text-with-async-script.html */,
> 				F44C7A0420FAAE320014478C /* text-with-deferred-script.html */,
> 				F41AB9951EF4692C0083FA08 /* textarea-to-input.html */,
>+				C22FA32C228F877A009D7988 /* TextWidth.html */,
> 				F4451C751EB8FD7C0020C5DA /* two-paragraph-contenteditable.html */,
> 				CD57779B211CE6CE001B371E /* video-with-audio-and-web-audio.html */,
> 				CD577798211CDE8F001B371E /* web-audio-only.html */,
>@@ -4401,6 +4408,7 @@
> 				F45033F5206BEC95009351CE /* TextAutosizingBoost.mm in Sources */,
> 				93E6193B1F931B3A00AF245E /* TextCodec.cpp in Sources */,
> 				CE3524F91B1441C40028A7C5 /* TextFieldDidBeginAndEndEditing.cpp in Sources */,
>+				C22FA32B228F8708009D7988 /* TextWidth.mm in Sources */,
> 				7CCE7EDD1A411A9200447C4C /* TimeRanges.cpp in Sources */,
> 				7CCE7ED31A411A7E00447C4C /* TypingStyleCrash.mm in Sources */,
> 				57152B7821DD4E8D000C37CA /* U2fCommandConstructorTest.cpp in Sources */,
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextWidth.html b/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextWidth.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..eaf800a0ba4fcf3efb9d652a67d63b4715e72306
>--- /dev/null
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextWidth.html
>@@ -0,0 +1,23 @@
>+<!DOCTYPE html>
>+<html lang="en-US">
>+<head>
>+<style>
>+body {
>+    width: 99999999px;
>+}
>+#test1 {
>+    font: 24px "system-ui";
>+}
>+</style>
>+<script>
>+function runTest1() {
>+    return document.getElementById("test1").offsetWidth;
>+}
>+</script>
>+</head>
>+<body>
>+<p>
>+<span id="test1">This is a test string</span>
>+</p>
>+</body>
>+</html>
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextWidth.mm b/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextWidth.mm
>new file mode 100644
>index 0000000000000000000000000000000000000000..73cae4bfbf8ab23f51325eca58ae969905e68b5c
>--- /dev/null
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextWidth.mm
>@@ -0,0 +1,55 @@
>+/*
>+ * Copyright (C) 2019 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"
>+
>+#import "TestNavigationDelegate.h"
>+#import "Utilities.h"
>+#import <wtf/RetainPtr.h>
>+#import <wtf/text/WTFString.h>
>+
>+TEST(WebKit, TextWidth)
>+{
>+    RetainPtr<WKWebViewConfiguration> configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
>+    RetainPtr<WKWebView> webView = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:configuration.get()]);
>+
>+    [webView loadRequest:[NSURLRequest requestWithURL:[[NSBundle mainBundle] URLForResource:@"TextWidth" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"]]];
>+    [webView _test_waitForDidFinishNavigation];
>+    
>+    __block bool didEvaluateJavaScript = false;
>+    __block float webKitWidth;
>+    [webView evaluateJavaScript:@"runTest1()" completionHandler:^(id value, NSError *error) {
>+        webKitWidth = [(NSNumber *)value floatValue];
>+        didEvaluateJavaScript = true;
>+    }];
>+    TestWebKitAPI::Util::run(&didEvaluateJavaScript);
>+
>+    auto font = adoptCF(CTFontCreateUIFontForLanguage(kCTFontUIFontSystem, 24, static_cast<CFStringRef>(@"en-US")));
>+    NSAttributedString *attributedString = [[NSAttributedString alloc] initWithString:@"This is a test string" attributes:@{NSFontAttributeName : static_cast<NSFont *>(font.get())}];
>+    auto line = adoptCF(CTLineCreateWithAttributedString(static_cast<CFAttributedStringRef>(attributedString)));
>+    double coreTextWidth = CTLineGetTypographicBounds(line.get(), nullptr, nullptr, nullptr);
>+
>+    EXPECT_NEAR(webKitWidth, coreTextWidth, 3);
>+}

Flags:

dbates: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 198010: 370174 | 370359 | 370414 | 370438